On map page, team members are ordered case-sensitively by Launchpad ID

Bug #286470 reported by Matthew Paul Thomas
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Undecided
Guilherme Salgado

Bug Description

On <https://launchpad.net/~ubuntumembers/+members>, the list of members is correctly ordered case-insensitively by display name.

But on <https://launchpad.net/~ubuntumembers/+map>, the list of members is ordered case-sensitively by Launchpad ID, but this ID is not shown on the page, while the person's display name is. Someone's name usually, but not always, starts with the same character as their Launchpad ID. So the end result is a list that looks like it has been sorted by name, and then lightly scrambled.

That the List view of members is sorted correctly, while both the Map view and a list of meeting attendees (bug 203478) are not, suggests that groups of people are sorted the wrong way by default, and programmers need to (and fail to) remember to override the order to produce a sensible result. If so, perhaps the default ordering should be changed. Regardless, bug 203478 should be fixed at the same time as this bug.

Tags: lp-registry
Revision history for this message
Guilherme Salgado (salgado) wrote :

Groups of people are sorted correctly (case-insensitive) by default. These two places, though, override the default sort order.

In the case of the +map page, the overriding is completely unnecessary, but for the sprint attendees, the default sort order is not used by SQLObject, so it must be overridden -- unfortunately, the sort order chosen there is not the one people would expect.

Changed in launchpad-registry:
assignee: nobody → salgado
milestone: none → 2.1.11
status: New → In Progress
Revision history for this message
Guilherme Salgado (salgado) wrote :

mainline r7229

Changed in launchpad-registry:
status: In Progress → Fix Committed
Revision history for this message
Curtis Hovey (sinzui) wrote :

Fix released in Launchpad 2.1.11.

Changed in launchpad-registry:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.