Rockbox

Tasklist

FS#10717 - Add PLAYLIST_INSERT_LAST_SHUFFLED functionality to onplay menu

Attached to Project: Rockbox
Opened by Tomasz Kowalczyk (mitk) - Saturday, 24 October 2009, 17:53 GMT
Last edited by Dave Hooper (stripwax) - Wednesday, 28 October 2009, 22:28 GMT
Task Type Patches
Category Playlists
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This will Add PLAYLIST_INSERT_LAST_SHUFFLED functionality to onplay menu.
It mostly works but still needs testing. For example it is doesn't make sense when playlist is empty or we are inserting just one track and doesn't need to be displayed in such cases.
My question is: clean it further or leave it?
This task depends upon

Closed by  Dave Hooper (stripwax)
Wednesday, 28 October 2009, 22:28 GMT
Reason for closing:  Accepted
Comment by Tomasz Kowalczyk (mitk) - Saturday, 24 October 2009, 17:55 GMT
Forgot patch file.
Comment by Thomas Martitz (kugel.) - Saturday, 24 October 2009, 19:23 GMT
You also forgot the explain what PLAYLIST_INSERT_LAST_SHUFFLED is supposed to do. At least I didn't get it.
Comment by Tomasz Kowalczyk (mitk) - Saturday, 24 October 2009, 21:15 GMT
Simply: You can add shuffled tracks to the end of dynamic playlist. Without this patch you can add tracks to the end of playlist, but not shuffled.
Comment by Jonathan Gordon (jdgordon) - Sunday, 25 October 2009, 02:22 GMT
looks good... I tihnk its missing stuff so it adds the tracks to the .playlist_control file though?
Comment by Tomasz Kowalczyk (mitk) - Sunday, 25 October 2009, 08:11 GMT
Yes, it adds also to .playlist_control. In fact it uses "playlist API" and is doing everything same way like other playlist manipulating commands.
Comment by Tomasz Kowalczyk (mitk) - Sunday, 25 October 2009, 09:58 GMT
Tested and ready to commit version. Diff against r23340.
Comment by Tomasz Kowalczyk (mitk) - Wednesday, 28 October 2009, 10:54 GMT
Added patch to manual also.
Comment by Tomasz Kowalczyk (mitk) - Wednesday, 28 October 2009, 13:31 GMT
There is no make sense to display insert_last_shuffled option when just one track is selected.
Also some comments added.
Comment by Dave Hooper (stripwax) - Wednesday, 28 October 2009, 22:28 GMT
Committed!

Loading...