Skip to content
This repository has been archived by the owner on May 17, 2021. It is now read-only.

Commit

Permalink
Squeezebox Binding: Sync "remove" players incorrect
Browse files Browse the repository at this point in the history
The "sync:XXX" binding command for multi-player playback sync & its
respective OFF command is incorrect. It's removing the source player and
not the target player.
As the LMS "sync -" CLI command only supports a single playerID then it
should be the target synced player and not the source one.

Also means that when re-adding another player into another player group
the OFF command will always remove the correct respective player.
  • Loading branch information
wezhunter committed Sep 23, 2014
1 parent affdf50 commit c8fdf88
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ else if (command.equals(OnOffType.OFF))
if (command.equals(OnOffType.ON))
squeezeServer.syncPlayer(playerId, bindingConfig.getExtra());
else if (command.equals(OnOffType.OFF))
squeezeServer.unSyncPlayer(playerId);
squeezeServer.unSyncPlayer(bindingConfig.getExtra());
break;

default:
Expand Down

0 comments on commit c8fdf88

Please sign in to comment.