camunda-bpmn-js 0.11.3 → 0.12.1

package/lib/camunda-cloud/features/modeling/behavior/UpdatePropagateAllChildVariablesBehavior.js

@@ -1,11 +1,8 @@
 import {
-  getBusinessObject
+  getBusinessObject,
+  is
 } from 'bpmn-js/lib/util/ModelUtil';
 
-import inherits from 'inherits';
-
-import { is } from 'bpmn-js/lib/util/ModelUtil';
-
 import {
   getCalledElement,
   isPropagateAllChildVariables
@@ -14,148 +11,99 @@ import {
 import {
   getInputParameters,
   getOutputParameters,
-  getInputOutput
+  getIoMapping
 } from '../../../helper/InputOutputHelper';
 
 import CommandInterceptor from 'diagram-js/lib/command/CommandInterceptor';
 
-const HIGH_PRIORITY = 15000;
+const HIGH_PRIORITY = 5000;
 
 
 /**
- * UpdatePropagateAllChildVariablesBehavior reacts to either (1) toggling on propagateAllChildVariables
- * when there are outputParameters present or (2) to adding outputParameters when
- * propagateAllChildVariables is set to true.
- * It will ensure that the propagateAllChildVariables attribute on calledElement
- * extensionElements for callActivities is always consistent with outputParameter mappings
+ * Zeebe BPMN behavior for updating zeebe:propagateAllChildVariables.
  */
-export default function UpdatePropagateAllChildVariablesBehavior(
-    eventBus) {
-
-  CommandInterceptor.call(this, eventBus);
-
-  // Behavior when toggling propagateAllChildVariables /////////////////////////
-  /**
-   * remove outputParameters from zeebe:IoMapping when setting propgateAlLChildVariables
-   * to true in the proeprties panel
-   */
-  this.executed('properties-panel.update-businessobject' , HIGH_PRIORITY, function(context) {
-    const {
-      element,
-      properties
-    } = context;
-
-    // (1) Don't execute this behavior if we are not in a call activity or not
-    // have properties to update or not update the propagateAllChildVariables
-    // to false
-    if (!is(element, 'bpmn:CallActivity') ||
-        !properties ||
-        !!properties.propagateAllChildVariables === false) {
-      return;
-    }
-
-    // (2) Check whether we have outputParameters
-    const outputParameters = getOutputParameters(element),
-          inputParameters = getInputParameters(element);
-
-    if (!outputParameters ||
-      outputParameters.length === 0) {
-      return;
-    }
-
-    // (3) Store old outputParameters and remove them
-    context.oldOutputParameters = outputParameters;
-
-    const inputOutput = getInputOutput(element);
-    inputOutput.outputParameters = [];
-
-    // (4) if we also have no inputParameters, store IOMapping and remove it
-    if (!inputParameters || inputParameters.length === 0) {
-      const extensionElements = getBusinessObject(element).extensionElements;
-      context.oldExtensionElements = extensionElements.values;
-
-      extensionElements.values = extensionElements.values.filter(ele => ele.$type !== 'zeebe:IoMapping');
-    }
-  }, true);
-
-  // Revert behavior when toggling propagateAllChildVariables //////////////////
-  this.reverted('properties-panel.update-businessobject', HIGH_PRIORITY, function(context) {
-    const {
-      element,
-      oldOutputParameters,
-      oldExtensionElements
-    } = context;
-
-    // (1) Only intercept the revert, if the behavior became active
-    if (oldOutputParameters) {
-
-      // (2) If we removed the IOMapping, bring it back first
-      if (oldExtensionElements) {
-        const extensionElements = getBusinessObject(element).extensionElements;
-
-        extensionElements.values = oldExtensionElements;
+export default class UpdatePropagateAllChildVariablesBehavior extends CommandInterceptor {
+  constructor(eventBus, modeling) {
+    super(eventBus);
+
+    /**
+     * Remove zeebe:OutputParameters when zeebe:propagateAllChildVariables is set to true.
+     */
+    this.postExecute('properties-panel.update-businessobject' , HIGH_PRIORITY, function(context) {
+      const {
+        element,
+        properties
+      } = context;
+
+      if (
+        !is(element, 'bpmn:CallActivity')
+        || !properties
+        || !!properties.propagateAllChildVariables === false
+      ) {
+        return;
       }
 
-      // (3) Bring back the outputParameters
-      const inputOutput = getInputOutput(element);
-      inputOutput.outputParameters = oldOutputParameters;
-    }
-  }, true);
-
+      const inputParameters = getInputParameters(element),
+            outputParameters = getOutputParameters(element);
 
-  // Behavior when adding outputParameters ////////////////////////////////////
-  /**
-   * un-toggle propgateAlLChildVariables when adding output parameters
-   */
-  this.executed('properties-panel.update-businessobject-list' , HIGH_PRIORITY, function(context) {
-    const {
-      element,
-      objectsToAdd
-    } = context;
-
-
-    // (1) Exit if we are not in a CallActivity, not adding an OutputParameter or not
-    // having set propagateAllChildVariables to false
-    if (!is(element, 'bpmn:CallActivity') ||
-     !objectsToAdd ||
-     objectsToAdd.length === 0 ||
-     objectsToAdd.filter(obj => is(obj, 'zeebe:Output')).length === 0 ||
-    isPropagateAllChildVariables(element) === false) {
-      return;
-    }
+      if (!outputParameters || !outputParameters.length) {
+        return;
+      }
 
-    // (2) Store the old propAllChildVariables value and update it then
-    const bo = getBusinessObject(element),
-          calledElement = getCalledElement(bo);
+      const ioMapping = getIoMapping(element);
 
-    context.oldPropagateAllChildVariables = true;
+      modeling.updateModdleProperties(element, ioMapping, {
+        'zeebe:outputParameters': []
+      });
 
-    calledElement.propagateAllChildVariables = false;
-  }, true);
+      if (!inputParameters || !inputParameters.length) {
+        const businessObject = getBusinessObject(element),
+              extensionElements = businessObject.get('extensionElements');
 
-  // Revert behavior when adding outputParmaeters ////////////////////////////////////
-  this.reverted('properties-panel.update-businessobject-list' , HIGH_PRIORITY, function(context) {
-    const {
-      element,
-      oldPropagateAllChildVariables
-    } = context;
+        const values = extensionElements.get('values').filter((element) => {
+          return !is(element, 'zeebe:IoMapping');
+        });
 
-    // (1) Only intercept the revert, if the behavior became active
-    if (oldPropagateAllChildVariables) {
-      const bo = getBusinessObject(element),
-            calledElement = getCalledElement(bo);
+        modeling.updateModdleProperties(element, extensionElements, {
+          values
+        });
+      }
+    }, true);
+
+
+    /**
+     * Set zeebe:propagateAllChildVariables to false on zeebe:Output added.
+     */
+    this.postExecute('properties-panel.update-businessobject-list' , HIGH_PRIORITY, function(context) {
+      const {
+        currentObject,
+        element,
+        objectsToAdd,
+        propertyName
+      } = context;
+
+      if (!is(element, 'bpmn:CallActivity')
+        || !is(currentObject, 'zeebe:IoMapping')
+        || (propertyName !== 'outputParameters' && propertyName !== 'zeebe:outputParameters')
+        || !objectsToAdd
+        || !objectsToAdd.length
+        || !objectsToAdd.find((object) => is(object, 'zeebe:Output'))
+        || !isPropagateAllChildVariables(element)) {
+        return;
+      }
 
-      calledElement.propagateAllChildVariables = oldPropagateAllChildVariables;
-    }
-  }, true);
+      const businessObject = getBusinessObject(element),
+            calledElement = getCalledElement(businessObject);
 
+      modeling.updateModdleProperties(element, calledElement, {
+        'zeebe:propagateAllChildVariables': false
+      });
+    }, true);
 
+  }
 }
 
-
 UpdatePropagateAllChildVariablesBehavior.$inject = [
-  'eventBus'
-];
-
-
-inherits(UpdatePropagateAllChildVariablesBehavior, CommandInterceptor);
+  'eventBus',
+  'modeling'
+];

package/lib/camunda-cloud/features/modeling/behavior/index.js

@@ -1,3 +1,4 @@
+import CleanUpBusinessRuleTaskBehavior from './CleanUpBusinessRuleTaskBehavior';
 import CreateZeebeBoundaryEventBehavior from './CreateZeebeBoundaryEventBehavior';
 import CreateZeebeCallActivityBehavior from './CreateZeebeCallActivityBehavior';
 import UpdatePropagateAllChildVariablesBehavior from './UpdatePropagateAllChildVariablesBehavior';
@@ -6,11 +7,13 @@ import FormDefinitionBehavior from './FormDefinitionBehavior';
 
 export default {
   __init__: [
+    'cleanUpBusinessRuleTaskBehavior',
     'createZeebeBoundaryEventBehavior',
     'createZeebeCallActivityBehavior',
     'updatePropagateAllChildVariablesBehavior',
     'formDefinitionBehavior'
   ],
+  cleanUpBusinessRuleTaskBehavior: [ 'type', CleanUpBusinessRuleTaskBehavior ],
   createZeebeBoundaryEventBehavior: [ 'type', CreateZeebeBoundaryEventBehavior ],
   createZeebeCallActivityBehavior: [ 'type', CreateZeebeCallActivityBehavior ],
   updatePropagateAllChildVariablesBehavior: [ 'type', UpdatePropagateAllChildVariablesBehavior ],

package/lib/camunda-cloud/features/properties-provider/parts/implementation/InputOutput.js

@@ -15,11 +15,12 @@ import entryFieldDescription from 'bpmn-js-properties-panel/lib/factory/EntryFie
 import {
   areOutputParametersSupported,
   areInputParametersSupported,
-  createIOMapping,
+  createIoMapping,
   createElement as createParameter,
-  determineParamGetFunc,
-  getInputOutput,
-  isInputOutputSupported
+  getIoMapping,
+  isInputOutputSupported,
+  getInputParameters,
+  getOutputParameters
 } from '../../../../helper/InputOutputHelper';
 
 import InputOutputParameter from './InputOutputParameter';
@@ -130,12 +131,12 @@ function getParametersHeadingEntry(element, bpmnFactory, options) {
     }
 
     // Get the IOMapping
-    let inputOutput = getInputOutput(element);
+    let inputOutput = getIoMapping(element);
 
     if (!inputOutput) {
       const parent = extensionElements;
 
-      inputOutput = createIOMapping(parent, bpmnFactory, {
+      inputOutput = createIoMapping(parent, bpmnFactory, {
         inputParameters: [],
         outputParameters: []
       });
@@ -190,7 +191,7 @@ function getParametersHeadingEntry(element, bpmnFactory, options) {
 function getIOMappingEntries(element, bpmnFactory, translate, options) {
 
   // Get the IOMapping and determine whether we are dealing with input or output parameters
-  const inputOutput = getInputOutput(element, false),
+  const inputOutput = getIoMapping(element),
         params = determineParamGetFunc(options.prop)(element, false);
 
   if (!params.length) {
@@ -258,3 +259,16 @@ function getIOMappingEntries(element, bpmnFactory, translate, options) {
     });
   }
 }
+
+
+// helpers //////////
+
+function determineParamGetFunc(property) {
+  if (property == 'inputParameters') {
+    return getInputParameters;
+  }
+
+  if (property == 'outputParameters') {
+    return getOutputParameters;
+  }
+}

package/lib/camunda-cloud/features/rules/BpmnRules.js

@@ -22,7 +22,7 @@ import {
   getBoundaryAttachment as isBoundaryAttachment
 } from 'bpmn-js/lib/features/snapping/BpmnSnappingUtil';
 
-const HIGH_PRIORITY = 15000;
+const HIGH_PRIORITY = 5000;
 
 /**
  * Zeebe rule provider that allows to create boundary events with catch events

package/lib/camunda-cloud/helper/CalledElementHelper.js

@@ -1,70 +1,72 @@
-import {
-  has
-} from 'min-dash';
+import { has } from 'min-dash';
 
-import {
-  getOutputParameters
-} from './InputOutputHelper';
+import { getOutputParameters } from './InputOutputHelper';
 
-import {
-  getExtensionElements
-} from 'bpmn-js-properties-panel/lib/helper/ExtensionElementsHelper';
-
-import {
-  getBusinessObject
-} from 'bpmn-js/lib/util/ModelUtil';
+import { getExtensionElements } from 'bpmn-js-properties-panel/lib/helper/ExtensionElementsHelper';
 
 import {
+  getBusinessObject,
   is
 } from 'bpmn-js/lib/util/ModelUtil';
 
+
 /**
-  * Determine default value for propagateAllChildVariables attribute
-  * @param {Object} element representing a bpmn:CallActivity
-  *
-  * @returns {boolean}
-  */
-function determinePropAllChildVariablesDefault(element) {
+ * Get default value for zeebe:propagateAllChildVariables.
+ *
+ * @param {djs.model.Base|ModdleElement} element
+ *
+ * @returns {boolean}
+ */
+function getPropagateAllChildVariablesDefault(element) {
+  if (!is(element, 'bpmn:CallActivity')) {
+    return;
+  }
+
   const outputParameters = getOutputParameters(element);
 
   if (outputParameters) {
-    return (outputParameters.length > 0) ? false : true;
+    return !outputParameters.length;
   }
 }
 
 /**
-  * Get the 'zeebe:CalledElement' extension element for a given business Object
-  * @param {Object} bo businessObject
-  *
-  * @returns {Object} the calledElement Moddle Object or undefined if zeebe:CalledElement does not exist
-  */
+ * Get zeebe:CalledElement of an element.
+ *
+ * @param {djs.model.Base|ModdleElement} element
+ *
+ * @returns {ModdleElement}
+ */
 export function getCalledElement(element) {
   const calledElements = getCalledElements(element);
-  return calledElements[0];
+
+  return calledElements[ 0 ];
 }
 
 export function getCalledElements(element) {
-  const bo = getBusinessObject(element);
-  const extElements = getExtensionElements(bo, 'zeebe:CalledElement');
-  return extElements;
+  const businessObject = getBusinessObject(element);
+
+  return getExtensionElements(businessObject, 'zeebe:CalledElement');
 }
 
 /**
-  * Check whether the propagateAllChildVariables attribute is set on an element.
-  * Note that a default logic will be determine if it is not explicitly set.
-  * @param {Object} element
-  *
-  * @returns {boolean}
-  */
+ * Check whether zeebe:propagateAllChildVariables is set on an element.
+ * Fall back to default if zeebe:propagateAllChildVariables not set.
+ *
+ * @param {djs.model.Base|ModdleElement} element
+ *
+ * @returns {boolean}
+ */
 export function isPropagateAllChildVariables(element) {
   if (!is(element, 'bpmn:CallActivity')) {
-    return undefined;
+    return;
   }
 
-  const bo = getBusinessObject(element),
-        calledElement = getCalledElement(bo);
+  const businessObject = getBusinessObject(element),
+        calledElement = getCalledElement(businessObject);
 
-  return calledElement && has(calledElement, 'propagateAllChildVariables') ?
-    calledElement.get('propagateAllChildVariables') :
-    determinePropAllChildVariablesDefault(element);
+  if (calledElement && has(calledElement, 'propagateAllChildVariables')) {
+    return calledElement.get('propagateAllChildVariables');
+  } else {
+    return getPropagateAllChildVariablesDefault(element);
+  }
 }

package/lib/camunda-cloud/helper/FormsHelper.js

@@ -1,6 +1,4 @@
-import {
-  getExtensionElements
-} from 'bpmn-js-properties-panel/lib/helper/ExtensionElementsHelper';
+import { getExtensionElements } from 'bpmn-js-properties-panel/lib/helper/ExtensionElementsHelper';
 
 import elementHelper from 'bpmn-js-properties-panel/lib/helper/ElementHelper';
 
@@ -9,47 +7,12 @@ import {
   is
 } from 'bpmn-js/lib/util/ModelUtil';
 
-import {
-  nextId
-} from 'bpmn-js-properties-panel/lib/Utils';
-
-import {
-  find
-} from 'min-dash';
-
-const USER_TASK_FORM_PREFIX = 'userTaskForm_';
-
-
-export function getUserTaskForm(element, parent) {
+import { nextId } from 'bpmn-js-properties-panel/lib/Utils';
 
-  const rootElement = parent || getRootElement(element);
-
-  // (1) get form definition from user task
-  const formDefinition = getFormDefinition(element);
-
-  if (!formDefinition) {
-    return;
-  }
+import { find } from 'min-dash';
 
-  const formKey = formDefinition.get('formKey');
+const USER_TASK_FORM_PREFIX = 'UserTaskForm_';
 
-  // (2) retrieve user task form via form key
-  const userTaskForm = findUserTaskForm(formKey, rootElement);
-
-  return userTaskForm;
-}
-
-export function getFormDefinition(element) {
-  const businessObject = getBusinessObject(element);
-
-  const formDefinitions = getExtensionElements(businessObject, 'zeebe:FormDefinition');
-
-  return formDefinitions[0];
-}
-
-export function createFormKey(formId) {
-  return 'camunda-forms:bpmn:' + formId;
-}
 
 export function createFormDefinition(properties, extensionElements, bpmnFactory) {
   return elementHelper.createElement(
@@ -60,6 +23,14 @@ export function createFormDefinition(properties, extensionElements, bpmnFactory)
   );
 }
 
+export function createFormId() {
+  return nextId(USER_TASK_FORM_PREFIX);
+}
+
+export function createFormKey(formId) {
+  return `camunda-forms:bpmn:${ formId }`;
+}
+
 export function createUserTaskForm(properties, extensionElements, bpmnFactory) {
   return elementHelper.createElement(
     'zeebe:UserTaskForm',
@@ -69,19 +40,22 @@ export function createUserTaskForm(properties, extensionElements, bpmnFactory) {
   );
 }
 
-export function createFormId() {
-  return nextId(USER_TASK_FORM_PREFIX);
-}
+function findUserTaskForm(formKey, rootElement) {
+  const userTaskForms = getExtensionElements(rootElement, 'zeebe:UserTaskForm');
 
+  return find(userTaskForms, function(userTaskForm) {
+    const id = userTaskForm.get('zeebe:id');
 
-// helpers /////////////////////
+    return createFormKey(id) === formKey;
+  });
+}
 
-function findUserTaskForm(formKey, rootElement) {
-  const forms = getExtensionElements(rootElement, 'zeebe:UserTaskForm');
+export function getFormDefinition(element) {
+  const businessObject = getBusinessObject(element);
 
-  return find(forms, function(userTaskForm) {
-    return createFormKey(userTaskForm.id) === formKey;
-  });
+  const formDefinitions = getExtensionElements(businessObject, 'zeebe:FormDefinition');
+
+  return formDefinitions[ 0 ];
 }
 
 function getRootElement(element) {
@@ -93,4 +67,18 @@ function getRootElement(element) {
   }
 
   return parent;
+}
+
+export function getUserTaskForm(element, parent) {
+  const rootElement = parent || getRootElement(element);
+
+  const formDefinition = getFormDefinition(element);
+
+  if (!formDefinition) {
+    return;
+  }
+
+  const formKey = formDefinition.get('zeebe:formKey');
+
+  return findUserTaskForm(formKey, rootElement);
 }