Skip to content

Commit 4308192

Browse files
committed
ref: remove unnecessary custom signals for preferences dialog
1 parent e9486e0 commit 4308192

3 files changed

Lines changed: 3 additions & 7 deletions

File tree

CHANGELOG

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
- enh: allow to disable update check on startup via settings
55
- enh: apply button in upload settings only affects cache location
66
- enh: move "apply" button in downloads settings next to "browse" button
7+
- ref: remove unnecessary custom signals for preferences dialog
78
- ref: decorate PyQtSlots
89
- tests: restore previous users settings after testing
910
0.17.3

dcoraid/gui/main.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ def __init__(self, *args, **kwargs):
8989
# Disable native menubar (e.g. on Mac)
9090
self.menubar.setNativeMenuBar(False)
9191
# File menu
92-
self.actionPreferences.triggered.connect(self.dlg_pref.show_server)
92+
self.actionPreferences.triggered.connect(self.dlg_pref.show)
9393
self.actionSetupWizard.triggered.connect(self.on_wizard)
9494
# Help menu
9595
self.actionSoftware.triggered.connect(self.on_action_software)

dcoraid/gui/preferences/dlg_preferences.py

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,6 @@
1818

1919

2020
class PreferencesDialog(QtWidgets.QMainWindow):
21-
show_server = QtCore.pyqtSignal()
22-
show_user = QtCore.pyqtSignal()
23-
2421
def __init__(self, *args, **kwargs):
2522
"""Create a new window for preferences
2623
"""
@@ -35,8 +32,6 @@ def __init__(self, *args, **kwargs):
3532
self.checkBox_check_updates.toggled.connect(
3633
self.on_general_check_for_updates)
3734
# server
38-
self.show_server.connect(self.on_show_server)
39-
self.show_user.connect(self.on_show_user)
4035
self.toolButton_server_update.clicked.connect(self.on_update_server)
4136
self.tabWidget.currentChanged.connect(self.on_tab_changed)
4237
self.toolButton_api_token_renew.clicked.connect(
@@ -243,7 +238,7 @@ def on_show_server(self):
243238
self.comboBox_server.setCurrentText(
244239
self.settings.value("auth/server", "dcor.mpl.mpg.de"))
245240
self.lineEdit_api_key.setText(self.settings.value("auth/api key", ""))
246-
self.tabWidget.setCurrentIndex(0) # server settings
241+
self.tabWidget.setCurrentWidget(self.tab_server)
247242
self.lineEdit_api_key.setEchoMode(
248243
QtWidgets.QLineEdit.EchoMode.PasswordEchoOnEdit)
249244
self.show()

0 commit comments

Comments
 (0)