logging

Author: wrecks-code

smct/multimonitortool.py

@@ -22,7 +22,8 @@ def get_monitor_selection_list():
         _display_string = f"{_monitor_id} | {_monitor_name} | {_monitor_serial}"
         _monitor_selection_list.append(_display_string)
         log(f"Monitor detected: {_display_string}")
-        _monitor_selection_list.sort()
+
+    _monitor_selection_list.sort()
     return _monitor_selection_list
 
 
@@ -42,28 +43,23 @@ def _run_mmt_command(command, destination):
             ],
             check=True,
         )
+        log(f"MultiMonitorTool.exe {command} {destination}")
     except subprocess.CalledProcessError as error:
         log(f"MultiMonitorTool.exe {command} {destination} failed: {error}")
 
 
 def save_mmt_config():
-    _command = "/SaveConfig"
-    log(f"MultiMonitorTool.exe {_command} {paths.MMT_CONFIG_PATH}")
     _run_mmt_command("/SaveConfig", paths.MMT_CONFIG_PATH)
 
 
 def enable_monitor():
-    _command = "/LoadConfig"
-    log(f"MultiMonitorTool.exe {_command} {paths.MMT_CONFIG_PATH}")
-    _run_mmt_command(_command, paths.MMT_CONFIG_PATH)
+    _run_mmt_command("/LoadConfig", paths.MMT_CONFIG_PATH)
 
 
 def disable_monitor():
-    _command = "/disable"
     selected_monitor_id = get_selected_monitor_id()
     if selected_monitor_id:
-        _run_mmt_command(_command, selected_monitor_id)
-        log(f"MultiMonitorTool.exe {_command} {selected_monitor_id}")
+        _run_mmt_command("/disable", selected_monitor_id)
     else:
         log("disable_monitor() - Can't find selected monitor id")