Bugzilla – Bug 175
using sort order makes pagination unusable
Last modified: 2008-12-15 11:57:01 UTC
With the recently submitted patch, I finally have slim server paying attention to my TSOP tags. This makes the "Browse Artists" page listing much more intuitive, but unfortunately it makes the pagination of that frame completely unusable. Before I had TSOP working properly, the line at the top and bottom of that frame/page would read: "A B C D E F G H I J K L M N O P Q R S T U V W X Y Z" and I could jump exactly to the page with the artist I was looking for. Now the same area reads: "A B A H A M A D T A L A F A D A B J B J B L B P B J B N B D L B M B W B K B C J T C N C S M C N C P C L C J C H C E C J C D G D M D M D A D S D E D E D E F B F D F P D G L B G H N H M H M H B H B H I J I B I J E J L J G T J K A N K C K E K D K E L C L J A L D M A M H M L S M T M A G M A M V A W M P M N J N G N M S O P O P Q R L R L R S F S F S L S J S B S C A S R S T B U P V S V C V W A W B W X Y F Z" which isn't terribly useful for finding a particular artist. Perhaps the pagination functions could be made to use the sort ordering as well and thus restore it's usability? I can only assume that the same issue would affect TSOA and TSOT tags, though I haven't tried using those myself.
Created attachment 7 [details] Patch to fix the sort order pagebar problem This fixes the sort order pagebar problem by creating a new function in Slim/Music/Info.pm to return a SortName for an item, if it exists, and then calls that function in alphapagebar in Slim/Web/Pages.pm so that the pagebar is created using the sortnames where applicable.
Comment on attachment 7 [details] Patch to fix the sort order pagebar problem There's a typo in the patch. a ) is missing to close the call to getSortName.
Created attachment 8 [details] Corrected patch to fix sort order pagebar problem The previous patch was missing a ).
You Rock Ted! Thanks. Applied.
This works great. Thanks!
This bug has been fixed in the latest release of SqueezeCenter! Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already. If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.