From: Marcel van der Veldt <m.vanderveldt@outlook.com>
Date: Fri, 9 Aug 2024 12:07:53 +0000 (+0200)
Subject: Fix onboard done info
X-Git-Url: https://git.kitaultman.com/?a=commitdiff_plain;h=965de438e1bf0bd81726c3bd05b387cbdc39045a;p=music-assistant-server.git

Fix onboard done info
---

diff --git a/music_assistant/constants.py b/music_assistant/constants.py
index 8df52aea..c949429e 100644
--- a/music_assistant/constants.py
+++ b/music_assistant/constants.py
@@ -65,7 +65,6 @@ CONF_ANNOUNCE_VOLUME_MAX: Final[str] = "announce_volume_max"
 CONF_ICON: Final[str] = "icon"
 CONF_LANGUAGE: Final[str] = "language"
 CONF_SAMPLE_RATES: Final[str] = "sample_rates"
-CONF_ONBOARD_DONE: Final[str] = "onboard_done"
 
 # config default values
 DEFAULT_HOST: Final[str] = "0.0.0.0"
diff --git a/music_assistant/server/controllers/config.py b/music_assistant/server/controllers/config.py
index 9165e40b..5af24ec3 100644
--- a/music_assistant/server/controllers/config.py
+++ b/music_assistant/server/controllers/config.py
@@ -30,7 +30,6 @@ from music_assistant.common.models.enums import EventType, PlayerState, Provider
 from music_assistant.common.models.errors import InvalidDataError, PlayerUnavailableError
 from music_assistant.constants import (
     CONF_CORE,
-    CONF_ONBOARD_DONE,
     CONF_PLAYERS,
     CONF_PROVIDERS,
     CONF_SERVER_ID,
@@ -74,7 +73,6 @@ class ConfigController:
         """Async initialize of controller."""
         await self._load()
         self.initialized = True
-        self.set_default(CONF_ONBOARD_DONE, len(self._data.get(CONF_PROVIDERS, {})) > 0)
         # create default server ID if needed (also used for encrypting passwords)
         self.set_default(CONF_SERVER_ID, uuid4().hex)
         server_id: str = self.get(CONF_SERVER_ID)
@@ -88,7 +86,7 @@ class ConfigController:
     @property
     def onboard_done(self) -> bool:
         """Return True if onboarding is done."""
-        return self.get(CONF_ONBOARD_DONE, False)
+        return len(self._data.get(CONF_PROVIDERS, {})) > 0
 
     async def close(self) -> None:
         """Handle logic on server stop."""