sane-tag-music: refactor: remove extraneous "Various Artists" check

This commit is contained in:
2024-07-13 01:53:05 +00:00
parent e04dd6cb7d
commit 0e5f01f240

View File

@@ -375,21 +375,13 @@ class Tags:
self.artist = self.albumartist
def demote_producer(self) -> None:
"""
"Various Artists" is used for an album when i don't have a better grouping for it.
i should avoid setting it in the producer field when possible, so that if i locate the album artist
and fix the album artist that doesn't compete with an outdated "producer"
"""
if self.albumartist == [ "Various Artists" ] and self.producer == [ "Various Artists" ]:
self.producer = []
if self.producer == self.albumartist:
# it's mostly just noise to have producer when not actually needed
self.producer = []
producers = []
for p in self.producer:
if self.albumartist == [ p ]:
# it's mostly just noise to have producer when not actually needed
continue
if any(soft.lower() in p.lower() for soft in KNOWN_SOFTWARE_PRODUCERS):
# software like Adobe usee the "producer" field for itself
continue
producers.append(p)
self.producer = producers