chore: clear typehinting errors

dzgui/model/proxy_model.py

@@ -1,6 +1,6 @@
 import re
 
-from typing import Any, Optional, TYPE_CHECKING, Union
+from typing import Any, TYPE_CHECKING, Union
 from warnings import deprecated
 
 from dzgui.const.enum import FilterMode
@@ -14,7 +14,6 @@ if TYPE_CHECKING:
     from dzgui.model.servers import NewPlayerCount
 
 
-# TODO: annotate list contents (list[list[Any]])
 class ProxyModelManager:
     """
     Manages access to cached FastInsertListStore resources and
@@ -35,9 +34,13 @@ class ProxyModelManager:
         self.proxy_model: "FastInsertListStore"
         self.filter_man = filter_man
 
-        # TODO: list typehints
-        self.control_model: list | None = None
-        self.filtered: list = []
+        self.control_model: (
+            list[tuple[str, str, str, str, int, int, int, str, int, int, str, bool]]
+            | None
+        ) = None
+        self.filtered: list[
+            tuple[str, str, str, str, int, int, int, str, int, int, str, bool]
+        ] = []
         self.success = True
 
     def has_control_model(self) -> bool:
@@ -48,7 +51,9 @@ class ProxyModelManager:
     def append_row(self, row: list) -> None:
         self.proxy_model.append(row)
 
-    def append_row_to_control(self, row: list) -> None:
+    def append_row_to_control(
+        self, row: tuple[str, str, str, str, int, int, int, str, int, int, str, bool]
+    ) -> None:
         if self.control_model is None:
             raise AttributeError("Trying to add rows to a non-existent model")
         self.control_model.append(row)

dzgui/model/servers.py

@@ -60,6 +60,7 @@ class ServerModelManager:
         There may be cases where you want to instantiate this class without dumping servers,
         e.g., adding saved servers from another tab
         """
+
         self.first_iteration = True
         match self.enum:
             case ServerTab.BROWSER:
@@ -92,12 +93,10 @@ class ServerModelManager:
             futures = [executor.submit(job, key, APPID_DAYZ, param) for param in params]
             for future in as_completed(futures):
                 try:
-
                     # NOTE: faciliates early aborting via sigint
                     # TODO: make this logic available to all dump contexts
                     if self.controller.get_exit_event().is_set():
                         return
-
                     self.thread_man.increment_dialog()
                     res = future.result(timeout=API_TIMEOUT)
                     if res.status != 200 or not res.parsed:
@@ -236,7 +235,6 @@ class ServerModelManager:
     def add_to_history(self, record: dict[str, Any]) -> None:
         proxy_man = self._get_proxy_man()
         row = Servers.parse_json([record])
-        print(type(row[0]))
         proxy_man.append_row_to_history(row[0])
         self.update_history()
 
@@ -285,7 +283,7 @@ class ServerModelManager:
         proxy_man = self._get_proxy_man()
         config_man = self.controller.get_config_man()
         fqip = Servers.response_to_fqip(row)
-        record = response.get_record()  # Servers.response_to_record(row)
+        record = response.get_record()
 
         # TODO: less convoluted
         if delete:
@@ -394,7 +392,8 @@ class ServerModelManager:
 
         # TODO: distinguish signals, e.g. "servers_failed_to_load", "servers_loaded_empty"
         # customize statusbar and dialog accordingly
-        self.emitter.emit("servers_loaded", self.enum)
+        # TODO: drop, causes errors
+        # self.emitter.emit("servers_loaded", self.enum)
         # FIXME: destroy wait dialog first
         # see threadman.set_cleanup_func(_, destroy_first=True)
         if show_dialog: