Fix reacting to 'Use a bridge' checkbox state change

milotype · Oct 23, 2023 · 30ad1ca · 30ad1ca
1 parent f880a93
commit 30ad1ca
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/desktop/onionshare/tor_settings_tab.py b/desktop/onionshare/tor_settings_tab.py
@@ -97,7 +97,7 @@ def __init__(
         self.bridge_use_checkbox = QtWidgets.QCheckBox(
             strings._("gui_settings_bridge_use_checkbox")
         )
-        self.bridge_use_checkbox.stateChanged.connect(
+        self.bridge_use_checkbox.clicked.connect(
             self.bridge_use_checkbox_state_changed
         )
 
@@ -529,15 +529,17 @@ def connection_type_bundled_toggled(self, checked):
             self.connection_type_socks.hide()
             self.connection_type_bridges_radio_group.show()
 
-    def bridge_use_checkbox_state_changed(self, state):
+    def bridge_use_checkbox_state_changed(self):
         """
         'Use a bridge' checkbox changed
         """
-        if state == QtCore.Qt.Checked:
+        if self.bridge_use_checkbox.isChecked():
+            self.common.log("TorSettingsTab", "bridge_use_checkbox_state_changed", "Use bridge checkbox toggled on")
             self.bridge_settings.show()
             self.bridge_builtin_radio.click()
             self.bridge_builtin_dropdown.setCurrentText("obfs4")
         else:
+            self.common.log("TorSettingsTab", "bridge_use_checkbox_state_changed", "Use bridge checkbox toggled off")
             self.bridge_settings.hide()
 
     def bridge_builtin_radio_toggled(self, checked):