Merge lp:~vthompson/music-app/fix-1475768 into lp:music-app

[Victor Thompson]

Refresh cover art when model count changes

Refresh cover art when model count changes

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

PASSED: Continuous integration, rev:887
http://91.189.93.70:8080/job/music-app-ci/1339/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/music-app-vivid-amd64-ci/191

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

review: Approve (continuous-integration)

[Andrew Hayzen (ahayzen)]

LGTM, do you want to add a changelog item for this or start after this MP, as per our new changelog policy :-)

review: Approve

=== modified file 'app/ui/SongsView.qml'
--- app/ui/SongsView.qml	2015-07-01 01:42:04 +0000
+++ app/ui/SongsView.qml	2015-07-17 22:36:26 +0000
@@ -164,9 +164,6 @@
                 playlistChangedHelper()  // update recent/playlist models
 
                 albumTracksModel.filterPlaylistTracks(songStackPage.line2)
-
-                // refresh cover art
-                songStackPage.covers = Playlists.getPlaylistCovers(songStackPage.line2)
             }
         }
     ]
@@ -189,6 +186,9 @@
         model: isAlbum ? songsModel : albumTracksModel.model
         objectName: "songspage-listview"
         width: parent.width
+
+        onCountChanged: songStackPage.covers = Playlists.getPlaylistCovers(songStackPage.line2)
+
         header: BlurredHeader {
             id: blurredHeader
             rightColumn: Column {
@@ -357,9 +357,6 @@
                         playlistChangedHelper()  // update recent/playlist models
 
                         albumTracksModel.filterPlaylistTracks(songStackPage.line2)
-
-                        // refresh cover art
-                        songStackPage.covers = Playlists.getPlaylistCovers(songStackPage.line2)
                     }
                 }
             }