Bug 417409 - Albums that contain symbols, no album art is generated
Summary: Albums that contain symbols, no album art is generated
Status: CONFIRMED
Alias: None
Product: Elisa
Classification: Applications
Component: general (show other bugs)
Version: 19.12.1
Platform: Microsoft Windows Microsoft Windows
: NOR minor
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-02-11 00:31 UTC by Andrew
Modified: 2020-02-17 20:48 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
3x screenshots of the problem. (1.55 MB, application/x-zip-compressed)
2020-02-11 00:31 UTC, Andrew
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew 2020-02-11 00:31:31 UTC
Created attachment 125836 [details]
3x screenshots of the problem.

SUMMARY
The Elisa App from the Windows store is fantastic, but noticed that any albums (in the albums view) that have symbols in either the band name or album name don't produce the build-in tag image.

STEPS TO REPRODUCE
1. Added albums to library
2. Browse or search library for albums
3. Not all images are loaded for the albums

OBSERVED RESULT
Images that are part of the file tags aren't loaded

EXPECTED RESULT
Images that are part of the file tags should be loaded

SOFTWARE/OS VERSIONS
Windows: 10

ADDITIONAL INFORMATION
Looks to be symbol related ,that's the only commonality. Doesn't appear to be an issue if the song has a symbol in it. I have attached screenshots with examples.
Comment 1 Matthieu Gallien 2020-02-11 07:56:45 UTC
Thanks for your report.
I suspect this is about directory names with symbols in their names.
We have a known problem with them. Can you confirm this is your case?
Comment 2 Andrew 2020-02-11 19:26:59 UTC
I can confirm this is the case (I didn't think to look for that bug!).

Drive:\FLAC\Artist\Album is the directory structure in Windows, so yeah, lines up with your suspicions.

Cheers,

Andrew
Comment 3 Matthieu Gallien 2020-02-12 08:52:52 UTC
Thanks for the feedback. I knew that the fix in this version is only partial.
I will post update here when a fix is available.
Comment 4 Alexander Stippich 2020-02-17 20:48:30 UTC
@mgallien
This is probably due to KFileMetadata's embeddedimagedata using a different code path, which is a conceptual mistake I made when implementing it. This should be fixed when https://phabricator.kde.org/D25517 and the series lands and Elisa is ported. Your support would be appreciated for that since it is currently stuck somewhat