Version: 2.3.2 (using KDE 4.5.1) OS: Linux If there is 2 albums with same title but from different artists, the next track played is the next track from the same titled album instead of current playing album. Reproducible: Always Steps to Reproduce: 1. I prepare a playlist with albums "You're Under Arrest" from Mils Davis followed by "You're Under Arrest" from Serge Gainsbourg. 2. I read my playlist randomly by album. Actual Results: First track of album from Miles Davis is played. Next track is the smallest ID of track from "You're Under Arrest" album that has not been played yet, so it is #1 from Gainsbourg's album. Next track is #2 from Miles, then #2 from Gainsbourg, and so on... Expected Results: First track of album from Miles Davis is played. Next track should be #2 from Miles's album. When album's finished, it should select another album randomly. OS: Linux (x86_64) release 2.6.35-28-generic Compiler: cc
That is really a corner case: having only 2 albums with the same name in the playlist is really very restrictive and doesn't give the random engine much of a choice. It would be definitely an error if it also did change album with a different name. Moving this to the wishlist.
(In reply to comment #1) > That is really a corner case: having only 2 albums with the same name in the > playlist is really very restrictive and doesn't give the random engine much > of a choice. It would be definitely an error if it also did change album > with a different name. No, I think you misunderstood. This is an actual bug, "Random Albums" play mode should compare the albums by both album artist and album name, reverting back to normal prio. Sam, didn't you fix something similar in Dynamic Playlists lately? Perhaps you can have a look a this, too? ;-) ^-^
Actually Amarok has major problems with albums that share the same name, but are from different artists (despite a correct album artist tag): https://bugs.kde.org/show_bug.cgi?id=216759
Matĕj, I assume you tested it with latest git, setting version accordingly :)
*** Bug 311410 has been marked as a duplicate of this bug. ***
Is this still valid with Amarok 2.7 or current git?
Still wrong in Amarok 2.7
Thank you for the feedback.
I submitted a patch for review: https://git.reviewboard.kde.org/r/110070/
Let me comment some more on the issue. It appears there are actually two distinct bugs here. The one I provided the fix for is directly connected with playlist sorting, where two albums were merged together on the playlist if they had the same name name but different artist - so very similar to https://bugs.kde.org/show_bug.cgi?id=216759 (although that one was for collection, not playlist). I set up two albums with same name and different artist and got that merged result, so I automatically assumed it's that very bug mentioned here. So even after my fix, when playlist is sorted by album and album shuffle is enabled, the shuffle skips between albums playing the "next" track (like mentioned in first comment). But this scenario (album shuffle when sorting by album) seems to cause bugs beyond these with identically named albums. I actually just got the same behavior with two different albums for different artists, and I'm frequently getting crashes while skipping to the next track. So yes, there remains a bug here, and an annoying one. I'll take an in-depth look on it later today. If you happen to know if my fix covers an actual reported bug, please drop me a message.
Git commit 583158f4c4d25098aa1f0bb6b683813488580d52 by Matěj Laitl, on behalf of Konrad Zemek. Committed on 27/04/2013 at 11:08. Pushed by laitl into branch 'master'. Cleaned up playlist multilevel sorting algorithm. Added album artist special case to playlist sorting. It fixes an issue with two albums being merged on the playlist when they happen to have the same name, and sorting by album is enabled. Originally thought to fix bug 271105, it actually fixes some other, but connected, one (see my comment https://bugs.kde.org/show_bug.cgi?id=271105#c10 ). Thanks go to Konrad Zemek for the patch. REVIEW: 110070 M +114 -83 src/playlist/proxymodels/SortAlgorithms.cpp M +3 -0 src/playlist/proxymodels/SortAlgorithms.h M +19 -13 src/playlist/proxymodels/SortScheme.cpp M +11 -8 src/playlist/proxymodels/SortScheme.h M +2 -2 tests/mocks/MetaMock.h M +63 -44 tests/playlist/TestPlaylistModels.cpp http://commits.kde.org/amarok/583158f4c4d25098aa1f0bb6b683813488580d52
I tried to test it in amarok v2.8.0. I have 4 songs in playlist: song #1 - artist#1 song #2 - artist#1 song #1 - artist#2 song #2 - artist#2 I start playing with random album option. Played: song #1 - artist#1 song #1 - artist#2 song #2 - artist#1 song #2 - artist#2