FS#10482 - Warn about erasing dynamic playlist when loading bookmark

Attached to Project: Rockbox
Opened by Tuomas Airaksinen (tuma) - Saturday, 01 August 2009, 18:34 GMT
Last edited by Bertrik Sikken (bertrik) - Sunday, 26 September 2010, 21:13 GMT
Task Type Patches
Category Playlists
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


A bugfix.
This task depends upon

Closed by  Bertrik Sikken (bertrik)
Sunday, 26 September 2010, 21:13 GMT
Reason for closing:  Accepted
Additional comments about closing:  Committed as SVN r28176, thanks.
Comment by Tuomas Airaksinen (tuma) - Monday, 14 September 2009, 08:07 GMT
please apply this or give a reason for rejecting this simple but obviously useful patch. not too many lines to inspect in this patch.. :P
Comment by Bertrik Sikken (bertrik) - Thursday, 05 August 2010, 15:40 GMT
Please explain what the problem is, and why it is a bug.
Please explain how your patch fixes the problem.

Just stating "A bugfix" doesn't help to get this applied.
Comment by Tuomas Airaksinen (tuma) - Thursday, 05 August 2010, 15:48 GMT
The problem scenario: You have made a new (dynamic, not saved) playlist. You go to Recent bookmarks and load a bookmark of another list/folder. Your dynamic playlist is lost.
Fix: Add warning query before erasing dynamic playlist before the bookmark is loaded.