teams should be members of themselves
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Critical
|
Guilherme Salgado | ||
Bug Description
Carlos Perelló Marín wrote:
> Not really, I talked with Salgado about that because I assume the same
> but the test were failing, well, I assumed that
> rosetta_
> misunderstood that code). A team is never member of its own team but
Hmm... this sounds like a bug in the team machinery to me.
>>> person_set = getUtility(
>>> stub = person_
>>> stub.inTeam(stub)
True
>>> admins = person_
>>> admins.
False
>>>
IIRC People were members of their own team specifically so membership checks
could be done in a single database query.
affects /products/launchpad
--
Stuart Bishop <email address hidden> http://
Canonical Ltd. http://
| Diogo Matsubara (matsubara) wrote | #1 |
| Changed in launchpad: | |
| status: | Unconfirmed → Confirmed |
| Carlos Perelló Marín (carlos) wrote | #2 |
| Changed in launchpad: | |
| assignee: | nobody → salgado |
| Changed in launchpad: | |
| status: | Confirmed → In Progress |
| Stuart Bishop (stub) wrote | #3 |
| Changed in launchpad: | |
| importance: | Medium → Critical |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
| Данило Шеган (danilo) wrote | #4 |
| Changed in rosetta: | |
| status: | Unconfirmed → Confirmed |
| Carlos Perelló Marín (carlos) wrote | #5 |
| Joey Stanford (joey) wrote | #6 |
| Данило Шеган (danilo) wrote | #7 |
| Changed in rosetta: | |
| assignee: | nobody → danilo |
| milestone: | none → 1.2.3 |
| status: | Confirmed → In Progress |
| Данило Шеган (danilo) wrote | #8 |
| Changed in rosetta: | |
| status: | In Progress → Fix Committed |
| Changed in rosetta: | |
| status: | Fix Committed → Fix Released |
Confirming as per steve's email to the list