camunda / camunda-modeler

An integrated modeling solution for BPMN, DMN and Forms based on bpmn.io.
https://camunda.com/products/modeler
MIT License
1.51k stars 484 forks source link

Moving of Conditional Event causes removal of variableEvents #4495

Open mevryn opened 2 months ago

mevryn commented 2 months ago

Describe the bug

When start conditional event for event sub process is moved then any configured Variable event is deleted if name of start event activity is defined.

Steps to reproduce

  1. Create new Camunda 7 process
  2. Create event sub process
  3. Change event sub process' start event to non-interrupting conditional start event.
  4. In general properties tab set name e.g "test"
  5. In Condition tab set Variable events to e.g "update,create"
  6. Move this conditional start event, and variable events should be removed image image

Expected behavior

Variable events are not deleted when conditional start event is moved for event sub process.

Environment

Additional context

No response

barmac commented 2 months ago

Can you reproduce it v5.27.0? I wasn't able to do that.

https://github.com/user-attachments/assets/9e38f7a4-869d-4ef7-9a4b-3f8d794a580f

mevryn commented 2 months ago

Set name to this conditional event

barmac commented 2 months ago

Thanks, this way I was able to reproduce the problem. I'll move it to the backlog right now, but we are happy to accept contributions. I believe the bug is in this file: https://github.com/camunda/camunda-bpmn-js-behaviors/blob/main/lib/camunda-platform/RemoveVariableEventBehaviour.js