Continued:

- simplified code, no need for this extra indenting

diff --git a/fba/models/blocks.py b/fba/models/blocks.py
index 904286d19b78bfd997104eb07bce061037b638ef..601db5db08e328ca2e91b1e9ac7f0f4e098fe729 100644 (file)
--- a/fba/models/blocks.py
+++ b/fba/models/blocks.py
@@ -146,11 +146,9 @@ def add(blocker: str, blocked: str, reason: str, block_level: str):
     elif reason is not None and reason == "":
         raise Exception(f"blocker='{blocker}',blocked='{blocked}',block_level='{block_level}' has empty (not 'None') block reason set")
 
-    if reason is not None:
-        # Maybe needs cleaning
-        logger.debug("reason='%s' - BEFORE!")
-        reason = tidyup.reason(reason)
-        logger.debug("reason='%s' - AFTER!")
+    logger.debug("reason='%s' - BEFORE!")
+    reason = tidyup.reason(reason) if reason is not None and reason != "" else None
+    logger.debug("reason='%s' - AFTER!")
 
     logger.info("New block: blocker='%s',blocked='%s',reason='%s',block_level='%s'", blocker, blocked, reason, block_level)