[Fix apache#3513] Different jsonnode instance for parent and children…

… process
fjtirado · May 13, 2024 · bcf4451 · bcf4451
1 parent 4a7e77d
commit bcf4451
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/...rkflow-runtime/src/main/java/org/kie/kogito/serverless/workflow/models/JsonNodeModel.java b/...rkflow-runtime/src/main/java/org/kie/kogito/serverless/workflow/models/JsonNodeModel.java
@@ -35,6 +35,7 @@
 
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
+import com.fasterxml.jackson.databind.node.ObjectNode;
 
 public class JsonNodeModel implements Model, MapInput, MapInputId, MapOutput, MappableToModel<JsonNodeModelOutput> {
 
@@ -94,7 +95,11 @@ public Map<String, Object> updatePartially(Map<String, Object> params) {
     private void update(Map<String, Object> params, Function<JsonNode, JsonNode> merger) {
         if (params.containsKey(SWFConstants.DEFAULT_WORKFLOW_VAR)) {
             params = mutableMap(params);
-            this.workflowdata = merger.apply(JsonObjectUtils.fromValue(params.remove(SWFConstants.DEFAULT_WORKFLOW_VAR)));
+            Object workflowVar = params.remove(SWFConstants.DEFAULT_WORKFLOW_VAR);
+            if (workflowVar instanceof JsonNode) {
+                workflowVar = ((JsonNode) workflowVar).deepCopy();
+            }
+            this.workflowdata = merger.apply(JsonObjectUtils.fromValue(workflowVar));
             this.additionalProperties = params;
         } else {
             this.workflowdata = merger.apply(JsonObjectUtils.fromValue(params));