Merge lp:~vthompson/music-app/remix-fix-1386940 into lp:music-app/remix

[Victor Thompson]

* Update the fix for catching a renamed playlist in the recent tab

Update the fix for catching a renamed playlist in the recent tab. Call the helper function prior to actually renaming the playlist in the Recent db.

[lp:~vthompson/music-app/remix-fix-1386940]

734. By Victor Thompson on 2014-11-11

add newline

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

PASSED: Continuous integration, rev:734
http://91.189.93.70:8080/job/music-app-remix-ci/245/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-vivid/135
        deb: http://91.189.93.70:8080/job/generic-mediumtests-vivid/135/artifact/work/output/*zip*/output.zip
    SUCCESS: http://91.189.93.70:8080/job/music-app-remix-vivid-amd64-ci/20

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/music-app-remix-ci/245/rebuild

review: Approve (continuous-integration)

[lp:~vthompson/music-app/remix-fix-1386940]

735. By Victor Thompson on 2014-11-11

Mark MusicStart page as changed when renaming a playlist in the recent db.

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

PASSED: Continuous integration, rev:735
http://91.189.93.70:8080/job/music-app-remix-ci/246/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-vivid/153
        deb: http://91.189.93.70:8080/job/generic-mediumtests-vivid/153/artifact/work/output/*zip*/output.zip
    SUCCESS: http://91.189.93.70:8080/job/music-app-remix-vivid-amd64-ci/21

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/music-app-remix-ci/246/rebuild

review: Approve (continuous-integration)

[lp:~vthompson/music-app/remix-fix-1386940]

736. By Victor Thompson on 2014-11-11

Update solution to problem.

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

PASSED: Continuous integration, rev:736
http://91.189.93.70:8080/job/music-app-remix-ci/247/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-vivid/154
        deb: http://91.189.93.70:8080/job/generic-mediumtests-vivid/154/artifact/work/output/*zip*/output.zip
    SUCCESS: http://91.189.93.70:8080/job/music-app-remix-vivid-amd64-ci/22

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/music-app-remix-ci/247/rebuild

review: Approve (continuous-integration)

[Andrew Hayzen (ahayzen)]

Thanks for updating the solution :)

review: Approve

=== modified file 'common/SongsPage.qml'
--- common/SongsPage.qml	2014-11-01 23:25:42 +0000
+++ common/SongsPage.qml	2014-11-11 18:54:57 +0000
@@ -522,11 +522,11 @@
                                 Library.recentRenamePlaylist(playlistName.placeholderText, playlistName.text)
                             }
 
+                            line2 = playlistName.text
+
                             playlistChangedHelper()  // update recent/playlist models
 
                             PopupUtils.close(dialogEditPlaylist)
-
-                            line2 = playlistName.text
                         }
                         else {
                             editplaylistoutput.text = i18n.tr("Playlist already exists")