Skip to content

Fix boundary event iterator desynchronization in ContinueProcessOperation #4158

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
uucoding wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix boundary event iterator desynchronization in ContinueProcessOperation #4158

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
92df09d
Fix boundary event iterator desynchronization in ContinueProcessOpera…
uucoding Jan 22, 2026
e8cbc18
Merge pull request #1 from uucoding/uucoding-patch-fix-ContinueProces…
uucoding Jan 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 14 additions & 5 deletions ...owable-engine/src/main/java/org/flowable/engine/impl/agenda/ContinueProcessOperation.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
import java.util.stream.Collectors;

import org.flowable.bpmn.model.Activity;
import org.flowable.bpmn.model.BoundaryEvent;
Expand Down Expand Up @@ -360,11 +361,8 @@ protected List<ExecutionEntity> createBoundaryEvents(List<BoundaryEvent> boundar
// The parent execution becomes a scope, and a child execution is created for each of the boundary events
for (BoundaryEvent boundaryEvent : boundaryEvents) {

if (!(boundaryEvent.getBehavior() instanceof BoundaryEventRegistryEventActivityBehavior)) {
if (CollectionUtil.isEmpty(boundaryEvent.getEventDefinitions())
|| (boundaryEvent.getEventDefinitions().get(0) instanceof CompensateEventDefinition)) {
continue;
}
if (!isExecutableBoundaryEvent(boundaryEvent)) {
continue;
}

// A Child execution of the current execution is created to represent the boundary event being active
Expand All @@ -389,6 +387,7 @@ protected List<ExecutionEntity> createBoundaryEvents(List<BoundaryEvent> boundar

protected void executeBoundaryEvents(List<BoundaryEvent> boundaryEvents, List<ExecutionEntity> boundaryEventExecutions) {
if (!CollectionUtil.isEmpty(boundaryEventExecutions)) {
boundaryEvents = boundaryEvents.stream().filter(this::isExecutableBoundaryEvent).collect(Collectors.toList());
Iterator<BoundaryEvent> boundaryEventsIterator = boundaryEvents.iterator();
Iterator<ExecutionEntity> boundaryEventExecutionsIterator = boundaryEventExecutions.iterator();

Expand All @@ -401,4 +400,14 @@ protected void executeBoundaryEvents(List<BoundaryEvent> boundaryEvents, List<Ex
}
}
}

private boolean isExecutableBoundaryEvent(BoundaryEvent boundaryEvent) {
if (!(boundaryEvent.getBehavior() instanceof BoundaryEventRegistryEventActivityBehavior)) {
if (CollectionUtil.isEmpty(boundaryEvent.getEventDefinitions())
|| (boundaryEvent.getEventDefinitions().get(0) instanceof CompensateEventDefinition)) {
return false;
}
}
return true;
}
}