> 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 > >
_______________________________________________ Plasma-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/plasma-devel
