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

[Shantanu Tushar]

> On Jan. 7, 2014, 6:20 p.m., Shantanu Tushar wrote:
> > libs/mediacenter/playlistmodel.cpp, lines 306-307
> > <https://git.reviewboard.kde.org/r/114892/diff/1/?file=233345#file233345line306>
> >
> >     this path is calculated at two places, one here and one on lines 203-204. Try sharing code between checkPlaylistPathExists() and playlistFilePath()

This issue still exists.


- Shantanu


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


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/65827e08/attachment.html>