Merge lp:~lifeless/launchpad/features into lp:launchpad
Proposed by
Robert Collins
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Robert Collins | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 11863 | ||||
Proposed branch: | lp:~lifeless/launchpad/features | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
147 lines (+65/-4) 5 files modified
lib/lp/registry/interfaces/person.py (+5/-2) lib/lp/registry/model/person.py (+3/-0) lib/lp/registry/tests/test_person.py (+7/-0) lib/lp/services/features/tests/test_webapp.py (+25/-2) lib/lp/services/features/webapp.py (+25/-0) |
||||
To merge this branch: | bzr merge lp:~lifeless/launchpad/features | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jeroen T. Vermeulen (community) | code | Approve | |
Review via email: mp+40049@code.launchpad.net |
Commit message
Add a team: scope.
Description of the change
Not much to say. Team based scope. \o/. Drive by fix in inTeam to handle nonexistant teams more gracefully. (We could raise, but the existing handling for e.g. teams that are Persons is to return False, so I followed that style, for now).
To post a comment you must log in.
As discussed on IRC. Note that existing behaviour is for teams that are Persons is not actually to return False as opposed to raising an exception; it is to return False if the "team" is not the same person as self (which is the only circumstance under which there could be a membership).
Since you made a change in the interface where a mistake that would previously raise an error will now be quietly accepted, you're adding a test and documentation for the change.