[gnome-music/wip/jfelder/coverstack-destroy] coverstack: Do not update the cover on destroy

[Jean Felder <jfelder src gnome org>]

commit fb3f423274dad37318e4e7f292eebedbf42131aa
Author: Jean Felder <jfelder src gnome org>
Date:   Sat Mar 14 19:50:23 2020 +0100

    coverstack: Do not update the cover on destroy
    
    On rare occasions, the CoverStack can be destroyed while the art is
    updated. This can result in an error once the art is retrieved because
    the CoverStack does not have children anymore.

 gnomemusic/widgets/coverstack.py | 11 +++++++++++
 1 file changed, 11 insertions(+)
---
diff --git a/gnomemusic/widgets/coverstack.py b/gnomemusic/widgets/coverstack.py
index 72d0a3af..88da698b 100644
--- a/gnomemusic/widgets/coverstack.py
+++ b/gnomemusic/widgets/coverstack.py
@@ -67,6 +67,8 @@ class CoverStack(Gtk.Stack):
         self.props.transition_type = Gtk.StackTransitionType.CROSSFADE
         self.props.visible_child_name = "loading"
 
+        self.connect("destroy", self._on_destroy)
+
         self.show_all()
 
     @GObject.Property(type=object, flags=GObject.ParamFlags.READWRITE)
@@ -134,3 +136,12 @@ class CoverStack(Gtk.Stack):
         self._art = None
 
         self.emit('updated')
+
+    def _on_destroy(self, widget):
+        # If the CoverStack is destroyed while the art is updated,
+        # an error can be coccur once the art is retrieved because
+        # the CoverStack does not have children anymore.
+        if (self._art is not None
+                and self._handler_id is not None):
+            self._art.disconnect(self._handler_id)
+            self._handler_id = None