GNOME Bugzilla – Bug 380746
Podcasts should display album art
Last modified: 2007-01-23 08:08:11 UTC
Some podcasts have embedded album art. Rhythmbox should display this in the usual album art place.
Created attachment 79738 [details] [review] Add Podcast Cover Image support - Add cover art support for podcasts - Fix parsing "itunes:image" tags in feeds
Created attachment 79742 [details] [review] Add Podcast Cover Image support - Add cover art support for podcasts - Fix parsing "itunes:image" tags in feeds * Added a check to properly work only on podcast rhythmdb entries
Works for me on most of the Podcasts that have images.
If I under(In reply to comment #3) > Works for me on most of the Podcasts that have images. > I pick that from writing "most", some of those Podcasts you expect from having images don't show them with this patch. The podcast xml parser had sometimes skipped adding <image> entries to the rhythmdb podcast-feed prior to the patch. Thus you will most likely have to to re-subscribe to any of your current Podcasts in order for the rhythmdb entry to properly re-parse and add an <image> property. This could be fixed by altering "rb_podcast_manager_insert_feed()" to also set the data->* when updating the feed and not just when creating a new one. I can do the required change if that would be desired.
(In reply to comment #4) > > Works for me on most of the Podcasts that have images. > I pick that from writing "most", some of those Podcasts you expect from having > images don't show them with this patch. Sorry, badly worded on my part, I really mean that it works on all the podcasts that have images, which is "most" of my feeds. It always finds images for podcasts for which I know do (or should) have associated images. (I haven't checked all my podcasts exhaustively, however.)
Committed to svn with the small change required to update feed info. Thanks.