Polish/fixes for productseries/+setbranch
Bug #559200 reported by
Brad Crittenden
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
High
|
Brad Crittenden |
Bug Description
The branch for bug 524302 landed with some deficiencies that now need to be cleaned up:
* If the Branch name already exists, it appears as if the form does nothing.
* If there is an existing imported SVN branch on a given URL, the form will give you the correct error message, but if there is an existing imported BZR branch on a given URL, it will give you an exception.
* There is no test to verify the script is loaded on the page. Either show that the script is in the page or use windmill to verify it executed.
Related branches
lp:~bac/launchpad/bug-559200
Merged
into
lp:launchpad
- Aaron Bentley (community): Approve
-
Diff: 192 lines (+130/-15)3 files modifiedlib/lp/code/windmill/tests/test_productseries_setbranch.py (+58/-0)
lib/lp/registry/browser/productseries.py (+34/-15)
lib/lp/registry/browser/tests/productseries-setbranch-view.txt (+38/-0)
Changed in launchpad-registry: | |
status: | New → In Progress |
importance: | Undecided → High |
assignee: | nobody → Brad Crittenden (bac) |
milestone: | none → 10.04 |
Changed in launchpad-registry: | |
status: | In Progress → Fix Committed |
tags: |
added: qa-ok removed: qa-needstesting |
To post a comment you must log in.
Fixed in stable r10697 <http:// bazaar. launchpad. net/~launchpad- pqm/launchpad/ stable/ revision/ 10697>