Cleaned up playlist multilevel sorting algorithm. Added album artist special...
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 CCBUG: 271105
Showing
- src/playlist/proxymodels/SortAlgorithms.cpp 114 additions, 83 deletionssrc/playlist/proxymodels/SortAlgorithms.cpp
- src/playlist/proxymodels/SortAlgorithms.h 3 additions, 0 deletionssrc/playlist/proxymodels/SortAlgorithms.h
- src/playlist/proxymodels/SortScheme.cpp 19 additions, 13 deletionssrc/playlist/proxymodels/SortScheme.cpp
- src/playlist/proxymodels/SortScheme.h 11 additions, 8 deletionssrc/playlist/proxymodels/SortScheme.h
- tests/mocks/MetaMock.h 2 additions, 2 deletionstests/mocks/MetaMock.h
- tests/playlist/TestPlaylistModels.cpp 63 additions, 44 deletionstests/playlist/TestPlaylistModels.cpp
Loading
Please register or sign in to comment