Review Request: Bug 261062 - JJ: Mark episodes to keep

Lucas Gomes x8lucas8x at gmail.com
Tue Jan 31 01:20:37 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100998/
-----------------------------------------------------------

(Updated Jan. 31, 2012, 1:20 a.m.)


Review request for Amarok.


Changes
-------

Fixed some function declaration problems.


Description
-------

Now downloaded episodes are kept in HD, if "keep episode" checkbox is checked.
These "kept" episodes are shown in the list regardless of the purgecount, even if they
are not downloaded.

BUG Description:

Currently downloaded files are not kept when the episode is no longer shown in
the Podcast section. This behavior can not be configured but is tied to the
purge settings of each channel.

It's possible to add a "keep downloaded file" checkbox to episodes to prevent
the deletion from happening. We should perhaps also show these "kept" episodes
in the list regardless of the purgecount. So even if a channel has purgecount 5
and 3 episodes marked to keep it will show 5 + maximum 3 older kept episodes.

Reproducible: Always


This addresses bug 261062.
    https://bugs.kde.org/show_bug.cgi?id=261062


Diffs (updated)
-----

  src/browsers/playlistbrowser/PodcastModel.h e88f4a1 
  src/browsers/playlistbrowser/PodcastModel.cpp 18334f6 
  src/core-impl/podcasts/sql/SqlPodcastMeta.h 42ad039 
  src/core-impl/podcasts/sql/SqlPodcastMeta.cpp 1c3bdf4 
  src/core-impl/podcasts/sql/SqlPodcastProvider.h c3d5e56 
  src/core-impl/podcasts/sql/SqlPodcastProvider.cpp 183005f 
  src/core/podcasts/PodcastMeta.h 679f7ac 
  src/playlistmanager/SyncedPodcast.h c186df2 

Diff: http://git.reviewboard.kde.org/r/100998/diff/diff


Testing
-------


Thanks,

Lucas Gomes

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20120131/85e09574/attachment.html>


More information about the Amarok-devel mailing list