[gnome-music/wip/mschraal/grltrackerwrapper-check-smart-playlist-crash: 3/3] grltrackerwrapper: Fix early playlists change check crash



commit 1b28202506ea1b29f26a5bdd1633349427493435
Author: Marinus Schraal <mschraal gnome org>
Date:   Thu Apr 16 15:53:18 2020 +0200

    grltrackerwrapper: Fix early playlists change check crash
    
    In 41c60ec7c3 the playlist wrapper creation gets delayed until after the
    initial songs list has filled. It is now possible however that on larger
    collections change tracking is already starting on the songs list before the
    playlist wrapper is loaded. This can cause a crash when checking for
    smart playlist changes on an uninitialized GrlTrackerPlaylists object.
    
    Add a check for existence of the GrlTrackerPlaylists object to avoid
    this.
    
    Fixes: #382

 gnomemusic/grilowrappers/grltrackerwrapper.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
---
diff --git a/gnomemusic/grilowrappers/grltrackerwrapper.py b/gnomemusic/grilowrappers/grltrackerwrapper.py
index 4997640e..4a4bcc52 100644
--- a/gnomemusic/grilowrappers/grltrackerwrapper.py
+++ b/gnomemusic/grilowrappers/grltrackerwrapper.py
@@ -159,7 +159,8 @@ class GrlTrackerWrapper(GObject.GObject):
 
         self._check_album_change()
         self._check_artist_change()
-        self._tracker_playlists.check_smart_playlist_change()
+        if self._tracker_playlists is not None:
+            self._tracker_playlists.check_smart_playlist_change()
 
         self._batch_changed_media_ids = {}
         self._content_changed_timeout = None


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