Attempt to subscribe team that is not associated to distribution fails
Bug #745660 reported by
Brad Crittenden
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Brad Crittenden | ||
Bug Description
In order to restrict who can subscribe to structural subscriptions for large distributions (i.e. Ubuntu) a rule is in place that says if the distro has a bug supervisor set, then only members of the bug supervisor team may be subscribed, unless the subscription is done by an admin.
Access to the subscribe link is controlled and only admins and members of the bug supervisor team can see it. However a member of the bug supervisor team is presented with a list of all of his teams. If he selects one that is not a member of the bug supervisor team then an error occurs.
The list of teams shown in this instance needs to be restricted to those that are members of the bug supervisor team.
Related branches
lp:~bac/launchpad/bug-745660
- Gavin Panella (community): Approve
-
Diff: 764 lines (+145/-518)5 files modified.bzrignore (+1/-0)
Makefile (+17/-12)
lib/canonical/launchpad/icing/icon-sprites.positioning (+0/-484)
lib/lp/bugs/browser/structuralsubscription.py (+13/-3)
lib/lp/bugs/browser/tests/test_expose.py (+114/-19)
| summary: |
- Attempt to subscribe unassociated team to distribution fails + Attempt to subscribe team that is not associated to distribution fails |
| Changed in launchpad: | |
| assignee: | nobody → Brad Crittenden (bac) |
| status: | New → In Progress |
| importance: | Undecided → High |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| Changed in launchpad: | |
| milestone: | none → 11.04 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| William Grant (wgrant) wrote | #2 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
| Changed in launchpad: | |
| milestone: | 11.04 → 11.05 |
Revision history for this message
| Gary Poster (gary) wrote | #3 |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
| Changed in launchpad: | |
| status: | Fix Released → Fix Committed |
| milestone: | 11.05 → none |
| tags: | removed: qa-ok |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r12717 <http://