[gnome-music/wip/mschraal/type-checking: 1/2] grltrackerwrapper: Do not pass data on resolve




commit b622abb3af3231bf84134ed7a0370c65674e8d6d
Author: Marinus Schraal <mschraal gnome org>
Date:   Wed Oct 21 16:41:25 2020 +0200

    grltrackerwrapper: Do not pass data on resolve
    
    It is None, it is confusing with regards to the callback arguments order
    and it is in line with all the other Grilo callbacks in Music.

 gnomemusic/grilowrappers/grltrackerwrapper.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
---
diff --git a/gnomemusic/grilowrappers/grltrackerwrapper.py b/gnomemusic/grilowrappers/grltrackerwrapper.py
index 2954f3c6..bf151dc8 100644
--- a/gnomemusic/grilowrappers/grltrackerwrapper.py
+++ b/gnomemusic/grilowrappers/grltrackerwrapper.py
@@ -1198,7 +1198,7 @@ class GrlTrackerWrapper(GObject.GObject):
         """
         media = coreartist.props.media
 
-        def art_resolved_cb(source, op_id, resolved_media, data, error):
+        def art_resolved_cb(source, op_id, resolved_media, error):
             if error:
                 self._log.warning("Error: {}".format(error))
                 coreartist.props.thumbnail = "generic"
@@ -1212,7 +1212,7 @@ class GrlTrackerWrapper(GObject.GObject):
 
         self.props.source.resolve(
             media, [Grl.METADATA_KEY_THUMBNAIL], self._full_options,
-            art_resolved_cb, None)
+            art_resolved_cb)
 
     def stage_playlist_deletion(self, playlist):
         """Prepares playlist deletion.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]