From: Santiago Soto <santiago@soto.uy>
Date: Fri, 14 Jun 2024 19:58:33 +0000 (-0300)
Subject: Snapcast add guard for sync_many on cmd_sync (#1363)
X-Git-Url: https://git.kitaultman.com/?a=commitdiff_plain;h=faef3b60dd5708772be396363af7368b83f35f37;p=music-assistant-server.git

Snapcast add guard for sync_many on cmd_sync (#1363)

refactor cmd_sync
---

diff --git a/music_assistant/server/providers/snapcast/__init__.py b/music_assistant/server/providers/snapcast/__init__.py
index ae45bf5e..2c307a16 100644
--- a/music_assistant/server/providers/snapcast/__init__.py
+++ b/music_assistant/server/providers/snapcast/__init__.py
@@ -341,8 +341,9 @@ class SnapCastProvider(PlayerProvider):
     async def cmd_sync(self, player_id: str, target_player: str) -> None:
         """Sync Snapcast player."""
         group = self._get_snapgroup(target_player)
-        await group.add_client(self._get_snapclient_id(player_id))
-        self._handle_update()
+        if self._get_snapclient_id(player_id) not in group.clients:
+            await group.add_client(self._get_snapclient_id(player_id))
+            self._handle_update()
 
     async def cmd_unsync(self, player_id: str) -> None:
         """Unsync Snapcast player."""