fix(bpmnRules): disallow boundaryEvents as message flow targets

closes #1300
bpmn-io · Jun 22, 2020 · f9e6539 · f9e6539
1 parent a445144
commit f9e6539
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/features/rules/BpmnRules.js b/lib/features/rules/BpmnRules.js
@@ -280,6 +280,7 @@ function isMessageFlowSource(element) {
 function isMessageFlowTarget(element) {
   return (
     is(element, 'bpmn:InteractionNode') &&
+    !is(element, 'bpmn:BoundaryEvent') &&
     !isForCompensation(element) && (
       !is(element, 'bpmn:Event') || (
         is(element, 'bpmn:CatchEvent') &&

diff --git a/test/spec/features/rules/BpmnRulesSpec.js b/test/spec/features/rules/BpmnRulesSpec.js
@@ -656,7 +656,7 @@ describe('features/modeling/rules - BpmnRules', function() {
 
       expectCanConnect('Task_in_OtherProcess', 'BoundaryEvent_on_SubProcess', {
         sequenceFlow: false,
-        messageFlow: true,
+        messageFlow: false,
         association: false,
         dataAssociation: false
       });