Bug 229391 - No cover image for downloaded podcasts
Summary: No cover image for downloaded podcasts
Alias: None
Product: amarok
Classification: Unclassified
Component: Podcast (show other bugs)
Version: 2.3.1-GIT
Platform: Debian testing Linux
: NOR normal (vote)
Target Milestone: ---
Assignee: Amarok Developers
: 231836 240249 (view as bug list)
Depends on:
Reported: 2010-03-04 14:10 UTC by Marcel Dischinger
Modified: 2010-08-29 16:04 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 2.3.2


Note You need to log in before you can comment on or make changes to this bug.
Description Marcel Dischinger 2010-03-04 14:10:55 UTC
Version:           2.3-git (using KDE 4.3.4)
OS:                Linux
Installed from:    Debian testing/unstable Packages

For podcast episodes that are *not* downloaded, the cover image of the podcast is shown in the playlist as expected.
However, if I download a podcast episode and put it into the playlist, the cover image is not shown but the default image (the gray CD). If I remove the downloaded episode, the cover image is shown again in the playlist.
Comment 1 Myriam Schweingruber 2010-08-09 17:41:30 UTC
Still valid in latest git. The cover image is in the same folder as the episodes, so no reason it should not display.
Comment 2 Myriam Schweingruber 2010-08-12 00:16:09 UTC
*** Bug 231836 has been marked as a duplicate of this bug. ***
Comment 3 missive 2010-08-12 00:29:23 UTC
*** Bug 240249 has been marked as a duplicate of this bug. ***
Comment 4 Bart Cerneels 2010-08-29 13:05:35 UTC
commit 13dd2dd457ec68bdd98324fe1461a6f0b9e42b32
Author: Bart Cerneels <bart.cerneels@kde.org>
Date:   Sun Aug 29 13:05:02 2010 +0200

    Always use PodcastChannel image for episodes.

diff --git a/ChangeLog b/ChangeLog
index f9c41a0..0c6a404 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -12,6 +12,7 @@ VERSION 2.3.2
     * Use system date/time format for default name when saving user playlists.
+    * Also use podcast channel image for downloaded episodes. (BR 229391)
     * Collection directories that were symlinks could end up storing the wrong
       absolute path, causing those files to be removed during incremental
diff --git a/src/core-impl/podcasts/sql/SqlPodcastMeta.cpp b/src/core-impl/podcasts/sql/SqlPodcastMeta.cpp
index f1d6167..07ceae4 100644
--- a/src/core-impl/podcasts/sql/SqlPodcastMeta.cpp
+++ b/src/core-impl/podcasts/sql/SqlPodcastMeta.cpp
@@ -321,15 +321,6 @@ SqlPodcastEpisode::setTitle( const QString &title )
     m_title = title;
-SqlPodcastEpisode::album() const
-    if( m_localFile.isNull() )
-        return m_albumPtr;
-    return m_localFile->album();
 SqlPodcastEpisode::artist() const
diff --git a/src/core-impl/podcasts/sql/SqlPodcastMeta.h b/src/core-impl/podcasts/sql/SqlPodcastMeta.h
index 6409334..a6c95cf 100644
--- a/src/core-impl/podcasts/sql/SqlPodcastMeta.h
+++ b/src/core-impl/podcasts/sql/SqlPodcastMeta.h
@@ -64,7 +64,6 @@ class SqlPodcastEpisode : public Podcasts::PodcastEpisode
         virtual bool isEditable() const;
         virtual void finishedPlaying( double playedFraction );
-        virtual Meta::AlbumPtr album() const;
         virtual Meta::ArtistPtr artist() const;
         virtual Meta::ComposerPtr composer() const;
         virtual Meta::GenrePtr genre() const;
diff --git a/src/core/podcasts/PodcastImageFetcher.cpp b/src/core/podcasts/PodcastImageFetcher.cpp
index 15966a6..735b3d1 100644
--- a/src/core/podcasts/PodcastImageFetcher.cpp
+++ b/src/core/podcasts/PodcastImageFetcher.cpp
@@ -102,6 +102,7 @@ PodcastImageFetcher::run()
         debug() << "Solid reports we are not online, canceling podcast image download";
         emit( done( this ) );
+        //TODO: schedule another run after Solid reports we are online again