PersonSet.merge must delete the team email address
Bug #577079 reported by
Curtis Hovey
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Curtis Hovey | ||
Bug Description
Teams can have only one contact address, or None. They may have a mailing list address. When teams are merged, the merged team's email addresses are reassigned. Calling from_person.
Related branches
lp:~sinzui/launchpad/delete-team-3
- Henning Eggers (community): Approve (code)
-
Diff: 788 lines (+227/-207)10 files modifiedlib/lp/registry/browser/peoplemerge.py (+37/-8)
lib/lp/registry/browser/team.py (+7/-9)
lib/lp/registry/browser/tests/mailinglist-views.txt (+6/-8)
lib/lp/registry/browser/tests/peoplemerge-views.txt (+2/-28)
lib/lp/registry/browser/tests/test_peoplemerge.py (+62/-1)
lib/lp/registry/interfaces/person.py (+21/-8)
lib/lp/registry/model/person.py (+23/-11)
lib/lp/registry/stories/mailinglists/lifecycle.txt (+14/-125)
lib/lp/registry/templates/team-delete.pt (+2/-2)
lib/lp/registry/tests/test_teammembership.py (+53/-7)
| tags: | added: merge-deactivate |
Revision history for this message
| Curtis Hovey (sinzui) wrote | #1 |
| Changed in launchpad-registry: | |
| milestone: | none → 10.11 |
| assignee: | nobody → Curtis Hovey (sinzui) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #2 |
| tags: | added: qa-needstesting |
| Changed in launchpad-registry: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad-registry: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Move the call to setContactAddre