Backup_Repos/beets
1
0
Fork 0
mirror of https://github.com/beetbox/beets.git synced 2026-02-02 05:21:45 +01:00
Code Issues Projects Releases Wiki Activity

Don't uselessly split/join early returned genres

Browse source
This commit is contained in:
J0J0 Todos 2025-01-21 12:38:20 +01:00
parent 169ec20a2f
commit c9187b40bd
1 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
beetsplug/lastgenre/__init__.py
View file

@ -334,15 +334,16 @@ class LastGenrePlugin(plugins.BeetsPlugin):
indicates that existing genres were combined with new last.fm genres,
while "artist" means only new last.fm genres are included.
"""
keep_genres = []
separator = self.config["separator"].as_str()
genres = self._get_existing_genres(obj)
if genres and not self.config["force"]:
if not self.config["force"]:
# Without force pre-populated tags are returned as-is.
return separator.join(genres), "keep"
if isinstance(obj, library.Item):
return obj.get("genre", with_album=False), "keep, no-force"
return obj.get("genre"), "keep, no-force"
if self.config["force"]:
genres = self._get_existing_genres(obj)
keep_genres = []
# Force doesn't keep any unless keep_existing is set.
# Whitelist validation is handled in _resolve_genres.
if self.config["keep_existing"]: