FS#8752 - In Party Mode rockbox thinks applications are media

Attached to Project: Rockbox
Opened by kousu (kousu) - Monday, 17 March 2008, 23:59 GMT
Last edited by Jonathan Gordon (jdgordon) - Monday, 14 April 2008, 10:05 GMT
Task Type Bugs
Category User Interface
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


If Party Mode is on and you try to run an application (by pressing Select) the screen flashes "Party Mode" at you and the application doesn't run. If you press the button a little longer you will get the File-browser Context menu and try to let you add an App to the current playlist.
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Monday, 14 April 2008, 10:05 GMT
Reason for closing:  Fixed
Additional comments about closing:  I also couldnt get it to show the playlist options, but I have just removed the playlist catalog items from the menu...
Comment by Nils Wallménius (nls) - Tuesday, 18 March 2008, 22:30 GMT
The prevention of plugins starting is intended but adding them to the playlist is clearly not :)
Comment by kousu (kousu) - Tuesday, 18 March 2008, 22:33 GMT
Is it? Why is that? The oscilloscope plugin certainly is meant for use during audio being played; what is the rationale behind Party Mode having plugins locked off?
Comment by Nils Wallménius (nls) - Tuesday, 18 March 2008, 22:45 GMT
Most likely it was made that way because quite a few plugins will stop playback when they are started and rockbox has no way of knowing if the started plugin is one of those.
Comment by Sascha Wolf (Horscht) - Friday, 28 March 2008, 09:44 GMT
couldn't this be solved by some sort of whitelist?
Comment by Marc Guay (Marc_Guay) - Thursday, 03 April 2008, 14:15 GMT
I couldn't reproduce the adding-rocks-to-current-playlist-in-party-mode part of this report on the e200 w/ r16929, but it does seem that Rockbox will allow you to add any file type (*.txt, *.sdk, etc) to a new or existing playlist through the Playlist Catalog menu, which should probably be prevented.