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

Proposed by Victor Thompson
Status: Merged
Approved by: Victor Thompson
Approved revision: 810
Merged at revision: 812
Proposed branch: lp:~vthompson/music-app/fix-clear-queue
Merge into: lp:music-app/remix
Diff against target: 13 lines (+1/-1)
1 file modified
music-app.qml (+1/-1)
To merge this branch: bzr merge lp:~vthompson/music-app/fix-clear-queue
Reviewer Review Type Date Requested Status
Andrew Hayzen Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+247533@code.launchpad.net

Commit message

* Clear the recent db before the model

Description of the change

* Clear the recent db before the model

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Andrew Hayzen (ahayzen) wrote :

Doesn't break anything, so if it fixes the issue top approve :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'music-app.qml'
--- music-app.qml 2015-01-23 13:22:29 +0000
+++ music-app.qml 2015-01-25 18:38:52 +0000
@@ -891,8 +891,8 @@
891891
892 function clear()892 function clear()
893 {893 {
894 Library.clearQueue()
894 model.clear()895 model.clear()
895 Library.clearQueue()
896896
897 queueIndex = 0 // reset otherwise when you append and play 1 track it doesn't update correctly897 queueIndex = 0 // reset otherwise when you append and play 1 track it doesn't update correctly
898 }898 }

Subscribers

People subscribed via source and target branches