Bug 17835 - playlistcontrol cmd:load ... play_index:yyyy doesn't work with folders
: playlistcontrol cmd:load ... play_index:yyyy doesn't work with folders
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: CLI
: 7.6.1
: PC Other
: -- normal (vote)
: 7.7.2
Assigned To: Alan Young
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-12-22 01:19 UTC by Joerg Schwieder
Modified: 2011-12-22 06:25 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments
allow play_index in playlistcontrol folder_id (1.74 KB, patch)
2011-12-22 05:22 UTC, Michael Herger
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joerg Schwieder 2011-12-22 01:19:11 UTC
The

playlistcontrol cmd:load folder_id:xxxx play_index:yyy 

command doesn't work (when used with a folder id) but always plays from track 1.
This is in all server releases since 7.6.1.

See also https://bugs-archive.lyrion.org/show_bug.cgi
Comment 1 Michael Herger 2011-12-22 04:42:30 UTC
Some random notes for myself: the "onebrowser" code doesn't use the playlistcontrol command, but a sequence of "playlist loadtracks listref" followed by a "playlist jump xy" command instead.
Comment 2 Michael Herger 2011-12-22 05:16:02 UTC
Some more notes:

- the playlistcontrol command uses "playlist load /some/path play_index:99"

- the "playlist load" command does _not_ accept tagged parameters, thus play_index is always undefined

- "playlist load/play" would always set play_index to 0 for load/play actions, even though play_index can't be defined for them

-> playlistcontrol for folders cannot work with a play_index


In order to fix this issue we need to:

- accept tagged parameters for those actions

- only set play_index to 0 if not defined
Comment 3 Michael Herger 2011-12-22 05:22:12 UTC
Created attachment 7588 [details]
allow play_index in playlistcontrol folder_id
Comment 4 Joerg Schwieder 2011-12-22 06:25:26 UTC
Thanks. That was quick.
Comment 5 SVN Bot 2011-12-22 06:25:35 UTC
 == Auto-comment from SVN commit #33761 to the slim repo by ayoung ==
 == http://svn.slimdevices.com/slim?view=revision&revision=33761 ==

Fixed bug 17835: playlistcontrol cmd:load ... play_index:yyyy doesn't work with folders 
Make sure that play_index parameter is passed from playlistcontrol to "playlist load" and that it is acted upon.