[gnome-music/wip/mschraal/type-checking: 1/2] grltrackerwrapper: Cleanup art retrieval
- From: Marinus Schraal <mschraal src gnome org>
- To: commits-list gnome org
- Cc: 
- Subject: [gnome-music/wip/mschraal/type-checking: 1/2] grltrackerwrapper: Cleanup art retrieval
- Date: Wed, 21 Oct 2020 14:35:11 +0000 (UTC)
commit 3a3229e12592f0f92bc732a22df244f0e3ca9c0f
Author: Marinus Schraal <mschraal gnome org>
Date:   Wed Oct 21 16:26:32 2020 +0200
    grltrackerwrapper: Cleanup art retrieval
    
    Art retrieval callbacks were convoluted. Do some cleanups and
    simplifications.
 gnomemusic/grilowrappers/grltrackerwrapper.py | 23 ++++++-----------------
 1 file changed, 6 insertions(+), 17 deletions(-)
---
diff --git a/gnomemusic/grilowrappers/grltrackerwrapper.py b/gnomemusic/grilowrappers/grltrackerwrapper.py
index 50b1655b..2954f3c6 100644
--- a/gnomemusic/grilowrappers/grltrackerwrapper.py
+++ b/gnomemusic/grilowrappers/grltrackerwrapper.py
@@ -1150,16 +1150,11 @@ class GrlTrackerWrapper(GObject.GObject):
                 coresong.props.thumbnail = "generic"
                 return
 
-            if (remaining == 0
-                    and queried_media is None):
+            if queried_media is None:
                 coresong.props.thumbnail = "generic"
                 return
 
-            thumbnail_uri = queried_media.get_thumbnail()
-            if thumbnail_uri is None:
-                coresong.props.thumbnail = "generic"
-            else:
-                StoreArt(coresong, thumbnail_uri)
+            StoreArt(coresong, queried_media.get_thumbnail())
 
         song_id = media.get_id()
         query = self._get_album_for_media_id_query(song_id)
@@ -1181,17 +1176,11 @@ class GrlTrackerWrapper(GObject.GObject):
                 corealbum.props.thumbnail = "generic"
                 return
 
-            if (remaining == 0
-                    and queried_media is None):
-                # art_retrieved_cb should be called two times, the
-                # empty result can be ignored.
+            if queried_media is None:
+                corealbum.props.thumbnail = "generic"
                 return
 
-            thumbnail_uri = queried_media.get_thumbnail()
-            if thumbnail_uri is None:
-                corealbum.props.thumbnail = "generic"
-            else:
-                StoreArt(corealbum, thumbnail_uri)
+            StoreArt(corealbum, queried_media.get_thumbnail())
 
         album_id = media.get_id()
         query = self._get_album_for_media_id_query(album_id, False)
@@ -1215,7 +1204,7 @@ class GrlTrackerWrapper(GObject.GObject):
                 coreartist.props.thumbnail = "generic"
                 return
 
-            if resolved_media.get_thumbnail() is None:
+            if resolved_media is None:
                 coreartist.props.thumbnail = "generic"
                 return
 
[
Date Prev][
Date Next]   [
Thread Prev][
Thread Next]   
[
Thread Index]
[
Date Index]
[
Author Index]