Bug 413052 - When an album comprises multiple discs, the secondary level of sorting should be by track number, not alphabetical
Summary: When an album comprises multiple discs, the secondary level of sorting should...
Status: RESOLVED NOT A BUG
Alias: None
Product: Elisa
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-10-16 19:04 UTC by Nate Graham
Modified: 2019-10-25 12:12 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Mis-ordered songs within each disc (79.88 KB, image/png)
2019-10-16 19:04 UTC, Nate Graham
Details
Video reproduction of the bug (2.08 MB, video/webm)
2019-10-18 16:27 UTC, Nate Graham
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2019-10-16 19:04:52 UTC
Created attachment 123249 [details]
Mis-ordered songs within each disc

As a result, all the songs within each disc are out of order. See the attached screenshot.
Comment 1 Matthieu Gallien 2019-10-16 20:50:49 UTC
I will take care of it. It should be possible before the end of the week.
Comment 2 Nate Graham 2019-10-16 20:59:00 UTC
Thanks!
Comment 3 Matthieu Gallien 2019-10-18 16:08:22 UTC
I cannot yet reproduce this bug.

How are you adding the tracks to the playlist ?
Comment 4 Nate Graham 2019-10-18 16:27:47 UTC
Created attachment 123322 [details]
Video reproduction of the bug

Looking at the video after recording it, I'm realizing that the songs are mis-tagged and are actually all supposed to be a part of the same disc (it isn't a multi-disc album). But I'll leave that broken for the moment so we can triage this bug. :)
Comment 5 Matthieu Gallien 2019-10-20 21:21:45 UTC
(In reply to Nate Graham from comment #4)
> Created attachment 123322 [details]
> Video reproduction of the bug
> 
> Looking at the video after recording it, I'm realizing that the songs are
> mis-tagged and are actually all supposed to be a part of the same disc (it
> isn't a multi-disc album). But I'll leave that broken for the moment so we
> can triage this bug. :)

Thanks for the video.

This is strange as I cannot reproduce with the flatpak package from KDE CI servers.
I will maybe try to add more logs to better understand what could be going on here. Sorry
Comment 6 Matthieu Gallien 2019-10-23 15:54:28 UTC
In fact, the tracks are in the correct order.
The first number is the track number and the second is the disc number.
They are in increasing order 2 < 3 < 5 < 9 < 11 for disc 1.
They are in increasing order 1 < 4 < 6 < 7 < 8 < 10 < 12 for disc 2.

I am confused now.
Comment 7 Nate Graham 2019-10-23 16:21:09 UTC
Oh.

I'm so sorry for wasting your time. I think this really is a tagging issue with the songs in question. :(
Comment 8 Matthieu Gallien 2019-10-23 18:40:52 UTC
(In reply to Nate Graham from comment #7)
> Oh.
> 
> I'm so sorry for wasting your time. I think this really is a tagging issue
> with the songs in question. :(

No problem.
I had been unable to see that the track number where ordered correctly before. So I am guilty also ;)
Comment 9 Matthieu Gallien 2019-10-25 08:04:47 UTC
Git commit fe840744eeb2dc5266310d86e0660aaa783a7a74 by Matthieu Gallien.
Committed on 22/10/2019 at 15:31.
Pushed by mgallien into branch 'master'.

add logging in playlist with a new category

M  +14   -0    src/CMakeLists.txt
M  +8    -0    src/mediaplaylist.cpp
M  +3    -0    src/trackslistener.cpp

https://invent.kde.org/kde/elisa/commit/fe840744eeb2dc5266310d86e0660aaa783a7a74