User appears twice on +mailing-list-subscribers
Bug #568114 reported by
Francis J. Lacoste
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Low
|
Curtis Hovey |
Bug Description
If you visit https:/
There are indeed two subscribers named Alain portal, but one is ~alain-portal and the other one is ~alain-
There must be some bad caching going on somewhere
Related branches
lp:~sinzui/launchpad/mailing-list-subscribers-0
- Aaron Bentley (community): Approve
-
Diff: 200 lines (+46/-44)3 files modifiedlib/lp/registry/browser/team.py (+2/-2)
lib/lp/registry/model/mailinglist.py (+15/-13)
lib/lp/registry/tests/test_mailinglist.py (+29/-29)
Changed in launchpad-registry: | |
assignee: | nobody → Edwin Grubbs (edwin-grubbs) |
status: | New → Triaged |
Changed in launchpad-registry: | |
assignee: | Edwin Grubbs (edwin-grubbs) → nobody |
importance: | Undecided → Low |
tags: |
added: merge-deactivate removed: trivial |
summary: |
- Two different persons have the same URL on +mailing-list-subscribers + Merged user appears twice on +mailing-list-subscribers |
tags: | removed: merge-deactivate |
Changed in launchpad-registry: | |
milestone: | series-future → 10.09 |
status: | Triaged → In Progress |
summary: |
- Merged user appears twice on +mailing-list-subscribers + User appears twice on +mailing-list-subscribers |
Changed in launchpad-registry: | |
assignee: | nobody → Curtis Hovey (sinzui) |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in launchpad-registry: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
The code is calling the person formatter for each person in mailing_ list.getSubscri bers(). I think the issue is in getSubscribers() which does this order_by( Person. displayname)
results.
I believe storm is doing this. I suspect adding order_by( Person. displayname, Person.id)
results.