lastfmcovers plugin should not fetch "generic" cover

Bug #417346 reported by Matteo Beniamino
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Exaile
Fix Released
Undecided
Unassigned

Bug Description

Exaile from http://bazaar.launchpad.net/%7Eexaile-devel/exaile/exaile-0.3.0/ at revision r2391 .

If an album exists in the lastfm DB but the actual cover is missing, lastfm shows a generic picture. The plugin imports this generic picture as the album cover in Exaile. I think that the expected behaviour should be to ignore this generic cover.
To check the current behavior assign "Wolfgang Amadeus Mozart" as the author and "Mozart Symphonies 10" as the album to a custom track and fetch the cover with lastfmcovers as the default plugin.

The proposed attached patch introduces a (black)list of sha1 signatures of pictures to be ignored.

Tags: lastfmcovers
Revision history for this message
Matteo Beniamino (m-beniamino) wrote :
Revision history for this message
reacocard (reacocard) wrote :

patch committed in r2396, thanks!

Changed in exaile:
milestone: none → 0.3.0
status: New → Fix Committed
reacocard (reacocard)
Changed in exaile:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.