Summary: | Wrong album is played if 2 albums with same title but different artist | ||
---|---|---|---|
Product: | [Applications] amarok | Reporter: | gabriel.ruffieux |
Component: | Playlist | Assignee: | Amarok Developers <amarok-bugs-dist> |
Status: | CONFIRMED --- | ||
Severity: | normal | CC: | e.m.a.t.i.r.o.v, joerg.schaible, konrad.zemek, matej, nhn, sam, teo, tim |
Priority: | NOR | ||
Version: | 2.8.0 | ||
Target Milestone: | 2.9 | ||
Platform: | Debian stable | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
gabriel.ruffieux
2011-04-16 16:55:25 UTC
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 |