Merge pull request #440 from jmlich/systemd-unit-recursion

onUnitNew is triggered with every property check.
piggz · Jan 15, 2025 · a1b3671 · a1b3671
2 parents e1443cf + 0297d86
commit a1b3671
Showing 1 changed file with 2 additions and 19 deletions.
diff --git a/ui/qml/harbour-amazfish.qml b/ui/qml/harbour-amazfish.qml
@@ -116,19 +116,9 @@ ApplicationWindowPL
         function updateProperties() {
             if (ENABLE_SYSTEMD === "YES"){
                 var activeProperty = systemdServiceIface.getProperty("ActiveState");
-                if (activeProperty === "active") {
-                    serviceActiveState = true;
-                } else {
-                    serviceActiveState = false;
-                }
-
+                serviceActiveState = (activeProperty === "active");
                 var serviceEnabledProperty = systemdServiceIface.getProperty("UnitFileState");
-                if (serviceEnabledProperty === "enabled") {
-                    serviceEnabledState = true;
-                }
-                else {
-                    serviceEnabledState = false;
-                }
+                serviceEnabledState = (serviceEnabledProperty === "enabled");
             }
         }
 
@@ -144,13 +134,6 @@ ApplicationWindowPL
         iface: "org.freedesktop.systemd1.Manager"
         signalsEnabled: true
 
-        signal unitNew(string name)
-        onUnitNew: {
-            if (name == "harbour-amazfish.service" && ENABLE_SYSTEMD === "YES") {
-                systemdServiceIface.updateProperties()
-            }
-        }
-
         function enableService() {
             if(ENABLE_SYSTEMD === "YES") {
                 systemdManager.typedCall("EnableUnitFiles", [{"type":"as", "value":["harbour-amazfish.service"]}, {"type":"b", "value":false}, {"type":"b", "value":true}])