mailing list subscription email address are in NEW status
Bug #471770 reported by
Curtis Hovey
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
High
|
Unassigned |
Bug Description
OOPS-1402EB708 shows an oops that occurs when ~lhavelund tries to edit his email addresses. This failure is SimpleVocabular
The user has 2 emails. one preferred email address. There is another NEW email address that came from the merged account. I do not think this is correct given that the merged account was used. maybe I am mistaken. Either way, the terms are preferred plus validated, so I expect to see only one email address in the choice.
Related branches
lp://qastaging/~sinzui/launchpad/merge-mailing-list-bug-471770
- Abel Deuring (community): Approve (code)
-
Diff: 272 lines (+109/-32)5 files modifiedlib/canonical/launchpad/emailtemplates/person-merged.txt (+15/-0)
lib/lp/registry/doc/person-merge.txt (+26/-2)
lib/lp/registry/model/person.py (+14/-15)
lib/lp/registry/tests/test_personset.py (+52/-11)
lib/lp/testing/menu.py (+2/-4)
Changed in launchpad-registry: | |
assignee: | nobody → Curtis Hovey (sinzui) |
tags: | added: merge-deactivate |
Changed in launchpad: | |
assignee: | Curtis Hovey (sinzui) → nobody |
To post a comment you must log in.
This issue appears to be a case where the current value is is not in the vocabulary. Since the vocabulary contains only the user's preferredemail address, the user appears to have another address from the merge set as the address for the mailing list. This is probably the address in NEW status.
If I can verify this is the case, we can update the email address in the production db to fix the user's issue. What we do not understand is how a preferred or confirmed email address can be set to NEW. The mailing lists could be set to use the preferred email address during the merge, but I think it is wrong to not honour the user's original choice.