From 738876ef011c9917d753daf954b00c183e5956d9 Mon Sep 17 00:00:00 2001 From: Laurent Monin Date: Wed, 20 Sep 2023 18:41:23 +0200 Subject: [PATCH] CoverArt.retrieve(): invert the test and simplify code a bit --- picard/coverart/__init__.py | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/picard/coverart/__init__.py b/picard/coverart/__init__.py index a64500d05f..e1e672f1b5 100644 --- a/picard/coverart/__init__.py +++ b/picard/coverart/__init__.py @@ -58,13 +58,11 @@ def __repr__(self): def retrieve(self): """Retrieve available cover art images for the release""" config = get_config() - if (not config.setting["save_images_to_tags"] and not - config.setting["save_images_to_files"]): + if config.setting['save_images_to_tags'] or config.setting['save_images_to_files']: + self.providers = cover_art_providers() + self.next_in_queue() + else: log.debug("Cover art disabled by user options.") - return - - self.providers = cover_art_providers() - self.next_in_queue() def _set_metadata(self, coverartimage, data): try: