After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 667139 - broken DBus method PlaylistManager.GetPlaylists
broken DBus method PlaylistManager.GetPlaylists
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: Programmatic interfaces
HEAD
Other Linux
: Normal normal
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-01-02 15:52 UTC by Michael Biebl
Modified: 2012-01-04 23:09 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Correctly return list of playlist names trough D-Bus (1.14 KB, patch)
2012-01-02 15:54 UTC, Michael Biebl 		none Details | Review

Description Michael Biebl 2012-01-02 15:52:30 UTC 
Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=653373

Rhythmbox does not return list of playlists names trough DBus

Attempt to call this method:
$ dbus-send --dest=org.gnome.Rhythmbox3 --print-reply
/org/gnome/Rhythmbox3/PlaylistManager
org.gnome.Rhythmbox3.PlaylistManager.GetPlaylists
Error org.freedesktop.DBus.Error.NoReply: Did not receive a reply. Possible
causes include: the remote application did not send a reply, the message bus
security policy blocked the reply, the reply timeout expired, or the network
connection was broken.

error in ~/.xsession-errors:
(rhythmbox:7750): GLib-GIO-CRITICAL **:
g_dbus_method_invocation_return_value_internal: assertion `(parameters == NULL)
|| g_variant_is_of_type (parameters, G_VARIANT_TYPE_TUPLE)' failed

It seems this problem affects upstream as well  (
http://git.gnome.org/browse/rhythmbox/tree/shell/rb-playlist-manager.c#n1639 )
..

Patch attached.
Comment 1 Michael Biebl 2012-01-02 15:54:49 UTC 
Created attachment 204455 [details] [review]
Correctly return list of playlist names trough D-Bus
Comment 2 Jonathan Matthew 2012-01-04 23:09:44 UTC 
I pushed a simpler fix (just use g_variant_new ("(^as)", names)) as commit f831c30