Skip to content
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.

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

Event subscription is missing after migration of an active embedded sub process #3928

Closed
basclaessen opened this issue Jul 22, 2024 · 2 comments · Fixed by #3929
Closed

Event subscription is missing after migration of an active embedded sub process #3928

basclaessen opened this issue Jul 22, 2024 · 2 comments · Fixed by #3929

Comments

@basclaessen
Copy link
Contributor

basclaessen commented Jul 22, 2024

Describe the bug
When a process instance with an active embedded non interrupting event sub process is migrated, the subscription is missing.
For a signal, message or event register event the event subscription is missing.
For a timer event the timer job is missing.

Expected behavior
Event subscriptions and timer jobs should not be missing after migration of a process instance with an active embedded event sub process.

Code
I'll add a pull request with test cases and a fix.

Additional context
main branch
flowable-release-6.8.1 branch
flowable-release-7.1.0 branch
flowable-release-7.2.0 branch

basclaessen added a commit to basclaessen/flowable-engine that referenced this issue Jul 22, 2024
@basclaessen basclaessen changed the title Event subscription is missing after migration of an active sub process Event subscription is missing after migration of an active embedded sub process Jul 23, 2024
@basclaessen
Copy link
Contributor Author

@tijsrademakers Can you take a look at my PR and if you agree merge it before the next Flowable release please?

tijsrademakers pushed a commit that referenced this issue Sep 2, 2024
… interrupting event sub process (#3928) (#3929)

* Fixed missing subscriptions after migration of an active non interrupting event sub process (#3928)

* Added missing package import

* In case of a timer event, only process event sub process when no timer job is present

* Added testcases and fix when migrating an event sub process with two started sub processes

* Fixed imports
@tijsrademakers
Copy link
Contributor

@basclaessen thanks, I've merged the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants