PinUi: fix recursion issue

remember - don't try to overwrite Qt's `.connect` method!
Nitrokey · Dec 7, 2023 · 0c02f61 · 0c02f61
1 parent aa2a43e
commit 0c02f61
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/nitrokeyapp/secrets_tab/ui.py b/nitrokeyapp/secrets_tab/ui.py
@@ -42,7 +42,7 @@ def choose(self) -> None:
         else:
             self.cancelled.emit()
 
-    def connect(
+    def connect_actions(
         self,
         queried: Optional[Callable[[str], None]],
         chosen: Optional[Callable[[str], None]],

diff --git a/nitrokeyapp/secrets_tab/worker.py b/nitrokeyapp/secrets_tab/worker.py
@@ -84,7 +84,7 @@ def __init__(
         self.query_pin.connect(pin_ui.query)
         self.choose_pin.connect(pin_ui.choose)
 
-        self.pin_ui = pin_ui.connect(
+        self.pin_ui = pin_ui.connect_actions(
             self.pin_queried,
             self.pin_chosen,
             lambda: self.pin_verified.emit(False),