From 8bbb67c62c3f04728b761286cb8610d6c58a50d3 Mon Sep 17 00:00:00 2001 From: brainz34 Date: Mon, 15 Apr 2013 11:48:09 +0200 Subject: [PATCH] Fix: `load_cluster()` does not find existing cluster. `load_cluster()` uses "artist" for looking up existing clusters, but Cluster does not have a meta-data item "artist". This bug does not show up when clustering all files at once (which will be the case most times), as in this case a single cluster will be calculated and only looked up once. But when clustering files in two steps, a second cluster will be generated as the first one will not be found. Signed-off-by: brainz34 --- picard/tagger.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/picard/tagger.py b/picard/tagger.py index 759e45307..c3f5d4d7b 100644 --- a/picard/tagger.py +++ b/picard/tagger.py @@ -587,7 +587,7 @@ class Tagger(QtGui.QApplication): def load_cluster(self, name, artist): for cluster in self.clusters: cm = cluster.metadata - if name == cm["album"] and artist == cm["artist"]: + if name == cm["album"] and artist == cm["albumartist"]: return cluster cluster = Cluster(name, artist) self.clusters.append(cluster)