Archiva

should not be possible to create a proxy connector if there are no remote repositories

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.0.1
  • Fix Version/s: 1.1.2
  • Component/s: Web Interface
  • Labels:
    None
  • Number of attachments :
    1

Description

1) start a new archiva instance
2) delete all remote repositories
3) create a proxy connector (the remote repo list is empty)
4) add the connector

steps 3 and 4 should not be possible

Activity

Hide
Mark Jeffery added a comment -

This patch disables the Add link if there are no remote repositories defined.
It states so to the user

Show
Mark Jeffery added a comment - This patch disables the Add link if there are no remote repositories defined. It states so to the user
Hide
Mark Jeffery added a comment -

Howzit guys
I am just finding out if there are any pre-requisites that I missed when submitting the patch ?
I would really like to contribute to Archiva because I think it is a really useful project. If there was anything that I missed, please could you let me know here so that I know what to take into consideration for the next patch.
Thanks Mark

Show
Mark Jeffery added a comment - Howzit guys I am just finding out if there are any pre-requisites that I missed when submitting the patch ? I would really like to contribute to Archiva because I think it is a really useful project. If there was anything that I missed, please could you let me know here so that I know what to take into consideration for the next patch. Thanks Mark
Hide
James William Dumay added a comment -

Hey Mark, thanks heaps for the patch! We are always looking for new contributers

Are you able to fix up the formatting of that patch and write some unit tests for your modifications to the ProxyConnectorsAction ?

Otherwise, this patch is good to go

Come join us on IRC at irc.codehaus.org #archiva (if you have not already)

Cheers
James

Show
James William Dumay added a comment - Hey Mark, thanks heaps for the patch! We are always looking for new contributers Are you able to fix up the formatting of that patch and write some unit tests for your modifications to the ProxyConnectorsAction ? Otherwise, this patch is good to go Come join us on IRC at irc.codehaus.org #archiva (if you have not already) Cheers James
Hide
Maria Odea Ching added a comment -

Patch applied in -r693737 1.1.x branch. Thanks again for the patch Mark!

Show
Maria Odea Ching added a comment - Patch applied in -r693737 1.1.x branch. Thanks again for the patch Mark!

People

Vote (1)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: