From b8518d745f0dcab2b7b9e27c217e04c5d08066b8 Mon Sep 17 00:00:00 2001
From: jesopo
Date: Tue, 20 Nov 2018 20:43:26 +0000
Subject: Move all logic related to score reasons until after we've checked if
 we have reasons (mixed_unicode.py)

---
 modules/mixed_unicode.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'modules')

diff --git a/modules/mixed_unicode.py b/modules/mixed_unicode.py
index 2d06fa99..057b971a 100644
--- a/modules/mixed_unicode.py
+++ b/modules/mixed_unicode.py
@@ -77,11 +77,11 @@ class Module(ModuleManager.BaseModule):
             reasons.extend([ScoreReason.AdditionalScript]*(len(scripts)-1))
 
         score = len(reasons)
-        reasons_s = []
-        for reason, count in collections.Counter(reasons).items():
-            reasons_s.append("%s: %s" % (reason, count))
-
         if score > 0:
+            reasons_s = []
+            for reason, count in collections.Counter(reasons).items():
+                reasons_s.append("%s: %s" % (reason, count))
+
             self.log.trace(
                 "Message given a mixed-unicode score of %s (reasons: %s)",
                 [score, ", ".join(reasons_s)])
-- 
cgit v1.3.1-10-gc9f91