Album art from images in the same folder not exposed
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Canonical System Image |
High
|
Alejandro J. Cura | ||
| | Thumbnailer |
Invalid
|
Undecided
|
Unassigned | |
| | mediascanner2 (Ubuntu) |
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
- PS Jenkins bot: Approve (continuous-integration) on 2014-09-24
- Unity Team: Pending requested 2014-09-23
-
Diff: 158 lines (+73/-11)3 files modifiedsrc/thumbnailer.cpp (+47/-10)
src/videoscreenshotter.cpp (+1/-1)
tests/basic.cpp (+25/-0)
- Marcus Tomlinson (community): Disapprove on 2015-11-25
- PS Jenkins bot (community): Approve (continuous-integration) on 2015-01-20
-
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)
- Michi Henning (community): Approve on 2015-11-24
- PS Jenkins bot (community): Approve (continuous-integration) on 2015-11-23
-
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)
| Andrew Hayzen (ahayzen) wrote : | #1 |
| Jussi Pakkanen (jpakkane) wrote : | #2 |
Is there a specification or list of common filenames for this use case?
| Andrew Hayzen (ahayzen) wrote : | #3 |
These are the filenames i've seen:
folder.jpg
.folder.jpg
cover.jpg
album.jpg
albumart.jpg
front.jpg
$artist - $album.jpg
I managed to find this question [0] on askubuntu, which was actually has an answer from James Henstridge :)
Note that it also states what Rhythmbox does:
'From where is artwork fetched?
Rhythmbox uses last.fm to look up artwork. If you have local image files, saved in the same directory as the audio file(s) with the filename "cover", "album", "albumart", ".folder", "folder" or "$artist - $album" Rhythmbox will use them instead of looking up a cover on last.fm. It will also use images extracted from file tags.'
0 - http://
| Changed in mediascanner2: | |
| status: | New → Invalid |
| Changed in thumbnailer: | |
| status: | New → In Progress |
| Michi Henning (michihenning) wrote : | #4 |
Should we still integrate support for this into thumbnailer?
| Andrew Hayzen (ahayzen) wrote : | #5 |
We just had another duplicate report for this against music.
If still possible I would love to see this supported with the thumbnailer, I feel it is important in the move to convergence as we meet challenges with larger and more varied music library structures. And it is very useful for providing an image when the online source doesn't have one or overriding the online's suggestion.
| Ruben Maes (ruben-maes96) wrote : | #6 |
Also, I'd appreciate an option to entirely turn off internet lookup of albums. I don't like it that the names of all music in my collection are being sent to... where are they actually sent?
| Michi Henning (michihenning) wrote : | #7 |
Requests are sent to https:/
The requests made by the thumbnailer contain the artist, album, and an API key. Even if dash.ubuntu.com would want to spy on your album collection (it doesn't), what it could learn would be limited to the information it gets from the incoming HTTP headers and the incoming connection.
Turning online art lookup off is possible, but only on a system-wide basis because the thumbnailer is shared among a number of applications and scopes. Making this option available would require design input though.
| Cédric Bellegarde (gnumdk) wrote : | #8 |
Wanted feature... Google music handles this since lollipop.
| Changed in mediascanner2: | |
| status: | Invalid → In Progress |
| Changed in thumbnailer: | |
| status: | In Progress → Invalid |
| Ruben Maes (ruben-maes96) wrote : | #9 |
A fix has been written but has been sitting unreviewed for more than 9 months ...
https:/
| Pat McGowan (pat-mcgowan) wrote : | #10 |
@james Is this something you can pick up?
| Changed in mediascanner2: | |
| assignee: | nobody → James Henstridge (jamesh) |
| importance: | Undecided → High |
| Changed in canonical-devices-system-image: | |
| assignee: | nobody → Alejandro J. Cura (alecu) |
| importance: | Undecided → High |
| milestone: | none → ww46-2015 |
| status: | New → Confirmed |
| 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) |
| James Henstridge (jamesh) wrote : | #11 |
There are test packages for this in silo 40 if anyone wants to give them a try:
https:/
Things seem to work okay in my own testing (checking My Music scope and music-app). This is based on Jussi's earlier branch, with support extended to album art rather than just song art, and a small lookup cache added (i.e. so if you ask for art for every song in an album, it won't do the full directory scan each time).
| Launchpad Janitor (janitor) wrote : | #12 |
This bug was fixed in the package mediascanner2 - 0.109+16.
---------------
mediascanner2 (0.109+
[ James Henstridge ]
* If a folder contains an image file named
{cover,
art for songs in preference to online art if the songs do not have
embedded art. (LP: #1372000)
* Disable MountWatcher's callback during destruction so we don't call
back into a partially destroyed class. (LP: #1492393)
[ CI Train Bot ]
* debian/
* New rebuild forced.
-- James Henstridge <email address hidden> Tue, 24 Nov 2015 08:56:52 +0000
| Changed in mediascanner2 (Ubuntu): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | In Progress → Fix Committed |
| Changed in canonical-devices-system-image: | |
| status: | Fix Committed → Fix Released |

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.