Review Request 119692: Properly restore the selected player after a plugin reconnect.

Job Noorman jobnoorman at gmail.com
Wed Aug 13 16:03:29 UTC 2014



> On Aug. 13, 2014, 3:50 p.m., Albert Vaca Cintora wrote:
> > Looks good! Do you have a developer account to push it to the repo? If not I can do it in your name.

I don't have a developer account so if you could push it for me, that would be great!


- Job


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119692/#review64484
-----------------------------------------------------------


On Aug. 10, 2014, 12:21 p.m., Job Noorman wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119692/
> -----------------------------------------------------------
> 
> (Updated Aug. 10, 2014, 12:21 p.m.)
> 
> 
> Review request for kdeconnect.
> 
> 
> Repository: kdeconnect-android
> 
> 
> Description
> -------
> 
> Properly restore the selected MPRIS player after a plugin reconnect.
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/kde/kdeconnect/Plugins/MprisPlugin/MprisPlugin.java 4ad57beaa773722c56e9f1245a3712c8ea25b479 
>   src/main/java/org/kde/kdeconnect/Plugins/MprisPlugin/MprisActivity.java fe5989ccd9fe2b33e31ba6c4badfb8b6f30257fe 
> 
> Diff: https://git.reviewboard.kde.org/r/119692/diff/
> 
> 
> Testing
> -------
> 
> Whenever the link with a device goes down (the easiest way to test this is to lock the phone's screen), the connection with the MPRIS plugin is re-established when the link goes up again. This patch makes sure that the currently selected player (the one stored on the MprisPlugin instance) is also selected in the MprisActivity. Currently, the first player in the list is always selected.
> 
> 
> Thanks,
> 
> Job Noorman
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20140813/ef262eb5/attachment.html>


More information about the KDEConnect mailing list