From 1b1cb33a6e99ee5b1dbd726a35713ffbaa12dcd8 Mon Sep 17 00:00:00 2001 From: aclist <92275929+aclist@users.noreply.github.com> Date: Fri, 27 Feb 2026 22:21:08 +0900 Subject: [PATCH] fix: do not pass BM key during ip method --- dzgui/managers/connection.py | 1 + dzgui/managers/contextmenu.py | 4 +--- dzgui/model/servers.py | 1 - 3 files changed, 2 insertions(+), 4 deletions(-) diff --git a/dzgui/managers/connection.py b/dzgui/managers/connection.py index e597047..aa82622 100644 --- a/dzgui/managers/connection.py +++ b/dzgui/managers/connection.py @@ -27,6 +27,7 @@ class ConnectionManager: if res is None: self.thread_man.set_cleanup_func(StoredFunc(self._connection_failure)) # TODO: add to history if successful + print(res) def _connection_failure(self) -> None: # TODO: more explicit warning message, not necessarily API failure? diff --git a/dzgui/managers/contextmenu.py b/dzgui/managers/contextmenu.py index ccace96..1171b01 100644 --- a/dzgui/managers/contextmenu.py +++ b/dzgui/managers/contextmenu.py @@ -65,9 +65,7 @@ class ContextMenuManager: # TODO: update tab with ! case ContextMenu.CONNECT: record = self.treeview.get_record_string() - # FIXME: context menu connections do not require BM key - key = self.controller.get_config_man().lookup(Preferences.BM) - ConnectionManager(self.controller).connect_ip(record, key) + ConnectionManager(self.controller).connect_by_ip(record) case ContextMenu.REFRESH_PLAYERS: # get record # call a2s on thread diff --git a/dzgui/model/servers.py b/dzgui/model/servers.py index 1c8dbb2..e6eb3ec 100644 --- a/dzgui/model/servers.py +++ b/dzgui/model/servers.py @@ -189,7 +189,6 @@ class ServerModelManager: @call_on_thread(dialog.querying) def add_by_id(self, addr: str) -> None: - # KEY config_man = self.controller.get_config_man() key = config_man.lookup(Preferences.BM) res = Servers.query_by_id(addr, key)