fix(copy-paste): do not copy message flows without participant

Closes #1902
bpmn-io · May 4, 2023 · d84c638 · d84c638
1 parent 22d59fd
commit d84c638
Show file tree

Hide file tree

Showing 3 changed files with 61 additions and 12 deletions.
diff --git a/lib/features/rules/BpmnRules.js b/lib/features/rules/BpmnRules.js
@@ -1143,16 +1143,6 @@ function canInsert(shape, connection, position) {
     canDrop(shape, connection.parent, position));
 }
 
-/**
- * @param {Element[]} elements
- * @param {Element} element
- *
- * @return {boolean}
- */
-function includes(elements, element) {
-  return (elements && element) && elements.indexOf(element) !== -1;
-}
-
 /**
  * @param {Element[]} elements
  * @param {Element} element
@@ -1164,10 +1154,18 @@ function canCopy(elements, element) {
     return true;
   }
 
-  if (is(element, 'bpmn:Lane') && !includes(elements, element.parent)) {
+  if (is(element, 'bpmn:Lane') && !elements.includes(element.parent)) {
     return false;
   }
 
+  if (is(element, 'bpmn:MessageFlow')) {
+    const source = element.source,
+          target = element.target;
+
+    return elements.includes(is(source, 'bpmn:Participant') ? source : getParent(source, 'bpmn:Participant'))
+      && elements.includes(is(target, 'bpmn:Participant') ? target : getParent(target, 'bpmn:Participant'));
+  }
+
   return true;
 }
 
@@ -1178,4 +1176,4 @@ function canCopy(elements, element) {
  */
 function getRootElement(element) {
   return getParent(element, 'bpmn:Process') || getParent(element, 'bpmn:Collaboration');
-}
+}
diff --git a/test/spec/features/copy-paste/BpmnCopyPasteSpec.js b/test/spec/features/copy-paste/BpmnCopyPasteSpec.js
@@ -964,6 +964,31 @@ describe('features/copy-paste', function() {
 
   });
 
+
+  describe('rules', function() {
+
+    beforeEach(bootstrapModeler(collaborationMultipleXML, {
+      modules: testModules,
+      moddleExtensions: {
+        camunda: camundaPackage
+      }
+    }));
+
+
+    it('should allow copying message flow with parent participants of source and target', inject(function(elementRegistry) {
+
+      // when
+      var tree = copy([ 'IntermediateThrowEvent_1', 'Task_2', 'MessageFlow_1' ]);
+
+      console.log(tree, keys(tree));
+
+      // then
+      expect(keys(tree)).to.have.length(1);
+      expect(getAllElementsInTree(tree, 0)).to.have.length(2);
+    }));
+
+  });
+
 });
 
 

diff --git a/test/spec/features/rules/BpmnRulesSpec.js b/test/spec/features/rules/BpmnRulesSpec.js
@@ -220,6 +220,32 @@ describe('features/modeling/rules - BpmnRules', function() {
       expectCanCopy(boundaryEvent, [ boundaryEvent ], true);
     }));
 
+
+    it('copy message flow with parent participants of source and target', inject(function(elementFactory) {
+
+      // given
+      var sourceParticipant = elementFactory.createShape({ type: 'bpmn:Participant' }),
+          targetParticipant = elementFactory.createShape({ type: 'bpmn:Participant' }),
+          source = elementFactory.createShape({ type: 'bpmn:Task', parent: sourceParticipant }),
+          target = elementFactory.createShape({ type: 'bpmn:Task', parent: targetParticipant }),
+          messageFlow = elementFactory.createConnection({ type: 'bpmn:MessageFlow', source: source, target: target, waypoints: [] });
+
+      // then
+      expectCanCopy(messageFlow, [ sourceParticipant, targetParticipant, source, target, messageFlow ], true);
+    }));
+
+
+    it('copy message flow without parent participants of source and target', inject(function(elementFactory) {
+
+      // given
+      var source = elementFactory.createShape({ type: 'bpmn:Task' }),
+          target = elementFactory.createShape({ type: 'bpmn:Task' }),
+          messageFlow = elementFactory.createConnection({ type: 'bpmn:MessageFlow', source: source, target: target, waypoints: [] });
+
+      // then
+      expectCanCopy(messageFlow, [ source, target, messageFlow ], false);
+    }));
+
   });