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
JBPM-9276 - Stunner Empty ErrorRef for the Error Start Event produces build errors #3386
Conversation
SonarCloud Quality Gate failed. 0 Bugs 100.0% Coverage The version of Java (1.8.0_202) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11. |
Jenkins execute compile downstream build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hasys code looks good to me! Thanks agian for this work 👍
Hi @romartin, @LuboTerifaj, @domhanak, I noticed some differences between XMLs generated by Legacy and New designers:
I didn't found any differences at runtime though. I used this (also mentioned in PR description) repo to test several Error Events at runtime and both processes (generated in Legacy and in Stunner) produced the same result: Console log for the process created in the Legacy designer
Console log for the process created in the Stunner
Since I can't see any difference at runtime and I didn't found any information in BPMN 2.0 specification how empty error reference should be addressed I think there is no need to fix it in this PR. Thank you! This PR is ready for the review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Original use case is possible and user can have one event sub process with error start node to handle errors in process.
When there is a named error start event it will take precedence over the one with empty errorRef so that is working too.
Also, there a no build errors for me.
Thanks @hasys
Hi @romartin,
I tested Kogito and XML looks good, will build Business Central locally to perform some End to End tests and will move it to full review. For now code is ready for the review.
Hi @domhanak, @LuboTerifaj, do we need your review here or we can assign it to two Software Engineers?
Thank you!
Business Central
https://drive.google.com/file/d/1OUikD9cxc008URF85eLjrfggPVueKLTK/view?usp=sharing
VS Code extension
https://drive.google.com/file/d/1ZvjXoy2St0A_J1gSXiQsxsgVmVFqUfVo/view?usp=sharing
Repository for tests
https://github.com/bpmn-tutorials/EmptyErrorTest