Merge pull request #678 from sudoAlphaX/master

fix: fail safely if unable to get player status (Media Control)
slgobinath · Feb 15, 2025 · 6a0be4a · 6a0be4a
2 parents 6061843 + 6349949
commit 6a0be4a
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/safeeyes/plugins/mediacontrol/plugin.py b/safeeyes/plugins/mediacontrol/plugin.py
@@ -59,10 +59,16 @@ def __active_players():
                 cancellable=None,
             )
 
-            status = player.get_cached_property('PlaybackStatus').unpack().lower()
+            playbackstatus = player.get_cached_property("PlaybackStatus")
+
+            if playbackstatus is not None:
+                status = playbackstatus.unpack().lower()
+
+                if status == "playing":
+                    players.append(player)
+            else:
+                logging.warning(f"Failed to get PlaybackStatus for {service}")
 
-            if status == "playing":
-                players.append(player)
     return players