[SOLVED] Cover images not loaded for all albums

[haeckse]

@haeckse
@Nutul 
@Tim Curtis 

What Tim said.

Sorry, I just noticed one last issue with the Show Me Something Good track you shared. Even after I converted to ID3 V2.3, removed the comments, and retyped the APIC frame as type 03 Cover (front) I was getting a thumbnail but not a full cover displayed during playback.

The track metadata is declaring the APIC format to be MIME type "image/PNG". I haven't had my coffee this morning so haven't browsed the code, but might we not be flattening upper- to lowercase when processing embedded images? Anyway, as a quick-n-dirty fix I edited the MIME type in the metadata to "image/png", and the cover now displays.

Regards,
Kent

Kent,

I can confirm your findings.

Those APIC type/MIME type settings displayed thumbnails as well as the full scale cover at playback:

FRONT_COVER Image: [Size: 58684 bytes] [Type: image/png]
FRONT_COVER Image: [Size: 260249 bytes] [Type: image/jpeg]

while those didn't:

OTHER Image: [Size: 50435 bytes] [Type: image/jpeg]
FRONT_COVER Image: [Size: 260249 bytes] [Type: image/JPEG]
FRONT_COVER Image: [Size: 58684 bytes] [Type: image/PNG]

So setting the image as type "front cover" plus a lowercase MIME type does the trick. Though it is a mystery to me why when reading my entire song library, the covers of a large number of files are displayed even though the APIC type is set to "other".

I can't think of an easy method to fix my entire library in batch mode yet. A quick look into the eyed3 params didn't look promising. But there is pretty verbose documentation for the tool that I will look into.

I have no idea about the complexity yet, but maybe it's worth investing some time diving into the Moode code to understand how and where the ID3 data is processed. Maybe I can manage to make the parser more fault-tolerant in dealing with messy metadata, which will probably be more fun than plodding through fixing the bad tags : )

Regards,
Andrea