From: Maxim Raznatovski <nda.mr43@gmail.com>
Date: Sun, 9 Feb 2025 19:40:25 +0000 (+0100)
Subject: Fix: Delete DSPConfig when removing players (#1951)
X-Git-Url: https://git.kitaultman.com/?a=commitdiff_plain;h=f4b9dbeee992ba5b1ebfc05b35805bc754101a16;p=music-assistant-server.git

Fix: Delete DSPConfig when removing players (#1951)

Removing the player config now also removes the DSP config, since it's
stored in a different key.
---

diff --git a/music_assistant/controllers/config.py b/music_assistant/controllers/config.py
index 5acbaef8..62df0b10 100644
--- a/music_assistant/controllers/config.py
+++ b/music_assistant/controllers/config.py
@@ -407,6 +407,7 @@ class ConfigController:
     async def remove_player_config(self, player_id: str) -> None:
         """Remove PlayerConfig."""
         conf_key = f"{CONF_PLAYERS}/{player_id}"
+        dsp_conf_key = f"{CONF_PLAYER_DSP}/{player_id}"
         existing = self.get(conf_key)
         if not existing:
             msg = f"Player configuration for {player_id} does not exist"
@@ -435,6 +436,8 @@ class ConfigController:
         self.mass.players.remove(player_id, cleanup_config=False)
         # remove the actual config if all of the above passed
         self.remove(conf_key)
+        # Also remove the DSP config if it exists
+        self.remove(dsp_conf_key)
 
     @api_command("config/players/dsp/get")
     def get_player_dsp_config(self, player_id: str) -> DSPConfig: