style: fix W0102, W1201, W1309, W0404, C0209

Backupper.py

@@ -471,7 +471,7 @@ def __queue_cleaner(process_output_queue__: multiprocessing.Queue or None) -> No
         return -1
 
     def _generate_tree(
-        self, entries: Dict, root_relative_to_dirname: bool = False, ignore_filepaths_abs: List = []
+        self, entries: Dict, root_relative_to_dirname: bool = False, ignore_filepaths_abs: List or None = None
     ) -> Dict or int:
         """Parses entries and generates dict of files and dirs with the following structure using multiprocessing:
         {
@@ -508,7 +508,7 @@ def _generate_tree(
         Args:
             entries (Dict): {"path": skip} ex. {"path_1": True, "path_2": False, ...}
             root_relative_to_dirname (bool): True to calculate path relative to dirname(root_dir) instead of root_dir
-            ignore_filepaths_abs (List, optional): list of absolute filepaths to exclude from tree. Defaults to [].
+            ignore_filepaths_abs (List, optional): list of absolute filepaths to exclude from tree. Defaults to None.
 
         Returns:
             Dict or int: parsed tree or exit status in case of cancel
@@ -532,7 +532,7 @@ def _generate_tree(
         parsed_queue = multiprocessing.Queue(-1)
         for path_abs, skip in entries.items():
             # Check if we need to exclude it
-            if path_abs in ignore_filepaths_abs:
+            if ignore_filepaths_abs and path_abs in ignore_filepaths_abs:
                 continue
 
             # Extract root directory
@@ -592,7 +592,7 @@ def _generate_tree(
                     path_abs = os.path.join(root_dir, path_rel)
 
                     # Check if we need to exclude it
-                    if path_abs in ignore_filepaths_abs:
+                    if ignore_filepaths_abs and path_abs in ignore_filepaths_abs:
                         continue
 
                     # Put into tree
@@ -1344,7 +1344,7 @@ def start_backup(
             del output_tree
 
             # Extract all existing files / directories inside backup (output_directory) again
-            logging.info(f"Generating output (existing files) tree again")
+            logging.info("Generating output (existing files) tree again")
             output_tree = self._generate_tree({output_dir: False}, ignore_filepaths_abs=[output_dir])
 
             # Exit ?

GUIHandler.py

@@ -30,7 +30,6 @@
 from PyQt5.QtGui import QIcon
 from PyQt5.QtWidgets import (
     QApplication,
-    QMainWindow,
     QHBoxLayout,
     QPushButton,
     QLineEdit,
@@ -108,7 +107,7 @@ def __init__(
 
         # Load font
         if os.path.exists(FONT):
-            logging.info("Loading font from: {}".format(FONT))
+            logging.info(f"Loading font from: {FONT}")
             font_id = QtGui.QFontDatabase.addApplicationFont(FONT)
             font_name = QtGui.QFontDatabase.applicationFontFamilies(font_id)[0]
             self.setFont(QtGui.QFont(font_name, 12 if sys.platform == "darwin" else 10, 300))

main.py

@@ -86,7 +86,7 @@ def main() -> None:
     LoggingHandler.worker_configurer(logging_handler.queue)
 
     # Log software version and GitHub link
-    logging.info("FloppyCat Simple Backup Utility version: " + str(__version__))
+    logging.info(f"FloppyCat Simple Backup Utility version: {__version__}")
     logging.info("https://github.com/F33RNI/FloppyCat")
 
     # Load configs