PersonMergeJob does not specify its operation type
Bug #918742 reported by
Aaron Bentley
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Aaron Bentley | ||
Bug Description
When PersonMergeJob oopses, it sends an email. That email says "unspecified operation", because getOperationDes
Related branches
lp:~abentley/launchpad/person-merge
- Abel Deuring (community): Approve (code)
-
Diff: 371 lines (+73/-41)12 files modifiedlib/lp/registry/browser/peoplemerge.py (+1/-1)
lib/lp/registry/browser/tests/test_peoplemerge.py (+5/-2)
lib/lp/registry/browser/tests/test_person.py (+3/-1)
lib/lp/registry/browser/tests/test_team.py (+1/-1)
lib/lp/registry/interfaces/person.py (+4/-1)
lib/lp/registry/model/person.py (+4/-3)
lib/lp/registry/model/persontransferjob.py (+14/-10)
lib/lp/registry/tests/test_person.py (+8/-3)
lib/lp/registry/tests/test_person_merge_job.py (+23/-17)
lib/lp/registry/tests/test_personset.py (+2/-1)
lib/lp/services/job/interfaces/job.py (+7/-0)
lib/lp/services/verification/browser/logintoken.py (+1/-1)
| tags: | added: merge-deactivate |
| Changed in launchpad: | |
| assignee: | nobody → Aaron Bentley (abentley) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-untestable removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r15710 <http://