[I] BPMN Editor: renaming/deleting a Message leaves a stale drools:msgref / messageRef on boundary events [incubator-kie-issues]

["AndreyCurious (via GitHub)" <gi...@apache.org>]

AndreyCurious opened a new issue, #2414:
URL: https://github.com/apache/incubator-kie-issues/issues/2414

   ## Describe the bug
   When a Message is renamed or deleted, the editor updates (`drools:msgref`) or clears (`messageRef` + `drools:msgref`) the reference on message events. The list of event types it visits includes `endEvent`, `intermediateThrowEvent`, `startEvent` and `intermediateCatchEvent` — but **not `boundaryEvent`**.
   
   As a result, a boundary event with a `messageEventDefinition`:
   - after **rename**: keeps the old `@_drools:msgref`;
   - after **delete**: keeps a dangling `@_messageRef` (and `@_drools:msgref`) pointing to the removed root `message` — the file then references a non-existent element and fails to parse on reopen.
   
   ## Steps to reproduce
   1. Create a process with a Message and a **boundary message event** that references it.
   2. Rename the message → the boundary event still carries the old `drools:msgref`.
   3. Or delete the message → the boundary event keeps `messageRef` to the deleted message; reopening the diagram fails to parse.
   
   ## Root cause
   In `packages/bpmn-editor/src/mutations/renameMessage.ts` and `packages/bpmn-editor/src/mutations/deleteMessage.ts`, the condition that selects message-bearing flow elements omits `boundaryEvent`.
   
   ## Suggested fix
   Add `boundaryEvent` to that condition in both files, so boundary message events are updated/cleaned like the other message events.
   
   ## Environment
   Apache KIE BPMN Editor, `main`.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org

Re: [I] BPMN Editor: renaming/deleting a Message, Error, Escalation or Signal leaves a stale ref on boundary events [incubator-kie-issues]

["jomarko (via GitHub)" <gi...@apache.org>]

jomarko closed issue #2414: BPMN Editor: renaming/deleting a Message, Error, Escalation or Signal leaves a stale ref on boundary events
URL: https://github.com/apache/incubator-kie-issues/issues/2414


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org