Album art from images in the same folder not exposed
Bug #1372000 reported by
Andrew Hayzen
This bug affects 12 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical System Image |
Fix Released
|
High
|
Alejandro J. Cura | ||
Thumbnailer |
Invalid
|
Undecided
|
Unassigned | ||
mediascanner2 (Ubuntu) |
Fix Released
|
High
|
James Henstridge |
Bug Description
The music app reads the art property from mediascanner2 which, from my understanding, uses embedded art if there is any but falls back to using the online image provider by sending the album and artist.
However I, and probably others, have a lot of albums which have the cover art in a file within the same directory such as Folder.jpg or AlbumArt.jpg etc.
I propose for this to become the second option, with the online image provider as the last fallback.
In order of preference:
1) Embedded art
2) Art within the same folder
3) Online provided art
Related branches
lp://staging/~jpakkane/thumbnailer/standalone-albumart
Rejected
for merging
into
lp://staging/thumbnailer
- PS Jenkins bot (community): Approve (continuous-integration)
- Unity Team: Pending requested
-
Diff: 158 lines (+73/-11)3 files modifiedsrc/thumbnailer.cpp (+47/-10)
src/videoscreenshotter.cpp (+1/-1)
tests/basic.cpp (+25/-0)
lp://staging/~jpakkane/mediascanner2/standalone-art-uri
Rejected
for merging
into
lp://staging/mediascanner2
- Marcus Tomlinson (community): Disapprove
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 129 lines (+69/-4)3 files modifiedsrc/mediascanner/MediaFile.cc (+52/-4)
test/test_mfbuilder.cc (+14/-0)
test/test_util.cc (+3/-0)
lp://staging/~jamesh/mediascanner2/folder-coverart
- Michi Henning (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 730 lines (+453/-36)16 files modifiedCMakeLists.txt (+1/-1)
debian/changelog (+9/-0)
debian/libmediascanner-2.0-3.symbols (+2/-0)
debian/libmediascanner-2.0-4.shlibs (+1/-1)
src/mediascanner/Album.cc (+25/-8)
src/mediascanner/Album.hh (+4/-0)
src/mediascanner/CMakeLists.txt (+1/-0)
src/mediascanner/FolderArtCache.cc (+169/-0)
src/mediascanner/MediaFile.cc (+10/-4)
src/mediascanner/MediaStore.cc (+1/-1)
src/mediascanner/internal/FolderArtCache.hh (+55/-0)
src/ms-dbus/dbus-codec.cc (+4/-2)
src/ms-dbus/dbus-codec.hh (+1/-1)
test/test_dbus.cc (+3/-2)
test/test_mfbuilder.cc (+164/-16)
test/test_util.cc (+3/-0)
Changed in mediascanner2: | |
status: | New → Invalid |
Changed in thumbnailer: | |
status: | New → In Progress |
Changed in mediascanner2: | |
status: | Invalid → In Progress |
Changed in thumbnailer: | |
status: | In Progress → Invalid |
Changed in canonical-devices-system-image: | |
status: | Confirmed → In Progress |
Changed in canonical-devices-system-image: | |
milestone: | ww46-2015 → ww02-2016 |
affects: | mediascanner2 → mediascanner2 (Ubuntu) |
Changed in canonical-devices-system-image: | |
status: | In Progress → Fix Committed |
Changed in canonical-devices-system-image: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
I wasn't sure if this should be reported against mediascanner or thumbnailer? so I have reported against both please mark as invalid where appropriate.