Bug 10395 - "power 0" switches off all SqueezeBoxes
: "power 0" switches off all SqueezeBoxes
Status: NEW
Product: Logitech Media Server
Classification: Unclassified
Component: xPL
: 7.4.0
: PC Other
: -- normal (vote)
: Future
Assigned To: Unassigned bug - please assign me!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-12-19 09:08 UTC by Timo Krieger
Modified: 2011-11-06 23:23 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Timo Krieger 2008-12-19 09:08:18 UTC
The audio.basic command switches off all connected SqueezeBoxes and not only the one named in as instance.
The xPL heartbeat follows for every connected SqueezeBox which has been switched off.
Comment 1 Chris Owens 2008-12-22 10:07:40 UTC
Chris to try to figure out who wrote xpl and see if they want to fix it.  Otherwise, we should consider removing xpl if we can't support it.
Comment 2 Chris Owens 2009-01-09 08:36:10 UTC
Michael, The log does not seem to go back far enough to determine the original author of this plugin.  I'm not an expert at svn.  Am I missing something?

Looking at http://svn.slimdevices.com/7.4/trunk/server/Slim/Plugin/xPL/Plugin.pm?view=log

In any case, we're not going to change the feature set in a point release.  Changing target to the next major release.

Comment 3 Ben Klaas 2009-08-26 07:46:44 UTC
this is an administrative shuffle on priority fields to help make better judgment on the top end of the priority list. P4->P5, P3->P4, and P2->P3.
Comment 4 KDF 2009-08-26 16:52:17 UTC
Michael, Chris...for reference.

A log for Xpl going back to when it was moved to a Plugin:
http://svn.slimdevices.com/slim/trunk/server/Plugins/xPL.pm?view=log&pathrev=10000

leg entry for svn457 suggests "John Bent" created the original module.  

The audio.basic is handled by the sub, handleAudioMessage, which defaults to ALL clients if none is provided.  Might be useful to start with a few debug lines in there to capture what's being sent in this case.  Perhaps verifying that the loop in checkInstances is finding a valid client to match the instance name given.
Comment 5 Chris Owens 2009-10-09 10:54:35 UTC
If this gets additional votes we can consider working on it.  Otherwise, patches from the community are welcome.
Comment 6 Alan Young 2011-11-06 23:23:39 UTC
Unassigned bugs cannot have a priority.