[gnome-music/wip/jfelder/mpris-cleanup: 7/13] mpris: Make playlists attribute internal



commit 22b30e350513f36a49c4c4f533cf782d75b8c0d4
Author: Jean Felder <jfelder src gnome org>
Date:   Thu Mar 21 08:57:14 2019 +0100

    mpris: Make playlists attribute internal

 gnomemusic/mpris.py | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)
---
diff --git a/gnomemusic/mpris.py b/gnomemusic/mpris.py
index a04cb103..fa4fc567 100644
--- a/gnomemusic/mpris.py
+++ b/gnomemusic/mpris.py
@@ -293,7 +293,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._playlists = []
         self._player_previous_type = None
         self._path_list = []
         self._metadata_list = []
@@ -417,14 +417,14 @@ class MPRIS(DBusInterface):
 
     @log
     def _get_playlist_from_dbus_path(self, playlist_path):
-        for playlist in self.playlists:
+        for playlist in self._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._playlists:
             if playlist_id == playlist.get_id():
                 return playlist
         return None
@@ -548,7 +548,7 @@ class MPRIS(DBusInterface):
     @log
     def _reload_playlists(self):
         def query_playlists_callback(playlists):
-            self.playlists = playlists
+            self._playlists = playlists
             self._properties_changed(
                 MPRIS.MEDIA_PLAYER2_PLAYLISTS_IFACE,
                 {
@@ -678,7 +678,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._playlists]
 
         if not reverse:
             return playlists[index:index + max_count]
@@ -739,7 +739,7 @@ class MPRIS(DBusInterface):
             }
         elif interface_name == MPRIS.MEDIA_PLAYER2_PLAYLISTS_IFACE:
             return {
-                'PlaylistCount': GLib.Variant('u', len(self.playlists)),
+                'PlaylistCount': GLib.Variant('u', len(self._playlists)),
                 '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]