This is an automated email from the ASF dual-hosted git repository.

mweiler pushed a commit to branch main
in repository 
https://gitbox.apache.org/repos/asf/incubator-kie-kogito-runtimes.git


The following commit(s) were added to refs/heads/main by this push:
     new b15fe2b042 [incubator-kie-issues#2171] Allow signaling of a process 
instance in ERROR state (#4130)
b15fe2b042 is described below

commit b15fe2b042ccf8d1cef05fbf39e7dd21826d58f3
Author: Martin Weiler <[email protected]>
AuthorDate: Fri Nov 21 07:34:56 2025 -0700

    [incubator-kie-issues#2171] Allow signaling of a process instance in ERROR 
state (#4130)
    
    * [incubator-kie-issues#2171] Allow signaling of a process instance in 
ERROR state
    
    * Process instance in ERROR state cannot be aborted
---
 .../main/java/org/jbpm/workflow/instance/impl/NodeInstanceImpl.java    | 3 ++-
 .../org/jbpm/workflow/instance/impl/WorkflowProcessInstanceImpl.java   | 2 +-
 .../src/main/java/org/kie/kogito/process/impl/ProcessServiceImpl.java  | 1 -
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/NodeInstanceImpl.java
 
b/jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/NodeInstanceImpl.java
index d2e8b2e8d9..3ba12c2e66 100755
--- 
a/jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/NodeInstanceImpl.java
+++ 
b/jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/NodeInstanceImpl.java
@@ -271,7 +271,8 @@ public abstract class NodeInstanceImpl implements 
org.jbpm.workflow.instance.Nod
                             getNodeDefinitionId(), processInstance.getId(), 
processInstance.getProcessId());
                     throw executionException;
                 }
-                logger.error("Error executing node instance '{}' (node '{}' 
id: '{}') in process instance '{}' (process: '{}') in a transactional 
environment (Wrapping)", getStringId(), getNodeName(),
+                logger.error("Error {} executing node instance '{}' (node '{}' 
id: '{}') in process instance '{}' (process: '{}') in a transactional 
environment (Wrapping)", e.getMessage(),
+                        getStringId(), getNodeName(),
                         getNodeDefinitionId(), processInstance.getId(), 
processInstance.getProcessId());
                 throw new 
ProcessInstanceExecutionException(this.getProcessInstance().getId(), 
this.getNodeDefinitionId(), this.getId(), e.getMessage(), e);
             }
diff --git 
a/jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/WorkflowProcessInstanceImpl.java
 
b/jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/WorkflowProcessInstanceImpl.java
index 261d41326c..4b2ddfc8bb 100755
--- 
a/jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/WorkflowProcessInstanceImpl.java
+++ 
b/jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/impl/WorkflowProcessInstanceImpl.java
@@ -681,7 +681,7 @@ public abstract class WorkflowProcessInstanceImpl extends 
ProcessInstanceImpl im
     public void signalEvent(String type, Object event) {
         logger.debug("Signal {} received with data {} in process instance {}", 
type, event, getStringId());
         synchronized (this) {
-            if (getState() != KogitoProcessInstance.STATE_ACTIVE) {
+            if (getState() != KogitoProcessInstance.STATE_ACTIVE && getState() 
!= KogitoProcessInstance.STATE_ERROR) {
                 return;
             }
 
diff --git 
a/jbpm/jbpm-flow/src/main/java/org/kie/kogito/process/impl/ProcessServiceImpl.java
 
b/jbpm/jbpm-flow/src/main/java/org/kie/kogito/process/impl/ProcessServiceImpl.java
index b1bf3916bc..80af762ec2 100644
--- 
a/jbpm/jbpm-flow/src/main/java/org/kie/kogito/process/impl/ProcessServiceImpl.java
+++ 
b/jbpm/jbpm-flow/src/main/java/org/kie/kogito/process/impl/ProcessServiceImpl.java
@@ -132,7 +132,6 @@ public class ProcessServiceImpl implements ProcessService {
                             pi.abort();
                             return pi;
                         })
-                        .map(ProcessInstance::checkError)
                         .map(ProcessInstance::variables)
                         .map(MappableToModel::toModel));
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to