Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Alphabetical sorting not working with non ASCII chars #3463

Open
1 task done
milanv-source opened this issue Jan 16, 2025 · 4 comments
Open
1 task done

Alphabetical sorting not working with non ASCII chars #3463

milanv-source opened this issue Jan 16, 2025 · 4 comments
Labels
bug Something isn't working frontend

Comments

@milanv-source
Copy link

What version of Music Assistant has the issue?

2.3.6

What version of the Home Assistant Integration have you got installed?

N/A

Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

  • Yes

The problem

When sorting items (Albums, Tracks, ...) , alphabetical sorting ignores non ASCII characters and place them at the and of the list (or beginning if descending).

How to reproduce

Add an item with non ASCII character. e.g. Artist Čechomor. Sort Artists alphabetically. Čechomor should be placed in the list between C and D not at the end after Z.

Music Providers

Filesystem (remote share).

Player Providers

not related to players

Full log output

music-assistant.log

Additional information

No response

What version of Home Assistant Core are your running

2025.1.2

What type of installation are you running?

Home Assistant OS

On what type of hardware are you running?

Generic x86-64 (e.g. Intel NUC)

@marcelveldt
Copy link
Member

Please be very descriptive which listing exact you mean, you can also include a screenshot to even make it more clear

@milanv-source
Copy link
Author

I dont have Playlists or Radios populated, so I can confirm that behavior only on Artists and Albums. Tracks actually as well, but the scroll bar hides behind player tab, so i cant navigate all the way down (im not gonna scroll by mouse wheel over thousands of Tracks...)
Image

but i believe if sorted alphabetically descending this issue surfaces, ascending will be the same.

Image

For reference Jellyfin is sorting albums alphabetically correctly including non ASCII characters.

@Jc2k
Copy link

Jc2k commented Jan 17, 2025

For reference, the Jellyfin provider does no sorting and it does not see which sorting option you select.

Your Jellyfin instance may set the SortName as well as the Name field. Name is shown in the Jellyfin ui. SortName is not. Typically they are the same, but "The" is special cased for sorting. We do sync that into MA for Album and Artist. It's possible that's why you see different behaviours. You should be able to see the Sort Name in the metadata editor. It would be good to confirm what that is set to for one of these albums or artists.

@milanv-source
Copy link
Author

milanv-source commented Jan 17, 2025

For reference, the Jellyfin provider does no sorting and it does not see which sorting option you select.

Your Jellyfin instance may set the SortName as well as the Name field. Name is shown in the Jellyfin ui. SortName is not. Typically they are the same, but "The" is special cased for sorting. We do sync that into MA for Album and Artist. It's possible that's why you see different behaviours. You should be able to see the Sort Name in the metadata editor. It would be good to confirm what that is set to for one of these albums or artists.

The issue has nothing to do with Jellyfin. In the issue report I state "Filesystem (remote share)." provider. I have mentioned Jellyfin just for reference, because Jellyfin UI sorts the items alphabetically correctly.

In my case Jellyfin :

Image

Čechomor correctly between Aurora and Cech Philharmonic. *on second look, i would actually expect Č after C, but never mind...

In MA:

Image

Čechomor incorrectly after WWW Neurobeat

@OzGav OzGav added bug Something isn't working frontend and removed triage labels Jan 18, 2025
@OzGav OzGav moved this from NOW to NEXT in Music Assistant (V2) backlog Jan 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend
Projects
Status: NEXT
Development

No branches or pull requests

4 participants