[Fix #3390] Do not create container instance, search it (#3432)

apache · Mar 11, 2024 · 5728052 · 5728052
1 parent ccdd173
commit 5728052
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/NodeInstanceContainer.java b/jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/NodeInstanceContainer.java
@@ -23,6 +23,7 @@
 import org.jbpm.workflow.core.node.AsyncEventNode;
 import org.kie.api.definition.process.Node;
 import org.kie.api.definition.process.NodeContainer;
+import org.kie.kogito.internal.process.runtime.KogitoNodeInstance;
 import org.kie.kogito.internal.process.runtime.KogitoNodeInstanceContainer;
 
 import static org.jbpm.ruleflow.core.Metadata.CUSTOM_ASYNC;
@@ -64,7 +65,12 @@ default NodeInstance getByNodeDefinitionId(final String nodeDefinitionId, NodeCo
 
             if (nodeDefinitionId.equals(node.getMetaData().get(UNIQUE_ID))) {
                 if (nodeContainer instanceof Node) {
-                    return ((NodeInstanceContainer) getNodeInstance((Node) nodeContainer)).getNodeInstance(node);
+                    Collection<KogitoNodeInstance> nodeInstances = getKogitoNodeInstances(ni -> ni.getNode().getId() == (((Node) nodeContainer).getId()), true);
+                    if (nodeInstances.isEmpty()) {
+                        return ((NodeInstanceContainer) getNodeInstance((Node) nodeContainer)).getNodeInstance(node);
+                    } else {
+                        return ((NodeInstanceContainer) nodeInstances.iterator().next()).getNodeInstance(node);
+                    }
                 } else {
                     return getNodeInstance(node);
                 }

diff --git a/...workflow-integration-test/src/test/java/org/kie/kogito/quarkus/workflows/RetriggerIT.java b/...workflow-integration-test/src/test/java/org/kie/kogito/quarkus/workflows/RetriggerIT.java
@@ -53,6 +53,12 @@ void testRetrigger() {
                 .accept(ContentType.JSON).when()
                 .post("/management/processes/division/instances/{id}/retrigger", id)
                 .then().statusCode(200).body("workflowdata.response", is(2));
+
+        given()
+                .contentType(ContentType.JSON)
+                .accept(ContentType.JSON).when()
+                .get("/division/{id}", id)
+                .then().statusCode(404);
     }
 
 }