Launchpad itself

Changing a branch's settings defaults to changing owner as well

Bug #575086 reported by Soren Hansen on 2010-05-04
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
​ Launchpad itself
Fix Released
High
Tim Penhey
Launchpad itself 10.05
Also affects project (?) Also affects distribution/package Nominate for series

Bug Description

If I go to e.g. https://code.edge.launchpad.net/~ubuntu-branches/ubuntu/maverick/ssss/maverick/+edit the "Owner" setting defaults to me, so unless I'm very attentive (which I wasn't), I'll accidentally steal the branch when all I really wanted to do was change the status of the branch.

Related branches

lp:~thumper/launchpad/package-branch-edit-owner
Jonathan Lange (community): Approve on 2010-05-24
Diff: 243 lines (+103/-28)
5 files modified
lib/lp/code/browser/branch.py (+23/-0)
lib/lp/code/stories/branches/xx-branch-edit.txt (+47/-1)
lib/lp/code/tests/helpers.py (+28/-2)
lib/lp/code/tests/test_branch.py (+4/-23)
lib/lp/registry/vocabularies.py (+1/-2)
James Westby (james-w) wrote :

Note that this is a source package branch where the person in question has upload
rights, and so can edit, but isn't in the owner team.

Thanks,

James

Curtis Hovey (sinzui) on 2010-05-04
affects: launchpad → launchpad-code
Tim Penhey (thumper) on 2010-05-04
Changed in launchpad-code:
status: New → Triaged
importance: Undecided → High
milestone: none → 10.05
tags: added: confusing-ui
tags: added: trivial
Tim Penhey (thumper) wrote :

Looking at the diff for this and I guess trivial is a matter of opinion

Changed in launchpad-code:
assignee: nobody → Tim Penhey (thumper)
Tim Penhey (thumper) on 2010-05-14
Changed in launchpad-code:
status: Triaged → In Progress
Tim Penhey (thumper) on 2010-05-14
tags: added: easy
removed: trivial
Ursula Junque (ursinha) wrote : Bug fixed by a commit

Fixed in stable r10925 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/10925>

Changed in launchpad-code:
status: In Progress → Fix Committed
tags: added: qa-needstesting
Tim Penhey (thumper) wrote :

Soren, could I get you to try this now on edge?

Soren Hansen (soren) wrote :

Well, if I go to the page that I linked to originally, the owner setting defaults to ubuntu-branches, so I think we're good. Thanks!

Tim Penhey (thumper) on 2010-06-01
tags: added: qa-ok
removed: qa-needstesting
Curtis Hovey (sinzui) on 2010-06-02
Changed in launchpad-code:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.
Subscribing...

Other bug subscribers