From 34e992366952ee103c99e2276c6285dc8ef4167f Mon Sep 17 00:00:00 2001 From: Adrian Sampson Date: Sun, 11 Jul 2010 19:22:22 -0700 Subject: [PATCH] fixed situation where the MusicBrainz match duplicated a metadata match --- beets/autotag/__init__.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/beets/autotag/__init__.py b/beets/autotag/__init__.py index a2b5cdbc8..ef236ea6f 100644 --- a/beets/autotag/__init__.py +++ b/beets/autotag/__init__.py @@ -361,7 +361,7 @@ def tag_album(items, search_artist=None, search_album=None): # The output list of result tuples: dist_ordered_cands = [] - # Try to find album indicated by MusicBrainz iDs. + # Try to find album indicated by MusicBrainz IDs. id_info = match_by_id(items) if id_info: ordered = order_items(items, id_info['tracks']) @@ -388,6 +388,10 @@ def tag_album(items, search_artist=None, search_album=None): # Get the distance to each candidate. for info in _first_n(candidates, MAX_CANDIDATES): + # Don't duplicate the MusicBrainz match. + if id_info and id_info['album_id'] == info['album_id']: + continue + # Make sure the album has the correct number of tracks. if len(items) != len(info['tracks']): continue