312128 2012-12-23 18:39:17 +0000 Amarok doesn't update database playlists when track's uid url changes 2012-12-23 22:46:28 +0000 1 1 2 Applications amarok Collections/Local 2.6.90 (2.7 beta) Other Linux RESOLVED FIXED NOR normal 2.7 308371 1 matej amarok-bugs-null graham matej ralf-engels walter.von.entferndt http://commits.kde.org/amarok/2f261b02ede0ecc9a566cbcb2680e06db092dbc1 2.7 0 oldest_to_newest 1326068 0 matej 2012-12-23 18:39:17 +0000 This is a bugreport for the second part of the bug 308371. From the original bug: > Maybe I edited the metadata of songs that were already referenced in a playlist in > the amarok database. When I opened the playlist, two songs were greyed and did > not play. > Right click -> searching for module AMAROK-SQLTRACKUID. Then amarok was not > responding for a few minutes, but the module does not seem to be installed, the > two songs still do not play. This part has been already solved. > Reproducible: Always > Steps to Reproduce: > 1. Save a playlist in the DB > 2. Edit the metadata of a song in the playlist > 3. Re-open the playlist Actual Results: The edited song is greyed and can not be played 1326071 1 matej 2012-12-23 18:46:04 +0000 Git commit 2f261b02ede0ecc9a566cbcb2680e06db092dbc1 by Matěj Laitl. Committed on 23/12/2012 at 19:42. Pushed by laitl into branch 'master'. Fix SqlTrack::updatePlaylistToDb(): we need to update url too This fixes all the remaining problems originally reported in bug 308371. I've confirmed it works both then the track uid change is triggered from within Amarok and from external tool when Amarok is not running. BUGFIXES: * Fix updating of Amarok database playlists then track uid changes. Related: bug 308371 FIXED-IN: 2.7 M +1 -0 ChangeLog M +15 -11 src/core-impl/collections/db/sql/SqlMeta.cpp M +1 -0 src/playlistmanager/sql/SqlPlaylist.cpp http://commits.kde.org/amarok/2f261b02ede0ecc9a566cbcb2680e06db092dbc1 1326120 2 walter.von.entferndt 2012-12-23 22:42:05 +0000 Dear Matěj, I regret having caused trouble. Thanx for your patience and insistence to fix the bug. I wish you a peaceful and merry X-Mas. 1326123 3 matej 2012-12-23 22:46:28 +0000 (In reply to comment #2) > Dear Matěj, Hi Torsten, > I regret having caused trouble. No serious trouble caused, you couldn't have known our bugzilla usage in advance. > Thanx for your patience and insistence to fix the bug. > > I wish you a peaceful and merry X-Mas. Thanks, to you too. :-)