Review Request 112802: Extend dbus unterface for reading and writing the rating of the current song

[Alex Busenius]

> On Sept. 18, 2013, 9:46 p.m., Alex Merry wrote:
> > I'm torn between following Amarok's internals (which you do) and following MPRIS2, meaning it should be called UserRating and return/allow a double between 0.0 and 1.0.
> > 
> > There's also the question of how urgent this is, as there's discussion on the MPRIS list (which had stalled for a few months, but I started up again) about adding a general way of editing Metadata entries.

I don't see anyy sign of UserRating in MPRIS2 specs: http://specifications.freedesktop.org/mpris-spec/latest/... Can you point me to the right place? If there is a more standard way to set rating, I would prefer doing it that way.

A general way of editing metadata would be great, but would it not be better to handle rating differently (e.g. standardize range 0.0 ... 1.0 and let applications map their understanding of rating into that range)?


- Alex


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112802/#review40313
-----------------------------------------------------------


On Sept. 18, 2013, 7:34 p.m., Alex Busenius wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112802/
> -----------------------------------------------------------
> 
> (Updated Sept. 18, 2013, 7:34 p.m.)
> 
> 
> Review request for Amarok and Alex Merry.
> 
> 
> Description
> -------
> 
> The D-Bus interface is currently missing convenient method for reading and setting user rating (the stars). The proposed patch adds an Mpris2 extension to /org/mpris/MediaPlayer2 for that: 
> 
> property readwrite int org.kde.amarok.Mpris2Extensions.Player.Rating
> 
> 
> Diffs
> -----
> 
>   src/dbus/mpris2/MediaPlayer2AmarokExtensions.h 964c7b8 
>   src/dbus/mpris2/MediaPlayer2AmarokExtensions.cpp ba8b54e 
>   src/dbus/mpris2/org.kde.amarok.Mpris2Extensions.Player.xml 3397df4 
> 
> Diff: http://git.reviewboard.kde.org/r/112802/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alex Busenius
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130919/4cb6650b/attachment.html>