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

Camunda Modeler allows multiple incoming sequence flows for events after an event based gateway #3984

Closed
ingorichtsmeier opened this issue Nov 14, 2023 · 5 comments
Labels
bug Something isn't working miwg needs more information Requires additional information to be actionable.

Comments

@ingorichtsmeier
Copy link

ingorichtsmeier commented Nov 14, 2023

Describe the bug

Since Version 5.5.1, the Camunda Modeler breaks the BPMN specification and allows multiple incoming sequence flows to an event that follows an event based gateway:

grafik

According to the BPMN specification this is not allowed (page 298 Business Process Model and Notation, v2.0):

grafik

Steps to reproduce

Create the model from above in Camunda Modeler 5.4.1 and 5.16.

In 5.4.1, you can not connect the events with an additional sequence flow:

grafik

Expected behavior

It should be impossible to connect both events in the latest version as well.

Environment

Impossible:
Camunda Modeler system information

  • Version: 5.4.1
  • Operating System: Windows 10 amd64
  • Plugins: Cloud Connect, bpmn-js Token Simulation, Migration, Transaction Boundaries, Camunda Property Infos, Tooltip Plugin
  • Execution Platform: BPMN - Camunda Platform 8

Possible:
Camunda Modeler system information

  • Version: 5.16.0
  • Operating System: Windows 10 amd64
  • Plugins: bpmn-js Token Simulation, Camunda Property Infos, Tooltip Plugin, Transaction Boundaries, Cloud Connect, Migration
  • Execution Platform:

Additional context

Maybe this issue happens in the BPMN-js library.

@ingorichtsmeier ingorichtsmeier added the bug Something isn't working label Nov 14, 2023
@nikku
Copy link
Member

nikku commented Nov 14, 2023

Cannot reproduce in Camunda Modeler v5.17.0:

capture WMaKcm_optimized

capture o6AwzE_optimized

Could you disable your plug-ins to verify these don't interfere?

@nikku nikku added the needs more information Requires additional information to be actionable. label Nov 14, 2023
@ingorichtsmeier
Copy link
Author

It happens in the Web modeler as well. I found it there first.

@ingorichtsmeier
Copy link
Author

In 5.16 without plugins it looks like in your screen movie and works as expected.

What breaks it in web modeler? (version: today).

Which plugin could influence it? Token simulation is available in both.

@nikku nikku added the miwg label Nov 15, 2023
@nikku
Copy link
Member

nikku commented Nov 15, 2023

It seems like token simulation breaks it:

capture lVY5Xs_optimized

@marstamm
Copy link
Member

I created bpmn-io/bpmn-js-token-simulation#139 in the token simulation repo. As this is not an issue in the desktop modeler itself, I am closing this issue in favor of the plugin issue.

We can consider opening an issue in the web-modeler for tracking, since we ship it there.

@marstamm marstamm closed this as not planned Won't fix, can't repro, duplicate, stale Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working miwg needs more information Requires additional information to be actionable.
Projects
None yet
Development

No branches or pull requests

3 participants