Display embedded cover images
ClosedPublic
Actions

Authored by mgallien on Nov 25 2018, 9:41 PM.

Details

Summary

introduce EmbeddedCoverageImageProvider to get access to embedded covers

store the fact that tracks have an embedded cover image in database

when importing files, check and store the presence of embedded cover

makes MusicAlbum and MusicAudioTrack use the embedded cover images

BUG: 380442

Test Plan

embedded covers are shown. albums without a cover but with tracks havong embedded cover are now showing a nice cover

Repository

R255 Elisa

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

mgallien requested review of this revision.Nov 25 2018, 9:41 PM

mgallien created this revision.

This revision was not accepted when it landed; it landed in state Needs Review.Dec 9 2018, 7:49 PM

This revision was automatically updated to reflect the committed changes.

		Path
M		autotests/albummodeltest.cpp (6 lines)
M		autotests/allalbumsmodeltest.cpp (4 lines)
M		autotests/allartistsmodeltest.cpp (2 lines)
M		autotests/alltracksmodeltest.cpp (8 lines)
M		autotests/alltracksproxymodeltest.cpp (8 lines)
M		autotests/databaseinterfacetest.cpp (130 lines)
M		autotests/databasetestdata.h (46 lines)
M		autotests/trackslistenertest.cpp (2 lines)
M		src/CMakeLists.txt (1 line)
M		src/abstractfile/abstractfilelisting.h (2 lines)
M		src/abstractfile/abstractfilelisting.cpp (18 lines)
M		src/baloo/localbaloofilelisting.cpp (3 lines)
M		src/databaseinterface.cpp (37 lines)
M		src/elisaqmlplugin.cpp (2 lines)
A	M	src/embeddedcoverageimageprovider.h (38 lines)
A	M	src/embeddedcoverageimageprovider.cpp (66 lines)
M		src/musicalbum.cpp (12 lines)
M		src/musicaudiotrack.h (6 lines)
M		src/musicaudiotrack.cpp (26 lines)