From 8ebf3645dfb2bb9be46dfadd74df61da6a225411 Mon Sep 17 00:00:00 2001 From: Laurent Monin Date: Wed, 10 Jul 2019 07:43:21 +0200 Subject: [PATCH] PICARD-1552: simplify code and drop a part that makes no sense --- picard/metadata.py | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/picard/metadata.py b/picard/metadata.py index 76b8465a4..351eeda42 100644 --- a/picard/metadata.py +++ b/picard/metadata.py @@ -150,18 +150,11 @@ class Metadata(MutableMapping): try: a = int(self["totaltracks"]) - except (ValueError, KeyError): - pass - else: - try: - if "title" in weights: - b = release['media'][0]['track-count'] - else: - b = release['track-count'] - except KeyError: - b = 0 + b = release['track-count'] score = 0.0 if a > b else 0.3 if a < b else 1.0 parts.append((score, weights["totaltracks"])) + except (ValueError, KeyError): + pass preferred_countries = config.setting["preferred_release_countries"] preferred_formats = config.setting["preferred_release_formats"]