From 52afea16d8413b29970fd0e032f2b911f0a148ec Mon Sep 17 00:00:00 2001 From: Laurent Monin Date: Sat, 10 May 2014 21:21:13 +0200 Subject: [PATCH] Differentiate case of no CAA info vs no cover art --- picard/coverart.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/picard/coverart.py b/picard/coverart.py index 5aab95636..a3e81b439 100644 --- a/picard/coverart.py +++ b/picard/coverart.py @@ -188,11 +188,13 @@ class CoverArt: # MB web service indicates if CAA has artwork # http://tickets.musicbrainz.org/browse/MBS-4536 - caa_has_suitable_artwork = False + if 'cover_art_archive' not in self.release.children: + log.debug("No Cover Art Archive information for %s" + % self.release.id) + return False - if 'cover_art_archive' in self.release.children: - caa_node = self.release.children['cover_art_archive'][0] - caa_has_suitable_artwork = (caa_node.artwork[0].text == 'true') + caa_node = self.release.children['cover_art_archive'][0] + caa_has_suitable_artwork = caa_node.artwork[0].text == 'true' if not caa_has_suitable_artwork: log.debug("There are no images in the Cover Art Archive for %s"