- Status Closed
- Percent Complete
- Task Type Bugs
- Category Playlists
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#9660 - Database selections trigger the dynamic playlist warning
Recipe:
1) Enable the “Warn when erasing dynamic playlist” option.
2) Play an album from the database.
3) Attempt to play a second album from the database.
Expected behaviour: No warning, as no manual playlist manipulation has taken place.
Actual behaviour: The warning is triggered.
Revision: r19463.
Closed by jdgordon
2009-12-21 05:44
Reason for closing: Fixed
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2009-12-21 05:44
Reason for closing: Fixed
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
ignore that last comment…. wasnt too
bad of a hack.. in r24089.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Besides of this bug, we should consider whether “dynamic” is confusing/missleading.
I’d understand this under dynamic playlist: The playlist that’s created when you start playing music by selecting a track. If you selected a playlist to start music, that’d not be dynamic.
If I understand correctly, the warning should be used to prevent the loss of unsaved playlist editing. If you’re using a saved playlist (i.e. an m3u), then the edits could be saved (I don’t know if we do this or not!). For a dynamic playlist, i.e. any on-the-fly selection then there’s no obvious place to save the changes.
However, if you’ve not performed any edits, this warning should not be needed. Unfortuanately the database sets up a playlist by using the same playlist API that manual insertions do, so from the point of view of the playlist code, edits are taking place.
No real surprise.. this is linked with the other database/playlist creation bugs where the db adds each track in the result instead of doing a single insert call. I dont tihnk there is a simple fix for this.