IPerson.addMember() should not demote admins
Bug #480157 reported by
Guilherme Salgado
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Triaged
|
Low
|
Unassigned | ||
Bug Description
If you call addMember() passing a person who's already an admin of the team to it, that person will be turned into a regular member.
The test below should pass, but it fails, demostrating the issue.
>>> Person.get(1) in Person.
True
>>> Person.
>>> Person.get(1) in Person.
True
Revision history for this message
| William Grant (wgrant) wrote | #1 |
| tags: | added: teams |
| Changed in launchpad: | |
| assignee: | nobody → Aaron Bentley (abentley) |
| status: | Triaged → In Progress |
| Changed in launchpad: | |
| assignee: | Aaron Bentley (abentley) → nobody |
| status: | In Progress → Triaged |
To post a comment you must log in.
This now affects the AJAX 'Add member' interface, which will silently demote admins. This is probably bad.