From 94c431048cc6e4429c24bf1b296fb9d03d264599 Mon Sep 17 00:00:00 2001 From: Marcel van der Veldt Date: Wed, 5 Apr 2023 16:27:38 +0200 Subject: [PATCH] skip matching on unique/file providers as it is pointless --- music_assistant/server/controllers/media/albums.py | 3 +++ music_assistant/server/controllers/media/artists.py | 3 +++ music_assistant/server/controllers/media/tracks.py | 3 +++ 3 files changed, 9 insertions(+) diff --git a/music_assistant/server/controllers/media/albums.py b/music_assistant/server/controllers/media/albums.py index 3bb2448e..e2c23b0a 100644 --- a/music_assistant/server/controllers/media/albums.py +++ b/music_assistant/server/controllers/media/albums.py @@ -429,6 +429,9 @@ class AlbumsController(MediaControllerBase[Album]): continue if ProviderFeature.SEARCH not in provider.supported_features: continue + if provider.is_unique: + # matching on unique provider sis pointless as they push (all) their content to MA + continue if await find_prov_match(provider): cur_provider_domains.add(provider.domain) else: diff --git a/music_assistant/server/controllers/media/artists.py b/music_assistant/server/controllers/media/artists.py index cd0f25fb..e8458636 100644 --- a/music_assistant/server/controllers/media/artists.py +++ b/music_assistant/server/controllers/media/artists.py @@ -196,6 +196,9 @@ class ArtistsController(MediaControllerBase[Artist]): continue if ProviderFeature.SEARCH not in provider.supported_features: continue + if provider.is_unique: + # matching on unique provider sis pointless as they push (all) their content to MA + continue if await self._match(db_artist, provider): cur_provider_domains.add(provider.domain) else: diff --git a/music_assistant/server/controllers/media/tracks.py b/music_assistant/server/controllers/media/tracks.py index d1eecaf5..fdbb1fdd 100644 --- a/music_assistant/server/controllers/media/tracks.py +++ b/music_assistant/server/controllers/media/tracks.py @@ -212,6 +212,9 @@ class TracksController(MediaControllerBase[Track]): for provider in self.mass.music.providers: if ProviderFeature.SEARCH not in provider.supported_features: continue + if provider.is_unique: + # matching on unique provider sis pointless as they push (all) their content to MA + continue self.logger.debug( "Trying to match track %s on provider %s", db_track.name, provider.name ) -- 2.34.1