From: Marcel van der Veldt <m.vanderveldt@outlook.com>
Date: Mon, 3 Apr 2023 08:26:41 +0000 (+0200)
Subject: available only
X-Git-Url: https://git.kitaultman.com/?a=commitdiff_plain;h=ebcc359ada8daba5ed8fb0fef64388f4d493ca6d;p=music-assistant-server.git

available only
---

diff --git a/music_assistant/server/controllers/media/base.py b/music_assistant/server/controllers/media/base.py
index 2ceb641d..1f35f066 100644
--- a/music_assistant/server/controllers/media/base.py
+++ b/music_assistant/server/controllers/media/base.py
@@ -284,18 +284,15 @@ class MediaControllerBase(Generic[ItemCls], metaclass=ABCMeta):
         if not getattr(item, "provider_mappings", None):
             # make sure we have a full object
             item = await self.get_db_item(item.item_id)
-        for prefer_available in (True, False):
-            for prefer_unique in (True, False):
-                for prov_mapping in item.provider_mappings:
-                    # returns the first provider that is available
-                    if prefer_available and not prov_mapping.available:
+        for prefer_unique in (True, False):
+            for prov_mapping in item.provider_mappings:
+                # returns the first provider that is available
+                if not prov_mapping.available:
+                    continue
+                if provider := self.mass.get_provider(prov_mapping.provider_instance):
+                    if prefer_unique and not provider.is_unique:
                         continue
-                    if provider := self.mass.get_provider(
-                        prov_mapping.provider_instance, return_unavailable=not prefer_available
-                    ):
-                        if prefer_unique and not provider.is_unique:
-                            continue
-                        return (prov_mapping.provider_instance, prov_mapping.item_id)
+                    return (prov_mapping.provider_instance, prov_mapping.item_id)
         return (None, None)
 
     async def get_db_items_by_query(