Skip to content

Commit

Permalink
Fix typo in sync players logic
Browse files Browse the repository at this point in the history
  • Loading branch information
marcelveldt committed Oct 22, 2024
1 parent 06f6657 commit 642b2ba
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion music_assistant/server/controllers/players.py
Original file line number Diff line number Diff line change
Expand Up @@ -685,7 +685,7 @@ async def cmd_sync_many(self, target_player: str, child_player_ids: list[str]) -
# forward command to the player provider after all (base) sanity checks
player_provider = self.get_player_provider(target_player)
async with self._player_throttlers[target_player]:
await player_provider.cmd_sync_many(target_player, child_player_ids)
await player_provider.cmd_sync_many(target_player, final_player_ids)

@api_command("players/cmd/unsync_many")
async def cmd_unsync_many(self, player_ids: list[str]) -> None:
Expand Down
2 changes: 0 additions & 2 deletions music_assistant/server/providers/sonos/provider.py
Original file line number Diff line number Diff line change
Expand Up @@ -188,8 +188,6 @@ async def cmd_sync(self, player_id: str, target_player: str) -> None:
async def cmd_sync_many(self, target_player: str, child_player_ids: list[str]) -> None:
"""Create temporary sync group by joining given players to target player."""
sonos_player = self.sonos_players[target_player]
# ensure we only send valid (and unique) player ids
child_player_ids = list({x for x in child_player_ids if x in self.sonos_players})
await sonos_player.client.player.group.modify_group_members(
player_ids_to_add=child_player_ids, player_ids_to_remove=[]
)
Expand Down

0 comments on commit 642b2ba

Please sign in to comment.