Bug 328451

Summary: Support for VorbisComment METADATA_BLOCK_PICTURE tag broken?
Product: amarok Reporter: Martin Brodbeck <martin>
Component: Metadata Editing and ReadingAssignee: Amarok Developers <amarok-bugs-dist>
Status: RESOLVED FIXED    
Severity: normal CC: 123kash
Priority: NOR    
Version: 2.8.0   
Target Milestone: 2.9   
Platform: unspecified   
OS: Linux   
URL: http://git.reviewboard.kde.org/r/114406/
Latest Commit: Version Fixed In: 2.9

Description Martin Brodbeck 2013-12-05 13:14:41 UTC
I just realized that amarok doesn't show cover art if it is provided the recommended way, that is, using the VorbisComment tag METADATA_BLOCK_PICTURE instead of the (unofficial and deprecated) ALBUMART.

It seems that, after a first look at the source code, this METADATA_BLOCK_PICTURE tag *should* be supported. 

Reproducible: Always

Steps to Reproduce:
1. Open a Ogg Vorbis or Opus file which provides a Cover front Picture with METADATA_BLOCK_PICTURE
Actual Results:  
There's no cover picture shown.

Expected Results:  
There should be a cover picture shown. With FLAC files, which also use this METADATA_BLOCK_PICTURE tag, it works (though they are no ogg files).
Comment 1 Martin Brodbeck 2013-12-11 14:40:43 UTC
I've added review request #114406 which fixes this.
Comment 2 Mark Kretschmann 2013-12-19 05:58:35 UTC
Git commit 19fc23445bdb2da894ad6ca4096236de2e9fcb1f by Mark Kretschmann, on behalf of Martin Brodbeck.
Committed on 19/12/2013 at 05:48.
Pushed by markey into branch 'master'.

Fetch cover picture from METADATA_BLOCK_PICTURE tag.

With this patch, covers from Ogg files (for example Vorbis and Opus)
using the recommended METADATA_BLOCK_PICTURE tag will be displayed.
FIXED-IN: 2.9
REVIEW: 114406

M  +2    -0    ChangeLog
M  +4    -1    shared/tag_helpers/VorbisCommentTagHelper.cpp

http://commits.kde.org/amarok/19fc23445bdb2da894ad6ca4096236de2e9fcb1f