Review Request 118767: Fix for "Bug 328532 - Remember the playlist I was using when I closed PMC last time"

[R.Harish Navnit]

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


Right now, when you play a song from a playlist and wish to view the songs in another playlist, the currentIndex is lost(is being set to -1, I believe) when you select another playlist. As a result of this when you return to the playlist the next button plays the first song in the list instead of the song next to the currently playing song in the list. I think this shouldn't happen. Can you try fixing the issue(if this is something that needs to be fixed) with the currentIndex along with this patch ? 

P.S : I don't know if it must be fixed(along with this, or be fixed at all !). Just a request, if you will :)

- R.Harish  Navnit


On June 15, 2014, 4:41 p.m., Ashish Madeti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118767/
> -----------------------------------------------------------
> 
> (Updated June 15, 2014, 4:41 p.m.)
> 
> 
> Review request for Plasma, Bhushan Shah, Nikolaos Chatzidakis, Shantanu Tushar, Sinny Kumari, and Sujith Haridasan.
> 
> 
> Bugs: 328532
>     http://bugs.kde.org/show_bug.cgi?id=328532
> 
> 
> Repository: plasma-mediacenter
> 
> 
> Description
> -------
> 
> PMC didn't used to remember the playlist I was on the last time, when restarting it.
> Now it saves the "lastPlaylist" while closing in the settings, and when starting it loads the value of "lastPlaylist" in settings
> 
> 
> Diffs
> -----
> 
>   libs/mediacenter/playlistmodel.cpp 0525708 
> 
> Diff: https://git.reviewboard.kde.org/r/118767/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ashish Madeti
> 
>

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