[mythtv] Ticket #1678: mythvideo sorting is case-sensitive

Anduin Withers awithers at anduin.com
Fri Apr 21 08:00:30 UTC 2006


> I still believe that the answer to getting #1569 into trunk is not to
> add more and more stuff to it, but to break it apart into small,
> easily-verifiable patches that each change one behavior--making it
> easier understand the changes (and less dangerous to commit).

Smaller patches, incrementally adding features, are preferred. In the end it
saves everyone time and frustration.

> The locale-aware sort uses the locale's LC_COLLATE collation order to
> determine the proper order for the sort.  In some locales, LC_COLLATE
> gives an order such as "AaBbCc", etc.

The current code is wrong. Locale aware, case insensitive, is the most
intuitive. It is also very likely the DB default as well (latin1 usual
equals latin1_swedish_ci). The difference between (AAA, aaa, All About Eve)
and (AAA, All About Eve, aaa) is noticeably annoying in longer lists.

> I seriously doubt the reporter is going to remember in which order the 5
> videos with the exact same title appear in both lists.  However, if we 
> want those sort orders to be identical, we just need to change the SQL 
> query to use "ORDER BY title, filename, intid" if the list uses the sort
> in this patch.

This assumes the DB collation order is the same, which would be better than
what is there but potentially still not identical (considering it won't
match the "leading article stripped" sort).

> Anyway, that's just my opinion.  Ask Anduin what he prefers an opinion
> that matters.

You can only blame slow mythdvd bug reviews on me. I'm just looking at the
mythvideo bugs because some have been sitting there for some time (and in
some cases should have probably just been bounced).

-- 
Anduin Withers



More information about the mythtv-dev mailing list