Changeset 18039 for branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters

Timestamp:

08/03/21 10:53:47 (3 years ago)

Author:

dpiringe

Message:

#3026

• fixed the possibility for duplicated result entries in template generation
• added a check for duplicated result entries in Runner to prevent an Key already exists exception (a user could manually manipulate the template file to generate this type of exception, thats the reason for a second check)
• fixed the relative to absolute path transformation in JsonTemplateInstantiator

File:

• branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters/AlgorithmConverter.cs (modified) (1 diff)

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

@@ -28 +28 @@
       IAlgorithm algorithm = value as IAlgorithm;
       foreach (var res in algorithm.Results) {
-        item.AddChildren(root.Extract(res, root));
-        /*
-        item.AddChildren(new ResultJsonItem() {
-          Name = res.Name,
-          Description = res.Description
-        });*/
+        var resultItem = root.Extract(res, root);
+        // fetch all result parameter items
+        var resultParameterItems = 
+          item.Where(x => x
+          .GetType()
+          .GetInterfaces()
+          .Any(y => y == typeof(IResultJsonItem)));
+        // check for duplicates (to prevent double result items,
+        // which can occur with result parameters)
+        if(!resultParameterItems.Any(x => x.Name == resultItem.Name))
+          item.AddChildren(resultItem);
       }
       item.AddChildren(root.Extract(algorithm.Problem, root));