Skip to content

Commit

Permalink
fix(BpmnRules): disallow boundary event as messageFlow target
Browse files Browse the repository at this point in the history
closes #1300
  • Loading branch information
MaxTru committed Jun 22, 2020
1 parent a445144 commit 390df08
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions lib/features/rules/BpmnRules.js
Original file line number Diff line number Diff line change
Expand Up @@ -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') &&
Expand Down
2 changes: 1 addition & 1 deletion test/spec/features/rules/BpmnRulesSpec.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
});
Expand Down

0 comments on commit 390df08

Please sign in to comment.