Changeset 513 for trunk/sources/HeuristicLab.CEDMA.Core/DatabaseOperatorLibrary.cs

Timestamp:

08/13/08 17:09:48 (16 years ago)

Author:

gkronber

Message:

Fixed the problem that sometimes an operator link was not patched before a view was created (which resulted in a NullException) by adding a property for the source database in OperatorLink! and updating this property recursively whenever an operator is loaded from the CEDMA DB.

The method for recursively setting the property is in static class OperatorLinkPatcher because the code is needed in Agent, DatabaseOperatorLibrary and OperatorLink.

(ticket #211)

File:

• trunk/sources/HeuristicLab.CEDMA.Core/DatabaseOperatorLibrary.cs (modified) (1 diff)

trunk/sources/HeuristicLab.CEDMA.Core/DatabaseOperatorLibrary.cs

@@ -85 +85 @@
       }
 
-      // patch all OperatorLinks
+      // set the DB source for each operator
       foreach(IOperator op in group.Operators) {
-        PatchLinks(op);
-      }
-    }
-
-    private void PatchLinks(IOperatorGraph opGraph) {
-      foreach(IOperator op in opGraph.Operators) {
-        PatchLinks(op);
-      }
-    }
-
-    private void PatchLinks(IOperator op) {
-      if(op is OperatorLink) {
-        OperatorLink link = op as OperatorLink;
-        link.Database = Database;
-      }
-      else if(op is CombinedOperator) {
-        CombinedOperator combinedOp = op as CombinedOperator;
-        foreach(IOperator internalOp in combinedOp.OperatorGraph.Operators) {
-          PatchLinks(internalOp);
-        }
-      }
-      // also patch operator links contained (indirectly) in variables
-      foreach(VariableInfo varInfo in op.VariableInfos) {
-        IVariable var = op.GetVariable(varInfo.ActualName);
-        if(var != null && var.Value is IOperatorGraph) {
-          PatchLinks((IOperatorGraph)var.Value);
-        } else if(var != null && var.Value is IOperator) {
-          PatchLinks((IOperator)var.Value);
-        }
+        OperatorLinkPatcher.LinkDatabase(op, Database);
       }
     }