MythMusic search result insert error

[otto at kolsi dot fi]

There's following bug in MythMusic:

Enter to MythMusic so that there are songs in playlist (automatically saved during earlier session). Using '3' and the music tree, clear the playlist and select something else to it and esc back to player. Now select Menu and Search and then select just some song and Enter -> select "Insert after current track".

-> Playlist is changed to contain the songs that you had there when you first entered MythMusic and not those that you later selected from the music tree.

[otto at kolsi dot fi]

If anyone is interested in helping, here are my findings so far:

Playlist::fillSonglistFromQuery() gets called when the new track is inserted into the current playlist from the search results. However, this function manipulates internal variable 'raw_playlist' which IS NOT in correct state when the bug occurs.

When fillSonglistFromQuery() gets called after changing current playlist through 'Edit playlist' function, the 'raw_playlist' still contains the previous playlist which was loaded earlier (when MythMusic was launched e.g.).

So my fix would be to update the 'raw_playlist' inside Playlist object immediately after current playlist is changed when using 'Edit playlist' function.

This sounds like some corner case, but this can easily happen when you search some songs from the library.. and then the whole current playlist is blown away. Not very nice when you are having party or something similar :)

[stuartm]

(In [18454]) Fix for old playlist re-appearing when appending tracks to the new list using the Search. Closes #5491. Will backport to -fixes once it receives sufficient testing in trunk.