SubmarinerApp / Submariner

A Subsonic client for macOS
https://submarinerapp.com
BSD 3-Clause "New" or "Revised" License
122 stars 3 forks source link

artist starting with lowercase ends up in [Unknown] #206

Closed dxlr8r closed 1 month ago

dxlr8r commented 1 month ago

As subject says. When renaming first letter to uppercase, sorting is fixed. IMO sorting should not care about case, so both A and a should be under "A".

NattyNarwhal commented 1 month ago

I'm not entirely pleased by the solution; it handles the [Unknown] entries poorly. I'm tempted to special case it, or preserve the ordering provided by the server.

NattyNarwhal commented 1 month ago

It should handle those entries better now. Server order raises some other implications (mostly around the fact we'd need to reorder it anyways or change the schema), so I'm hesitant but interested still.

dxlr8r commented 1 month ago

I have version 3.1.1, and I can confirm that an artist starting with a lower-case "a" appears under [Unknown]

NattyNarwhal commented 1 month ago

Next version will sort in a case insensitive way. I also special-cased the [Unknown] entries since they sorted oddly.