bpmn-js 8.8.0 → 9.0.0-subprocesses.1

package/lib/features/context-pad/ContextPadProvider.js

@@ -197,7 +197,7 @@ ContextPadProvider.prototype.getContextPadEntries = function(element) {
   }
 
 
-  if (isAny(businessObject, [ 'bpmn:Lane', 'bpmn:Participant' ]) && isExpanded(businessObject)) {
+  if (isAny(businessObject, [ 'bpmn:Lane', 'bpmn:Participant' ]) && isExpanded(element)) {
 
     var childLanes = getChildLanes(element);
 

package/lib/features/copy-paste/BpmnCopyPaste.js

@@ -1,5 +1,6 @@
 import {
   getBusinessObject,
+  getDi,
   is
 } from '../../util/ModelUtil';
 
@@ -45,23 +46,13 @@ export default function BpmnCopyPaste(bpmnFactory, eventBus, moddleCopy) {
         element = context.element;
 
     var businessObject = descriptor.oldBusinessObject = getBusinessObject(element);
+    var di = descriptor.oldDi = getDi(element);
 
     descriptor.type = element.type;
 
     copyProperties(businessObject, descriptor, 'name');
 
-    descriptor.di = {};
-
-    // colors will be set to DI
-    copyProperties(businessObject.di, descriptor.di, [
-      'fill',
-      'stroke',
-      'background-color',
-      'border-color',
-      'color'
-    ]);
-
-    copyProperties(businessObject.di, descriptor, 'isExpanded');
+    copyProperties(di, descriptor, 'isExpanded');
 
     if (isLabel(descriptor)) {
       return descriptor;
@@ -135,11 +126,13 @@ export default function BpmnCopyPaste(bpmnFactory, eventBus, moddleCopy) {
     var cache = context.cache,
         descriptor = context.descriptor,
         oldBusinessObject = descriptor.oldBusinessObject,
-        newBusinessObject;
+        oldDi = descriptor.oldDi,
+        newBusinessObject, newDi;
 
     // do NOT copy business object if external label
     if (isLabel(descriptor)) {
       descriptor.businessObject = getBusinessObject(cache[ descriptor.labelTarget ]);
+      descriptor.di = getDi(cache[ descriptor.labelTarget ]);
 
       return;
     }
@@ -151,6 +144,13 @@ export default function BpmnCopyPaste(bpmnFactory, eventBus, moddleCopy) {
       newBusinessObject
     );
 
+    newDi = bpmnFactory.create(oldDi.$type);
+
+    descriptor.di = moddleCopy.copyElement(
+      oldDi,
+      newDi
+    );
+
     // resolve references e.g. default sequence flow
     resolveReferences(descriptor, cache);
 

package/lib/features/copy-paste/ModdleCopy.js

@@ -215,11 +215,16 @@ ModdleCopy.prototype.copyProperty = function(property, parent, propertyName) {
 
   var propertyDescriptor = this._moddle.getPropertyDescriptor(parent, propertyName);
 
-  // do NOT copy Ids and references
-  if (propertyDescriptor.isId || propertyDescriptor.isReference) {
+  // do NOT copy references
+  if (propertyDescriptor.isReference) {
     return;
   }
 
+  // copy id
+  if (propertyDescriptor.isId) {
+    return this._copyId(property, parent);
+  }
+
   // copy arrays
   if (isArray(property)) {
     return reduce(property, function(childProperties, childProperty) {
@@ -258,6 +263,18 @@ ModdleCopy.prototype.copyProperty = function(property, parent, propertyName) {
   return property;
 };
 
+ModdleCopy.prototype._copyId = function(id, element) {
+
+  // disallow if already taken
+  if (this._moddle.ids.assigned(id)) {
+    return;
+  } else {
+
+    this._moddle.ids.claim(id, element);
+    return id;
+  }
+};
+
 // helpers //////////
 
 export function getPropertyNames(descriptor, keepDefaultProperties) {

package/lib/features/di-ordering/BpmnDiOrdering.js

@@ -1,5 +1,4 @@
-import { getDi } from '../../draw/BpmnRenderUtil';
-import { getBusinessObject } from '../../util/ModelUtil';
+import { getDi } from '../../util/ModelUtil';
 
 import {
   filter,
@@ -17,7 +16,7 @@ export default function BpmnDiOrdering(eventBus, canvas) {
 
   function orderDi() {
     var root = canvas.getRootElement(),
-        rootDi = getBusinessObject(root).di,
+        rootDi = getDi(root),
         elements,
         diElements;
 

package/lib/features/label-editing/LabelEditingPreview.js

@@ -6,7 +6,7 @@ import {
 } from 'tiny-svg';
 
 import {
-  getBusinessObject,
+  getDi,
   is
 } from '../../util/ModelUtil';
 
@@ -132,7 +132,7 @@ LabelEditingPreview.$inject = [
 // helpers ///////////////////
 
 function getStrokeColor(element, defaultColor) {
-  var bo = getBusinessObject(element);
+  var di = getDi(element);
 
-  return bo.di.get('stroke') || defaultColor || 'black';
+  return di.get('stroke') || defaultColor || 'black';
 }

package/lib/features/label-editing/cmd/UpdateLabelHandler.js

@@ -67,7 +67,8 @@ export default function UpdateLabelHandler(modeling, textRenderer) {
 
       modeling.createLabel(element, labelCenter, {
         id: businessObject.id + '_label',
-        businessObject: businessObject
+        businessObject: businessObject,
+        di: element.di
       });
     }
   }

package/lib/features/modeling/BpmnFactory.js

@@ -83,11 +83,10 @@ BpmnFactory.prototype.createDiLabel = function() {
 };
 
 
-BpmnFactory.prototype.createDiShape = function(semantic, bounds, attrs) {
-
+BpmnFactory.prototype.createDiShape = function(semantic, attrs) {
   return this.create('bpmndi:BPMNShape', assign({
     bpmnElement: semantic,
-    bounds: this.createDiBounds(bounds)
+    bounds: this.createDiBounds()
   }, attrs));
 };
 
@@ -110,14 +109,15 @@ BpmnFactory.prototype.createDiWaypoint = function(point) {
 };
 
 
-BpmnFactory.prototype.createDiEdge = function(semantic, waypoints, attrs) {
+BpmnFactory.prototype.createDiEdge = function(semantic, attrs) {
   return this.create('bpmndi:BPMNEdge', assign({
-    bpmnElement: semantic
+    bpmnElement: semantic,
+    waypoints: this.createDiWaypoints([])
   }, attrs));
 };
 
-BpmnFactory.prototype.createDiPlane = function(semantic) {
-  return this.create('bpmndi:BPMNPlane', {
+BpmnFactory.prototype.createDiPlane = function(semantic, attrs) {
+  return this.create('bpmndi:BPMNPlane', assign({
     bpmnElement: semantic
-  });
+  }, attrs));
 };

package/lib/features/modeling/BpmnUpdater.js

@@ -16,6 +16,7 @@ import {
 
 import {
   getBusinessObject,
+  getDi,
   is
 } from '../../util/ModelUtil';
 
@@ -307,8 +308,9 @@ BpmnUpdater.prototype.updateParent = function(element, oldParent) {
   var parentShape = element.parent;
 
   var businessObject = element.businessObject,
+      di = getDi(element),
       parentBusinessObject = parentShape && parentShape.businessObject,
-      parentDi = parentBusinessObject && parentBusinessObject.di;
+      parentDi = getDi(parentShape);
 
   if (is(element, 'bpmn:FlowNode')) {
     this.updateFlowNodeRefs(businessObject, parentBusinessObject, oldParent && oldParent.businessObject);
@@ -336,13 +338,13 @@ BpmnUpdater.prototype.updateParent = function(element, oldParent) {
     this.updateSemanticParent(businessObject.dataObjectRef, parentBusinessObject);
   }
 
-  this.updateDiParent(businessObject.di, parentDi);
+  this.updateDiParent(di, parentDi);
 };
 
 
 BpmnUpdater.prototype.updateBounds = function(shape) {
 
-  var di = shape.businessObject.di;
+  var di = getDi(shape);
 
   var target = (shape instanceof Label) ? this._getLabel(di) : di;
 
@@ -382,14 +384,17 @@ BpmnUpdater.prototype.updateFlowNodeRefs = function(businessObject, newContainme
 
 
 // update existing sourceElement and targetElement di information
-BpmnUpdater.prototype.updateDiConnection = function(di, newSource, newTarget) {
+BpmnUpdater.prototype.updateDiConnection = function(connection, newSource, newTarget) {
+  var connectionDi = getDi(connection),
+      newSourceDi = getDi(newSource),
+      newTargetDi = getDi(newTarget);
 
-  if (di.sourceElement && di.sourceElement.bpmnElement !== newSource) {
-    di.sourceElement = newSource && newSource.di;
+  if (connectionDi.sourceElement && connectionDi.sourceElement.bpmnElement !== getBusinessObject(newSource)) {
+    connectionDi.sourceElement = newSource && newSourceDi;
   }
 
-  if (di.targetElement && di.targetElement.bpmnElement !== newTarget) {
-    di.targetElement = newTarget && newTarget.di;
+  if (connectionDi.targetElement && connectionDi.targetElement.bpmnElement !== getBusinessObject(newTarget)) {
+    connectionDi.targetElement = newTarget && newTargetDi;
   }
 
 };
@@ -614,69 +619,72 @@ BpmnUpdater.prototype.updateSemanticParent = function(businessObject, newParent,
 
 
 BpmnUpdater.prototype.updateConnectionWaypoints = function(connection) {
-  connection.businessObject.di.set('waypoint', this._bpmnFactory.createDiWaypoints(connection.waypoints));
+  var di = getDi(connection);
+
+  di.set('waypoint', this._bpmnFactory.createDiWaypoints(connection.waypoints));
 };
 
 
 BpmnUpdater.prototype.updateConnection = function(context) {
-
   var connection = context.connection,
       businessObject = getBusinessObject(connection),
-      newSource = getBusinessObject(connection.source),
-      newTarget = getBusinessObject(connection.target),
+      newSource = connection.source,
+      newSourceBo = getBusinessObject(newSource),
+      newTarget = connection.target,
+      newTargetBo = getBusinessObject(connection.target),
       visualParent;
 
   if (!is(businessObject, 'bpmn:DataAssociation')) {
 
     var inverseSet = is(businessObject, 'bpmn:SequenceFlow');
 
-    if (businessObject.sourceRef !== newSource) {
+    if (businessObject.sourceRef !== newSourceBo) {
       if (inverseSet) {
         collectionRemove(businessObject.sourceRef && businessObject.sourceRef.get('outgoing'), businessObject);
 
-        if (newSource && newSource.get('outgoing')) {
-          newSource.get('outgoing').push(businessObject);
+        if (newSourceBo && newSourceBo.get('outgoing')) {
+          newSourceBo.get('outgoing').push(businessObject);
         }
       }
 
-      businessObject.sourceRef = newSource;
+      businessObject.sourceRef = newSourceBo;
     }
 
-    if (businessObject.targetRef !== newTarget) {
+    if (businessObject.targetRef !== newTargetBo) {
       if (inverseSet) {
         collectionRemove(businessObject.targetRef && businessObject.targetRef.get('incoming'), businessObject);
 
-        if (newTarget && newTarget.get('incoming')) {
-          newTarget.get('incoming').push(businessObject);
+        if (newTargetBo && newTargetBo.get('incoming')) {
+          newTargetBo.get('incoming').push(businessObject);
         }
       }
 
-      businessObject.targetRef = newTarget;
+      businessObject.targetRef = newTargetBo;
     }
   } else
 
   if (is(businessObject, 'bpmn:DataInputAssociation')) {
 
     // handle obnoxious isMsome sourceRef
-    businessObject.get('sourceRef')[0] = newSource;
+    businessObject.get('sourceRef')[0] = newSourceBo;
 
-    visualParent = context.parent || context.newParent || newTarget;
+    visualParent = context.parent || context.newParent || newTargetBo;
 
-    this.updateSemanticParent(businessObject, newTarget, visualParent);
+    this.updateSemanticParent(businessObject, newTargetBo, visualParent);
   } else
 
   if (is(businessObject, 'bpmn:DataOutputAssociation')) {
-    visualParent = context.parent || context.newParent || newSource;
+    visualParent = context.parent || context.newParent || newSourceBo;
 
-    this.updateSemanticParent(businessObject, newSource, visualParent);
+    this.updateSemanticParent(businessObject, newSourceBo, visualParent);
 
     // targetRef = new target
-    businessObject.targetRef = newTarget;
+    businessObject.targetRef = newTargetBo;
   }
 
   this.updateConnectionWaypoints(connection);
 
-  this.updateDiConnection(businessObject.di, newSource, newTarget);
+  this.updateDiConnection(connection, newSource, newTarget);
 };
 
 

package/lib/features/modeling/ElementFactory.js

@@ -6,7 +6,15 @@ import {
 
 import inherits from 'inherits';
 
-import { is } from '../../util/ModelUtil';
+import {
+  getBusinessObject,
+  getDi,
+  is
+} from '../../util/ModelUtil';
+
+import {
+  isAny
+} from '../modeling/util/ModelingUtil';
 
 import {
   isExpanded
@@ -18,6 +26,10 @@ import {
   DEFAULT_LABEL_SIZE
 } from '../../util/LabelUtil';
 
+import {
+  ensureCompatDiRef
+} from '../../util/CompatibilityUtil';
+
 
 /**
  * A bpmn-aware factory for diagram-js shapes
@@ -46,7 +58,8 @@ ElementFactory.prototype.create = function(elementType, attrs) {
   // we assume their businessObjects have already been created
   // and wired via attrs
   if (elementType === 'label') {
-    return this.baseCreate(elementType, assign({ type: 'label' }, DEFAULT_LABEL_SIZE, attrs));
+    var di = attrs.di || this._bpmnFactory.createDiLabel();
+    return this.baseCreate(elementType, assign({ type: 'label', di: di }, DEFAULT_LABEL_SIZE, attrs));
   }
 
   return this.createBpmnElement(elementType, attrs);
@@ -58,7 +71,8 @@ ElementFactory.prototype.createBpmnElement = function(elementType, attrs) {
 
   attrs = attrs || {};
 
-  var businessObject = attrs.businessObject;
+  var businessObject = attrs.businessObject,
+      di = attrs.di;
 
   if (!businessObject) {
     if (!attrs.type) {
@@ -66,22 +80,23 @@ ElementFactory.prototype.createBpmnElement = function(elementType, attrs) {
     }
 
     businessObject = this._bpmnFactory.create(attrs.type);
+
+    ensureCompatDiRef(businessObject);
   }
 
-  if (!businessObject.di) {
+  if (!isModdleDi(di)) {
+    var diAttrs = assign(
+      di || {},
+      { id: businessObject.id + '_di' }
+    );
+
     if (elementType === 'root') {
-      businessObject.di = this._bpmnFactory.createDiPlane(businessObject, [], {
-        id: businessObject.id + '_di'
-      });
+      di = this._bpmnFactory.createDiPlane(businessObject, diAttrs);
     } else
     if (elementType === 'connection') {
-      businessObject.di = this._bpmnFactory.createDiEdge(businessObject, [], {
-        id: businessObject.id + '_di'
-      });
+      di = this._bpmnFactory.createDiEdge(businessObject, diAttrs);
     } else {
-      businessObject.di = this._bpmnFactory.createDiShape(businessObject, {}, {
-        id: businessObject.id + '_di'
-      });
+      di = this._bpmnFactory.createDiShape(businessObject, diAttrs);
     }
   }
 
@@ -91,12 +106,6 @@ ElementFactory.prototype.createBpmnElement = function(elementType, attrs) {
     }, attrs);
   }
 
-  if (attrs.di) {
-    assign(businessObject.di, attrs.di);
-
-    delete attrs.di;
-  }
-
   applyAttributes(businessObject, attrs, [
     'processRef',
     'isInterrupting',
@@ -105,11 +114,11 @@ ElementFactory.prototype.createBpmnElement = function(elementType, attrs) {
   ]);
 
   if (attrs.isExpanded) {
-    applyAttribute(businessObject.di, attrs, 'isExpanded');
+    applyAttribute(di, attrs, 'isExpanded');
   }
 
   if (is(businessObject, 'bpmn:ExclusiveGateway')) {
-    businessObject.di.isMarkerVisible = true;
+    di.isMarkerVisible = true;
   }
 
   var eventDefinitions,
@@ -131,65 +140,69 @@ ElementFactory.prototype.createBpmnElement = function(elementType, attrs) {
     delete attrs.eventDefinitionType;
   }
 
-  size = this._getDefaultSize(businessObject);
+  size = this.getDefaultSize(businessObject, di);
 
   attrs = assign({
-    businessObject: businessObject,
     id: businessObject.id
-  }, size, attrs);
+  }, size, attrs, {
+    businessObject: businessObject,
+    di: di
+  });
 
   return this.baseCreate(elementType, attrs);
 };
 
 
-ElementFactory.prototype._getDefaultSize = function(semantic) {
+ElementFactory.prototype.getDefaultSize = function(element, di) {
 
-  if (is(semantic, 'bpmn:SubProcess')) {
+  var bo = getBusinessObject(element);
+  di = di || getDi(element);
 
-    if (isExpanded(semantic)) {
+  if (is(bo, 'bpmn:SubProcess')) {
+    if (isExpanded(bo, di)) {
       return { width: 350, height: 200 };
     } else {
       return { width: 100, height: 80 };
     }
   }
 
-  if (is(semantic, 'bpmn:Task')) {
+  if (is(bo, 'bpmn:Task')) {
     return { width: 100, height: 80 };
   }
 
-  if (is(semantic, 'bpmn:Gateway')) {
+  if (is(bo, 'bpmn:Gateway')) {
     return { width: 50, height: 50 };
   }
 
-  if (is(semantic, 'bpmn:Event')) {
+  if (is(bo, 'bpmn:Event')) {
     return { width: 36, height: 36 };
   }
 
-  if (is(semantic, 'bpmn:Participant')) {
-    if (isExpanded(semantic)) {
+  if (is(bo, 'bpmn:Participant')) {
+    if (isExpanded(bo, di)) {
       return { width: 600, height: 250 };
     } else {
       return { width: 400, height: 60 };
     }
   }
 
-  if (is(semantic, 'bpmn:Lane')) {
+  if (is(bo, 'bpmn:Lane')) {
     return { width: 400, height: 100 };
   }
 
-  if (is(semantic, 'bpmn:DataObjectReference')) {
+  if (is(bo, 'bpmn:DataObjectReference')) {
     return { width: 36, height: 50 };
   }
 
-  if (is(semantic, 'bpmn:DataStoreReference')) {
+  if (is(bo, 'bpmn:DataStoreReference')) {
     return { width: 50, height: 50 };
   }
 
-  if (is(semantic, 'bpmn:TextAnnotation')) {
+  if (is(bo, 'bpmn:TextAnnotation')) {
     return { width: 100, height: 30 };
   }
 
-  if (is(semantic, 'bpmn:Group')) {
+  if (is(bo, 'bpmn:Group')) {
     return { width: 300, height: 300 };
   }
 
@@ -252,4 +265,14 @@ function applyAttribute(element, attrs, attributeName) {
   element[attributeName] = attrs[attributeName];
 
   delete attrs[attributeName];
+}
+
+
+function isModdleDi(element) {
+  return isAny(element, [
+    'bpmndi:BPMNShape',
+    'bpmndi:BPMNEdge',
+    'bpmndi:BPMNDiagram',
+    'bpmndi:BPMNPlane',
+  ]);
 }

package/lib/features/modeling/behavior/DataStoreBehavior.js

@@ -4,6 +4,7 @@ import CommandInterceptor from 'diagram-js/lib/command/CommandInterceptor';
 
 import {
   getBusinessObject,
+  getDi,
   is
 } from '../../../util/ModelUtil';
 
@@ -45,8 +46,9 @@ export default function DataStoreBehavior(
 
       commandStack.execute('dataStore.updateContainment', {
         dataStoreBo: dataStoreBo,
+        dataStoreDi: getDi(dataStore),
         newSemanticParent: newDataStoreParentBo.processRef || newDataStoreParentBo,
-        newDiParent: newDataStoreParentBo.di
+        newDiParent: getDi(newDataStoreParent)
       });
     }
   }

package/lib/features/modeling/behavior/IsHorizontalFix.js

@@ -3,7 +3,8 @@ import inherits from 'inherits';
 import CommandInterceptor from 'diagram-js/lib/command/CommandInterceptor';
 
 import {
-  getBusinessObject
+  getBusinessObject,
+  getDi
 } from '../../../util/ModelUtil';
 
 import {
@@ -26,12 +27,14 @@ export default function IsHorizontalFix(eventBus) {
   ];
 
   this.executed([ 'shape.move', 'shape.create', 'shape.resize' ], function(event) {
-    var bo = getBusinessObject(event.context.shape);
+    var shape = event.context.shape,
+        bo = getBusinessObject(shape),
+        di = getDi(shape);
 
-    if (isAny(bo, elementTypesToUpdate) && !bo.di.get('isHorizontal')) {
+    if (isAny(bo, elementTypesToUpdate) && !di.get('isHorizontal')) {
 
       // set attribute directly to avoid modeling#updateProperty side effects
-      bo.di.set('isHorizontal', true);
+      di.set('isHorizontal', true);
     }
   });
 

package/lib/features/modeling/behavior/LabelBehavior.js

@@ -6,7 +6,8 @@ import inherits from 'inherits';
 
 import {
   is,
-  getBusinessObject
+  getBusinessObject,
+  getDi
 } from '../../../util/ModelUtil';
 
 import {
@@ -153,27 +154,27 @@ export default function LabelBehavior(
 
     var context = event.context,
         element = context.shape,
-        businessObject,
+        labelTarget = context.labelTarget,
         di;
 
     // we want to trigger on real labels only
-    if (!element.labelTarget) {
+    if (!labelTarget) {
       return;
     }
 
     // we want to trigger on BPMN elements only
-    if (!is(element.labelTarget || element, 'bpmn:BaseElement')) {
+    if (!is(labelTarget, 'bpmn:BaseElement')) {
       return;
     }
 
-    businessObject = element.businessObject,
-    di = businessObject.di;
-
+    di = getDi(labelTarget);
 
     if (!di.label) {
       di.label = bpmnFactory.create('bpmndi:BPMNLabel', {
         bounds: bpmnFactory.create('dc:Bounds')
       });
+
+      element.di = di;
     }
 
     assign(di.label.bounds, {