bpmn-js icon indicating copy to clipboard operation
bpmn-js copied to clipboard

Boundary event can be placed on process via append anything

Open barmac opened this issue 3 years ago • 5 comments

Describe the Bug

With the connectors extension, I can place a boundary event on process (canvas). When I use dragging, it's converted to an intermediate event, but it does not happen when I use the append action (click on the entry in the menu). This breaks the modeler in two ways:

  1. Modeler is broken immediately when I append a non-interrupting boundary event.
  2. Modeler is broken once I try to connect a task to it if it's an interrupting boundary event.

Steps to Reproduce

  1. Go to demo.bpmn.io
  2. Select an element and open the append anything menu
  3. Look for boundary event

Then:

  1. Click non-interrupting (breaks immediately)
  2. Click interrupting and then connect task to it

Expected Behavior

Optimal solution would be to convert the event as soon as appended. We should remove the event definition if it's incompatible.

We could also disable boundary events temporarily in the connectors extension, but it doesn't solve the core issue.

Environment

  • Browser: Chrome 104
  • OS: MacOS
  • Library version: Current + https://github.com/bpmn-io/bpmn-js-connectors-extension

barmac avatar Aug 29 '22 09:08 barmac

Sentry issue: BPMN-IO-DEMO-21H

sentry-io[bot] avatar Aug 29 '22 09:08 sentry-io[bot]

Sentry issue: BPMN-IO-DEMO-2YW

sentry-io[bot] avatar Aug 29 '22 09:08 sentry-io[bot]

@barmac is this the issue that generated the Sentry spike?

CatalinaMoisuc avatar Aug 29 '22 09:08 CatalinaMoisuc

There was no single spike, but this issue is responsible for the two top events-producing issues on sentry for the demo.bpmn.io project. Note that we generated many more events in Camunda Modeler project.

barmac avatar Aug 29 '22 09:08 barmac

The issue is not fixed upstream due to https://github.com/bpmn-io/bpmn-js-connectors-extension/issues/54

barmac avatar Nov 29 '22 08:11 barmac