Bug 17577 - Strings stored in sort tags should not be modified
: Strings stored in sort tags should not be modified
Status: RESOLVED DUPLICATE of bug 15988
Product: Logitech Media Server
Classification: Unclassified
Component: Scanner
: 7.6.2
: PC Windows XP
: -- normal with 3 votes (vote)
: ---
Assigned To: Unassigned bug - please assign me!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-09-17 18:27 UTC by Jim McAtee
Modified: 2014-08-06 10:31 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jim McAtee 2011-09-17 18:27:38 UTC
The strings in the database's sort columns (namesort, titlesort) are given the same treatment that search (namesearch, titlesearch) columns are given. They're capitalized, punctuation characters are replaced by spaces and spaces are folded. IMO, this is wrong, and I can't think of a reason why it's either necessary or desirable. Internal punctuation characters should have significance when sorting, as they do in determining the uniqueness of an entity such as an artist or album.

(500) Days of Summer Soundtrack

should sort before, not after

18 Blues Masterpieces

The only exceptions to this are that leading and trailing spaces should be removed, since they're being removed for the name and title columns. And they should have leading 'Articles To Ignore When Sorting' removed.

And it shouldn't be necessary to capitalize the values as long as the DBMS used is capable of correctly collating alphabets.
Comment 1 Jim McAtee 2012-02-08 20:28:46 UTC
Also see bug 15998.(In reply to comment #0)
> And they should have leading 'Articles To Ignore When Sorting' removed.
> 

This isn't 100% correct. For the correct treatment of explicit sorting tags, see bug 15998. The 'articles to remove when sorting' should only be removed when there are no sort tags such as ARTISTSORT, ALBUMARTISTSORT, ALBUMSORT, etc. Only when a sort string is copied from ARTIST, ALBUMARTIST, ALBUM, TITLE tags should they be stripped of leading articles.
Comment 2 Jim McAtee 2012-02-08 20:30:24 UTC
Correction: also see bug 15988.
Comment 3 Michael Herger 2014-08-06 10:31:37 UTC

*** This bug has been marked as a duplicate of bug 15988 ***