Changeset 17407 for branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters/MultiCheckedOperatorConverter.cs

Timestamp:

01/27/20 12:18:06 (4 years ago)

Author:

dpiringe

Message:

#3026:

• simplified converter inheritance:
  • BaseConverter now only has Inject and Extract from IJsonItemConverter as abstract methods
  • removed ParameterBaseConverter
  • concrete converters have to initialize their JsonItem now -> enables better handling with different types of JsonItem

File:

• branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters/MultiCheckedOperatorConverter.cs (modified) (3 diffs)

branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters/MultiCheckedOperatorConverter.cs

@@ -11 +11 @@
     public override int Priority => 3;
     public override Type ConvertableType => typeof(ICheckedMultiOperator<>);
+    
+    public override void Inject(IItem item, IJsonItem data, IJsonItemConverter root) {
+      base.Inject(item, data, root);
 
-    public override void Populate(IItem value, IJsonItem item, IJsonItemConverter root) {
-      base.Populate(value, item, root);
+      dynamic val = item as dynamic;
+      foreach (var op in val.Operators)
+        val.Operators.SetItemCheckedState(op, GetOperatorState(op.Name, data));
+    }
 
+    public override IJsonItem Extract(IItem value, IJsonItemConverter root) {
+      IJsonItem item = base.Extract(value, root);
       dynamic val = value as dynamic;
       foreach (var op in val.Operators) {
@@ -23 +30 @@
         });
       }
+      return item;
     }
 
-    public override void InjectData(IItem item, IJsonItem data, IJsonItemConverter root) {
-      base.InjectData(item, data, root);
 
-      dynamic val = item as dynamic;
-      foreach (var op in val.Operators)
-        val.Operators.SetItemCheckedState(op, GetOperatorState(op.Name, data));
-    }
-
+    #region Helper
     private bool GetOperatorState(string name, IJsonItem data) {
       foreach(var op in data.Children) {
@@ -39 +41 @@
       return false;
     }
+    #endregion
   }
 }