Review Request 114892: Plasma-mediacenter plays songs or videos: passed as arguments

Shantanu Tushar shantanu at kde.org
Wed Jan 8 14:00:21 UTC 2014


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


Seems to work as expected, though the code is more complicated than it ideally should be, mostly due to multiple playlists being not-so-good. Hope we will fix this for the next release.
There is one issue that still remains, post fixing that, this can go in.

- Shantanu Tushar


On Jan. 8, 2014, 6:09 a.m., Sujith Haridasan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114892/
> -----------------------------------------------------------
> 
> (Updated Jan. 8, 2014, 6:09 a.m.)
> 
> 
> Review request for Plasma, Shantanu Tushar and Sinny Kumari.
> 
> 
> Repository: plasma-mediacenter
> 
> 
> Description
> -------
> 
> PMC can play songs from the command line or when selected from file browsers like Dolphin( using "Open With" ). PMC creates a separate playlist called "Misc" which will store the files which are tried to open from command line or so. Name given for the playlist is a temperory one ("Misc"). We can discuss here and rename it to something meaningful. I have tested this functionality and it works. 
> 
> 
> Diffs
> -----
> 
>   shells/newshell/mainwindow.cpp b2793df 
>   mediaelements/playlist/MultiplePlaylists.qml 42877f0 
>   libs/mediacenter/multipleplaylistmodel.h 30b317b 
>   libs/mediacenter/playlistmodel.cpp 00a0cee 
>   libs/mediacenter/playlistmodel.h 3993ac3 
>   libs/mediacenter/multipleplaylistmodel.cpp d52fa66 
> 
> Diff: https://git.reviewboard.kde.org/r/114892/diff/
> 
> 
> Testing
> -------
> 
> Open pmc from the command line (or from Dolphin) with multiple files to be opened:
> 1) PMC starts playing the songs/videos with the ones passed.
> With Dolphin I have tested with only one file and it worked properly.
> 
> 
> Thanks,
> 
> Sujith Haridasan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140108/b2628d6d/attachment-0001.html>


More information about the Plasma-devel mailing list