Merge pull request #15 from AndreWohnsland/dev

Fix weekly hours input mapping

pyproject.toml

@@ -12,7 +12,6 @@ python = ">=3.10,<3.12"
 GitPython = "^3.1.43"
 matplotlib = "^3.9.0"
 pandas = "^2.2.2"
-pyqt5-qt5 = "5.15.2"
 xlsxwriter = "^3.2.0"
 pyqtdarktheme = "^2.1.0"
 qtawesome = "^1.3.1"

src/ui_config_window.py

@@ -86,7 +86,7 @@ def apply_config(self):
         CONFIG_HANDLER.config.subdiv = self.input_subdiv.currentText() or None
         CONFIG_HANDLER.config.name = self.input_name.text()
         CONFIG_HANDLER.config.daily_hours = self.input_daily_hours.value()
-        CONFIG_HANDLER.config.weekly_hours = self.input_daily_hours.value()
+        CONFIG_HANDLER.config.weekly_hours = self.input_weekly_hours.value()
         CONFIG_HANDLER.config.plot_pause = self.input_plot_pause.isChecked()
         selected_days: list[int] = []
         for day in range(7):

src/ui_controller.py

@@ -46,9 +46,7 @@ def user_okay(self, text: str):
         yes_button = message_box.addButton("Yes", QMessageBox.ButtonRole.YesRole)
         message_box.addButton("No", QMessageBox.ButtonRole.NoRole)
         message_box.exec()
-        if message_box.clickedButton() == yes_button:
-            return True
-        return False
+        return message_box.clickedButton() == yes_button
 
     def display_about(self):
         message = (

src/ui_data_window.py

@@ -79,9 +79,7 @@ def __init__(self, main_window: MainWindow):
 
     @property
     def view_day(self):
-        if self.switch_button.isChecked():
-            return True
-        return False
+        return self.switch_button.isChecked()
 
     @property
     def selected_date(self):