[gnome-music/wip/jfelder/restore-mpris-playlists: 7/13] mpris: Use a correct identifier for playlists



commit d67b9989929d878a1a1e9880831bbf1c9b8b568d
Author: Jean Felder <jfelder src gnome org>
Date:   Thu Sep 26 18:16:03 2019 +0200

    mpris: Use a correct identifier for playlists
    
    According to mpris specifications: "Media players may not use any
    paths starting with /org/mpris".
    If the playlist is not valid, It is suggested to use "/" as the
    playlist id.

 gnomemusic/mpris.py | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)
---
diff --git a/gnomemusic/mpris.py b/gnomemusic/mpris.py
index 8cd51642..962465c0 100644
--- a/gnomemusic/mpris.py
+++ b/gnomemusic/mpris.py
@@ -470,13 +470,12 @@ class MPRIS(DBusInterface):
         :return: a D-Bus id to uniquely identify the playlist
         :rtype: str
         """
-        # Smart Playlists do not have an id
-        if playlist:
-            pl_id = playlist.props.pl_id or playlist.props.tag_text
-        else:
-            pl_id = "Invalid"
+        if not playlist:
+            return "/"
 
-        return "/org/mpris/MediaPlayer2/Playlist/{}".format(pl_id)
+        # Smart Playlists do not have an id
+        pl_id = playlist.props.pl_id or playlist.props.tag_text
+        return "/org/gnome/GnomeMusic/Playlist/{}".format(pl_id)
 
     @log
     def _get_mpris_playlist_from_playlist(self, playlist):


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