Seems pretty weird to sort the based on value rather than the display.

Makes it looks like something is out of order or acting buggy.

Comments

johnhanley’s picture

johnhanley commented

The options are listed in the order of entry (which is alphabetical by default) and there's no ORDER BY clause used in the queries responsible for retrieving the states. This is a simple oversight versus some "weird" design choice.

doublejosh’s picture

doublejosh commented

I'll see if I can whip up a patch, sorry if my phrasing was snarky.

johnhanley’s picture

johnhanley commented

No worries. ;-)

I can add the code easily enough. In fact, I've been contemplating a major revision for this module to support an unlimited number of states (instead of the current primary and secondary). I'll try to put something together in the next few weeks (schedule permitting).

laogui’s picture

laogui commented

Eagerly waiting for new version

johnhanley’s picture

johnhanley commented
Status: Active » Closed (fixed)

This fixed is include in profile_location 6.x-1.4.

This applies to the 6.x branch only. The 5.x branch is no longer supported.

doublejosh’s picture

doublejosh commented

Great. Update worked for me.