From 95970757511be8ac71f158f0c0f31b4fbc54705f Mon Sep 17 00:00:00 2001 From: dunkla Date: Sun, 28 Dec 2025 20:48:15 +0100 Subject: [PATCH] Simplify MusicBrainz genres assignment Remove intermediate variable and assign directly to info.genres. Addresses PR review comment. --- beetsplug/musicbrainz.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/beetsplug/musicbrainz.py b/beetsplug/musicbrainz.py index 695ce8790..75933e6f9 100644 --- a/beetsplug/musicbrainz.py +++ b/beetsplug/musicbrainz.py @@ -644,11 +644,10 @@ class MusicBrainzPlugin(MusicBrainzAPIMixin, MetadataSourcePlugin): for source in sources: for genreitem in source: genres[genreitem["name"]] += int(genreitem["count"]) - genre_list = [ + info.genres = [ genre for genre, _count in sorted(genres.items(), key=lambda g: -g[1]) ] - info.genres = genre_list # We might find links to external sources (Discogs, Bandcamp, ...) external_ids = self.config["external_ids"].get()