Fix actiongroupswitch fetching persistent variables on startup

JonnyOThan · JonnyOThan · commit 1c3330f0a251 · 2023-05-11T18:25:36.000-04:00
diff --git a/RasterPropMonitor/Auxiliary modules/JSIActionGroupSwitch.cs b/RasterPropMonitor/Auxiliary modules/JSIActionGroupSwitch.cs
@@ -482,8 +482,10 @@ public void Start()
                     persistentVarValid = !string.IsNullOrEmpty(persistentVarName);
                     if (persistentVarValid)
                     {
-                        // fetch the variable once before registering a callback for it, because otherwise the default value will be -1 on creation
-                        rpmComp.GetPersistentVariable(persistentVarName, initialState, perPodPersistenceIsGlobal);
+                        if (!rpmComp.HasPersistentVariable(persistentVarName, perPodPersistenceIsGlobal))
+                        {
+                            rpmComp.SetPersistentVariable(persistentVarName, initialState, perPodPersistenceIsGlobal);
+                        }
                         rpmComp.RegisterVariableCallback("PERSISTENT_" + persistentVarName, PersistentVarChangedCallback);
                     }
                 }

Original file line number	Diff line number	Diff line change
`@@ -482,8 +482,10 @@ public void Start()`
`482`	`482`	`persistentVarValid = !string.IsNullOrEmpty(persistentVarName);`
`483`	`483`	`if (persistentVarValid)`
`484`	`484`	`{`
`485`		`- // fetch the variable once before registering a callback for it, because otherwise the default value will be -1 on creation`
`486`		`- rpmComp.GetPersistentVariable(persistentVarName, initialState, perPodPersistenceIsGlobal);`
	`485`	`+ if (!rpmComp.HasPersistentVariable(persistentVarName, perPodPersistenceIsGlobal))`
	`486`	`+ {`
	`487`	`+ rpmComp.SetPersistentVariable(persistentVarName, initialState, perPodPersistenceIsGlobal);`
	`488`	`+ }`
`487`	`489`	`rpmComp.RegisterVariableCallback("PERSISTENT_" + persistentVarName, PersistentVarChangedCallback);`
`488`	`490`	`}`
`489`	`491`	`}`