[gnome-music/wip/jfelder/mpris-cleanup: 98/117] mpris: Rename playlists attributes



commit 3df9572a3985f10efc9722cf792a5b2b0fdb57a4
Author: Jean Felder <jfelder src gnome org>
Date:   Mon Apr 15 08:36:17 2019 +0200

    mpris: Rename playlists attributes
    
    Avoid confusion between the Playlists object and the playlists
    populated by a query.

 gnomemusic/mpris.py | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)
---
diff --git a/gnomemusic/mpris.py b/gnomemusic/mpris.py
index 29e11b6a..634e7565 100644
--- a/gnomemusic/mpris.py
+++ b/gnomemusic/mpris.py
@@ -267,7 +267,7 @@ class MPRIS(DBusInterface):
         playlists.connect('playlist-created', self._on_playlists_count_changed)
         playlists.connect('playlist-deleted', self._on_playlists_count_changed)
         grilo.connect('ready', self._on_grilo_ready)
-        self.playlists = []
+        self._stored_playlists = []
         self._player_previous_type = None
         self._path_list = []
         self._metadata_list = []
@@ -398,14 +398,14 @@ class MPRIS(DBusInterface):
 
     @log
     def _get_playlist_from_dbus_path(self, playlist_path):
-        for playlist in self.playlists:
+        for playlist in self._stored_playlists:
             if playlist_path == self._get_playlist_dbus_path(playlist):
                 return playlist
         return None
 
     @log
     def _get_playlist_from_id(self, playlist_id):
-        for playlist in self.playlists:
+        for playlist in self._stored_playlists:
             if playlist_id == playlist.get_id():
                 return playlist
         return None
@@ -517,7 +517,7 @@ class MPRIS(DBusInterface):
     @log
     def _reload_playlists(self):
         def query_playlists_callback(playlists):
-            self.playlists = playlists
+            self._stored_playlists = playlists
             self.PropertiesChanged(MPRIS.MEDIA_PLAYER2_PLAYLISTS_IFACE,
                                    {
                                        'PlaylistCount': GLib.Variant('u', len(playlists)),
@@ -673,7 +673,7 @@ class MPRIS(DBusInterface):
 
         playlists = [(self._get_playlist_dbus_path(playlist),
                       utils.get_media_title(playlist), '')
-                     for playlist in self.playlists]
+                     for playlist in self._stored_playlists]
 
         if not reverse:
             return playlists[index:index + max_count]
@@ -738,8 +738,9 @@ class MPRIS(DBusInterface):
                 'CanEditTracks': GLib.Variant('b', False)
             }
         elif interface_name == MPRIS.MEDIA_PLAYER2_PLAYLISTS_IFACE:
+            playlist_count = len(self._stored_playlists)
             return {
-                'PlaylistCount': GLib.Variant('u', len(self.playlists)),
+                'PlaylistCount': GLib.Variant('u', playlist_count),
                 'Orderings': GLib.Variant('as', ['Alphabetical']),
                 'ActivePlaylist': GLib.Variant('(b(oss))', self._get_active_playlist()),
             }


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