Bug 17924 - ip3k - Save Playlist plugin - Backing out of save playlist breaks use via extras menu
: ip3k - Save Playlist plugin - Backing out of save playlist breaks use via ext...
Status: RESOLVED DUPLICATE of bug 15001
Product: Logitech Media Server
Classification: Unclassified
Component: Plugin
: 7.6.1
: All All
: -- normal (vote)
: ---
Assigned To: Unassigned bug - please assign me!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-12 13:38 UTC by Martin Williams
Modified: 2012-03-12 23:01 UTC (History)
0 users

See Also:
Category: Bug


Attachments
Patch to resolve issue (551 bytes, patch)
2012-03-12 13:38 UTC, Martin Williams
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Williams 2012-03-12 13:38:07 UTC
Created attachment 7648 [details]
Patch to resolve issue

User starts to 'Save Playlist' using 'Save Playlist' option in extras menu,
but then chooses to back out without actually saving playlist.

This generally breaks the ip3k menu - screen will go blank. I am also
finding that I usually (always?) need to restart the server in this
circumstance.


This appears to be an unintended side effect of:
http://svn.slimdevices.com/slim?view=revision&revision=29149.

Refer bug #15001, which was reopened as a result of this breakage.
(Comment #12).

That revision introduced an additional 'popModeRight' which is executed
when the playlist mode is exited. The mode may be exited either after saving
the playlist, or as a consequence of the user backing out.

In the latter case, the effect is now to back up one more level than is
actually required. This breaks use via the extras menu, which is only
two 'popModeRights' from the Home menu. Removal of one 'popModeRight' from
'savePluginCallback' 'backspace type' was also required. 


The attached patch resolves the issue on my local LMS.
Comment 1 Michael Herger 2012-03-12 23:01:56 UTC

*** This bug has been marked as a duplicate of bug 15001 ***