Merging people with active PPAs should be blocked
Bug #676966 reported by
Julian Edwards
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Edwin Grubbs | ||
Bug Description
This causes a lot of OOPSes in the PPA publisher and would be very, very hard to fix. The only reasonable solution is to block the merge until someone deletes their PPAs.
Related branches
lp:~edwin-grubbs/launchpad/bug-676966-disallow-merging-people-with-ppas
- Curtis Hovey (community): Approve (code)
- j.c.sackett (community): Approve (code*)
-
Diff: 251 lines (+123/-12)6 files modifiedlib/lp/registry/browser/peoplemerge.py (+8/-0)
lib/lp/registry/browser/person.py (+4/-8)
lib/lp/registry/browser/tests/test_peoplemerge.py (+54/-3)
lib/lp/registry/doc/person-merge.txt (+16/-0)
lib/lp/registry/interfaces/person.py (+9/-0)
lib/lp/registry/model/person.py (+32/-1)
Revision history for this message
| Julian Edwards (julian-edwards) wrote | #1 |
| tags: | added: opa |
| tags: |
added: ppa removed: opa |
Revision history for this message
| Curtis Hovey (sinzui) wrote | #2 |
| tags: | added: merge-deactivate |
| Changed in launchpad-registry: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| milestone: | none → 10.12 |
Revision history for this message
| Julian Edwards (julian-edwards) wrote Re: [Bug 676966] Re: Merging people with active PPAs should be blocked | #3 |
| Changed in launchpad-registry: | |
| milestone: | 10.12 → series-future |
| Changed in launchpad-registry: | |
| assignee: | nobody → Edwin Grubbs (edwin-grubbs) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #4 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
| Changed in launchpad: | |
| milestone: | none → 11.01 |
To post a comment you must log in.
This is high priority, it's causing thousands of OOPSes each day in the publisher when someone merges accounts with PPAs.