[FIX] Change Ownership on Parameter that is option dependent and over…

…ridden in EU overwrites existing values; fixes #353
STARIONGROUP · May 22, 2024 · d8ef101 · d8ef101
1 parent a03d153
commit d8ef101
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/CometServer/Services/BusinessLogic/OldParameterContextProvider.cs b/CometServer/Services/BusinessLogic/OldParameterContextProvider.cs
@@ -210,8 +210,17 @@ public ParameterValueSet GetsourceValueSet(Guid? option, Guid? state)
                         : this.OldValueSet.FirstOrDefault(x => x.ActualOption == option.Value);
                 }
 
-                // old is only state dependent
-                return this.OldDefaultState != null ? this.OldValueSet.FirstOrDefault(x => x.ActualState == this.OldDefaultState.Iid) : this.OldValueSet.FirstOrDefault();
+                if (!this.IsOldOptionDependent)
+                {
+                    // old is only state dependent
+                    return this.OldDefaultState != null ? this.OldValueSet.FirstOrDefault(x => x.ActualState == this.OldDefaultState.Iid) : this.OldValueSet.FirstOrDefault();
+                }
+
+                if (!this.IsOldStateDependent)
+                {
+                    // old is only option dependent
+                    return this.OldDefaultOption != null ? this.OldValueSet.FirstOrDefault(x => x.ActualOption == option.Value) : this.OldValueSet.FirstOrDefault();
+                }
             }
 
             // new is only state dependent