From e7f3987b424907b72a00a84e814b00d09212796c Mon Sep 17 00:00:00 2001 From: Tom Jaspers Date: Mon, 26 Jan 2015 22:19:02 +0100 Subject: [PATCH] Centralize check and string literal into MusicBrainzAPIError.__init__ for #1190 --- beets/autotag/mb.py | 22 ++-------------------- 1 file changed, 2 insertions(+), 20 deletions(-) diff --git a/beets/autotag/mb.py b/beets/autotag/mb.py index 50d146dbb..78baac054 100644 --- a/beets/autotag/mb.py +++ b/beets/autotag/mb.py @@ -38,6 +38,8 @@ class MusicBrainzAPIError(util.HumanReadableException): """ def __init__(self, reason, verb, query, tb=None): self.query = query + if isinstance(reason, musicbrainzngs.WebServiceError): + reason = 'MusicBrainz not reachable' super(MusicBrainzAPIError, self).__init__(reason, verb, tb) def get_message(self): @@ -325,11 +327,6 @@ def match_album(artist, album, tracks=None): try: res = musicbrainzngs.search_releases( limit=config['musicbrainz']['searchlimit'].get(int), **criteria) - except musicbrainzngs.WebServiceError: - log.debug(u'MusicBrainz is unreachable.') - raise MusicBrainzAPIError('MusicBrainz not reachable', - 'release search', criteria, - traceback.format_exc()) except musicbrainzngs.MusicBrainzError as exc: raise MusicBrainzAPIError(exc, 'release search', criteria, traceback.format_exc()) @@ -356,11 +353,6 @@ def match_track(artist, title): try: res = musicbrainzngs.search_recordings( limit=config['musicbrainz']['searchlimit'].get(int), **criteria) - except musicbrainzngs.WebServiceError: - log.debug(u'MusicBrainz is unreachable.') - raise MusicBrainzAPIError('MusicBrainz not reachable', - 'recording search', criteria, - traceback.format_exc()) except musicbrainzngs.MusicBrainzError as exc: raise MusicBrainzAPIError(exc, 'recording search', criteria, traceback.format_exc()) @@ -393,11 +385,6 @@ def album_for_id(releaseid): except musicbrainzngs.ResponseError: log.debug(u'Album ID match failed.') return None - except musicbrainzngs.WebServiceError: - log.debug(u'MusicBrainz is unreachable.') - raise MusicBrainzAPIError('MusicBrainz not reachable', - 'get release by ID', albumid, - traceback.format_exc()) except musicbrainzngs.MusicBrainzError as exc: raise MusicBrainzAPIError(exc, 'get release by ID', albumid, traceback.format_exc()) @@ -417,11 +404,6 @@ def track_for_id(releaseid): except musicbrainzngs.ResponseError: log.debug(u'Track ID match failed.') return None - except musicbrainzngs.WebServiceError: - log.debug(u'MusicBrainz is unreachable.') - raise MusicBrainzAPIError('MusicBrainz not reachable', - 'get recording by ID', trackid, - traceback.format_exc()) except musicbrainzngs.MusicBrainzError as exc: raise MusicBrainzAPIError(exc, 'get recording by ID', trackid, traceback.format_exc())