From 535d86148e8d80837fda1e8aa0f551f8b67ab396 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Roland=20H=C3=A4der?= <roland@mxchange.org>
Date: Mon, 21 Apr 2025 04:21:03 +0200
Subject: [PATCH] Continued: - fixed some tpzos (TypError -> TypeError,
 row[block] -> block[reason], ...) - RuntimeException is RuntimeError
 (confusing as errors are hard for my understanding) - used f-string instead
 (pylint is now a bit happier) - still it is confusing with import errors?!

---
 fba/commands.py          | 7 ++++---
 fba/helpers/software.py  | 2 +-
 fba/http/network.py      | 4 ++--
 fba/models/blocks.py     | 2 +-
 fba/networks/mastodon.py | 2 +-
 5 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/fba/commands.py b/fba/commands.py
index bdfb163..af28be6 100644
--- a/fba/commands.py
+++ b/fba/commands.py
@@ -17,11 +17,12 @@
 import inspect
 import json
 import logging
-import numpy
+
+import argparse
 import time
 import urllib
+import numpy
 
-import argparse
 import atoma
 import bs4
 import markdown
@@ -2090,7 +2091,7 @@ def check_obfuscations(args: argparse.Namespace) -> int:
         delete_sql = ", ".join([f"\'{pattern}\'" for pattern in deleted])
 
         logger.debug("delete_sql='%s'", delete_sql)
-        database.cursor.execute("DELETE FROM obfuscation WHERE pattern IN (%s)" % delete_sql)
+        database.cursor.execute(f"DELETE FROM obfuscation WHERE pattern IN ({delete_sql})")
 
         logger.debug("Invoking commit() ...")
         database.connection.commit()
diff --git a/fba/helpers/software.py b/fba/helpers/software.py
index 33c0305..2aadba9 100644
--- a/fba/helpers/software.py
+++ b/fba/helpers/software.py
@@ -298,7 +298,7 @@ def is_relay(software: str) -> bool:
     logger.debug("software='%s'- CALLED!", software)
 
     if not isinstance(software, str):
-        raise TypError(f"software[]='{type(software)}' is not type 'str'")
+        raise TypeError(f"software[]='{type(software)}' is not type 'str'")
     elif software == "":
         raise ValueError("Parameter 'software' is empty")
 
diff --git a/fba/http/network.py b/fba/http/network.py
index 006234f..fb7a7a9 100644
--- a/fba/http/network.py
+++ b/fba/http/network.py
@@ -445,7 +445,7 @@ def fetch_csv_rows (url: str) -> list:
     logger.debug("response.ok='%s',response.status_code=%d,response.text()=%d", response.ok, response.status_code, len(response.text))
     if not response.ok or response.status_code > 200 or len(response.content) == 0:
         logger.warning("Failed fetching url='%s': response.ok='%s',response.status_code=%d,response.content()=%d - EXIT!", url, response.ok, response.status_code, len(response.text))
-        raise RuntimeException(f"response.ok='{response.ok}',response.status_code={response.status_code},response.content()={len(response.content)} is unexpected")
+        raise RuntimeError(f"response.ok='{response.ok}',response.status_code={response.status_code},response.content()={len(response.content)} is unexpected")
 
     lines = response.content.decode("utf-8").splitlines()
     logger.debug("Reading %d lines, dialect=unix ...", len(lines))
@@ -455,7 +455,7 @@ def fetch_csv_rows (url: str) -> list:
 
     if reader is None:
         logger.warning("Failed parsing response.content()=%d as CSV content", len(response.content))
-        raise RuntimeException(f"reader is None after parsing {len(lines)} CSV lines")
+        raise RuntimeError(f"reader is None after parsing {len(lines)} CSV lines")
 
     # Init rows from CSV reader
     rows = list(reader)
diff --git a/fba/models/blocks.py b/fba/models/blocks.py
index 38f4950..b77c668 100644
--- a/fba/models/blocks.py
+++ b/fba/models/blocks.py
@@ -67,7 +67,7 @@ def update_reason(reason: str, blocker: str, blocked: str, block_level: str) ->
     domain_helper.raise_on(blocked)
 
     if not isinstance(reason, str) and reason is not None:
-        raise TypError(f"Parameter reason[]='{type(reason)}' has not expected type 'str'")
+        raise TypeError(f"Parameter reason[]='{type(reason)}' has not expected type 'str'")
     elif not isinstance(block_level, str):
         raise TypeError(f"Parameter block_level[]='{type(block_level)}' has not expected type 'str'")
     elif block_level == "":
diff --git a/fba/networks/mastodon.py b/fba/networks/mastodon.py
index 0e132fb..c4654c8 100644
--- a/fba/networks/mastodon.py
+++ b/fba/networks/mastodon.py
@@ -202,7 +202,7 @@ def fetch_blocks(domain: str) -> list:
                 logger.warning("block[domain]='%s' has invalid block[digest]='%s' - SKIPPED!", block["domain"], block["digest"])
                 continue
 
-            logger.debug("block[reason]='%s' - BEFORE!", row["block"])
+            logger.debug("block[reason]='%s' - BEFORE!", block["reason"])
             reason = tidyup.reason(block["reason"]) if "reason" in block and block["reason"] not in [None, ""] else None
 
             logger.debug("Appending domain='%s',block[domain]='%s',reason='%s',block_level='%s' ...", domain, block["domain"], reason, block_level)
-- 
2.39.5