@processmaker/modeler 1.40.0 → 1.40.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@processmaker/modeler",
-  "version": "1.40.0",
+  "version": "1.40.2",
   "scripts": {
     "serve": "vue-cli-service serve --mode development",
     "test:unit": "vue-cli-service test:unit",

package/src/components/crown/crownButtons/crownColorDropdown.vue

@@ -142,10 +142,10 @@ export default {
         id: this.node.definition.id,
         properties: {
           color: this.node.definition.color,
-          x: this.node.diagram.bounds.x,
-          y: this.node.diagram.bounds.y,
-          height: this.node.diagram.bounds.height,
-          width: this.node.diagram.bounds.width,
+          x: this.node.diagram.bounds?.x,
+          y: this.node.diagram.bounds?.y,
+          height: this.node.diagram.bounds?.height,
+          width: this.node.diagram.bounds?.width,
           type: this.node.type,
           id: this.node.definition.id,
           attachedToRefId: this.node.definition.get('attachedToRef')?.id,

package/src/components/inspectors/PreviewPanel.vue

@@ -56,7 +56,7 @@
 
       <iframe
         title="Preview"
-        v-show="!!previewUrl && !showSpinner"
+        :style="{visibility: displayPreviewIframe}"
         :src="previewUrl"
         class="preview-iframe"
         @load="loading"
@@ -95,20 +95,26 @@ export default {
       }
     },
     highlightedNode() {
+      // If there isn't preview configuration hide the panel
+      const nodeConfig = this.getConfig(this.data);
+      if (!nodeConfig) {
+        this.$emit('togglePreview', false);
+        return;
+      }
+
       document.activeElement.blur();
-      this.prepareData();
+      this.previewNode();
     },
-    'highlightedNode.definition'(current, previous) { this.handleAssignmentChanges(current, previous); },
-    'highlightedNode.definition.assignmentLock'(current, previous) { this.handleAssignmentChanges(current, previous); },
-    'highlightedNode.definition.allowReassignment'(current, previous) { this.handleAssignmentChanges(current, previous); },
-    'highlightedNode.definition.assignedUsers'(current, previous) { this.handleAssignmentChanges(current, previous); },
-    'highlightedNode.definition.assignedGroups'(current, previous) { this.handleAssignmentChanges(current, previous); },
-    'highlightedNode.definition.assignmentRules'(current, previous) { this.handleAssignmentChanges(current, previous); },
   },
   computed: {
     highlightedNode() {
       return store.getters.highlightedNodes[0];
     },
+    displayPreviewIframe() {
+      const result = !!this.previewUrl && !this.showSpinner;
+      console.log('mostrar iframe', result);
+      return result ? 'visible' : 'hidden';
+    },
   },
   methods: {
     loading() {
@@ -137,21 +143,6 @@ export default {
       this.taskTitle = this.data?.name;
 
       this.taskTitle = this?.highlightedNode?.definition?.name;
-      this.previewNode();
-    },
-
-    handleAssignmentChanges(currentValue, previousValue) {
-      if (currentValue === previousValue) {
-        return;
-      }
-
-      const nodeConfig = this.getConfig(this.data);
-
-      if (nodeConfig) {
-        this.prepareData();
-      } else {
-        this.$emit('togglePreview', false);
-      }
     },
     onSelectedPreview(item) {
       this.selectedPreview = item;
@@ -160,10 +151,8 @@ export default {
       if (!this.highlightedNode || (!this.visible && !force)) {
         return;
       }
-
-      const previewConfig = this.previewConfigs.find(config => {
-        return config.matcher(this.data);
-      });
+      this.prepareData();
+      const previewConfig = this.getConfig(this.data);
 
       let clone = {};
       for (let prop in this.data) {
@@ -176,7 +165,11 @@ export default {
 
       // if the node has the configurations (for example screenRef for a task in a task)
       const nodeHasConfigParams = Object.keys(clone).length > 0;
-      this.previewUrl = previewConfig &&  nodeHasConfigParams ? `${previewConfig.url}?node=${nodeData}` : null;
+      const nodeHasConfiguredAssets = !!previewConfig?.assetUrl(this.data);
+
+      this.previewUrl = nodeHasConfiguredAssets && previewConfig &&  nodeHasConfigParams
+        ? `${previewConfig.url}?node=${nodeData}`
+        : null;
       this.taskTitle = this.highlightedNode?.definition?.name;
     },
     onClose() {

package/src/components/inspectors/preview_panel.scss

@@ -41,15 +41,16 @@ $inspector-column-min-width: 200px;
   width: 100%;
 }
 
-.preview-column .control-bar .actions div {
+.preview-column .control-bar .actions {
   width: 100%;
-  padding-right: 15px;
+  padding-right: 45px;
   text-align: right;
 }
 
-.preview-column .control-bar .actions div i {
+.preview-column .control-bar .actions i {
   width: 20px;
   cursor: pointer;
+  margin-left: 5px;
 }
 
 .task-title {

package/src/components/modeler/Modeler.vue

@@ -69,7 +69,7 @@
         @previewResize="[onMouseMove($event), setInspectorButtonPosition($event)]"
         @startResize="onStartPreviewResize"
         @stopResize="onMouseUp"
-        :visible="isOpenPreview"
+        :visible="isOpenPreview && !(highlightedNodes.length > 1)"
         :nodeRegistry="nodeRegistry"
         :previewConfigs="previewConfigs"
         :panelWidth="previewPanelWidth"
@@ -139,6 +139,7 @@
         @clone-selection="cloneSelection"
         @default-flow="toggleDefaultFlow"
         @shape-resize="shapeResize"
+        @definition-changed="onNodeDefinitionChanged"
       />
 
       <RailBottom
@@ -420,6 +421,12 @@ export default {
     isMultiplayer: () => store.getters.isMultiplayer,
   },
   methods: {
+    onNodeDefinitionChanged() {
+      // re-render the preview just if the preview pane is open
+      if (this.isOpenPreview) {
+        this.handlePreview();
+      }
+    },
     onStartPreviewResize(event) {
       this.isResizingPreview = true;
       this.currentCursorPosition = event.x;
@@ -1106,6 +1113,11 @@ export default {
       diagram.bounds.x = data.x;
       diagram.bounds.y = data.y;
       const newNode = this.createNode(data.type, definition, diagram);
+      //verify if the node has a pool as a container
+      if (data.poolId) {
+        const pool = this.getElementByNodeId(data.poolId);
+        this.poolTarget = pool;
+      }
       await this.addNode(newNode, data.id, true);
       await this.$nextTick();
       await this.paperManager.awaitScheduledUpdates();
@@ -1172,11 +1184,10 @@ export default {
         'processmaker-modeler-sequence-flow',
         'processmaker-modeler-association',
         'processmaker-modeler-data-input-association',
-        'processmaker-modeler-data-input-association',
         'processmaker-modeler-boundary-timer-event',
         'processmaker-modeler-boundary-error-event',
         'processmaker-modeler-boundary-signal-event',
-        'processmaker-modeler-boundary-conditional-even',
+        'processmaker-modeler-boundary-conditional-event',
         'processmaker-modeler-boundary-message-event',
       ];
       if (!this.isMultiplayer) {
@@ -1202,6 +1213,7 @@ export default {
             loopCharacteristics: null,
             gatewayDirection: null,
             messageRef: null,
+            signalRef: null,
             extras: {},
           };
           if (node?.pool?.component) {
@@ -1223,12 +1235,14 @@ export default {
             targetRefId = node.definition.targetRef?.$parent?.$parent?.get('id');
           }
           const waypoint = [];
-          node.diagram.waypoint.forEach(point => {
+
+          node.diagram.waypoint?.forEach(point => {
             waypoint.push({
               x: point.x,
               y: point.y,
             });
           });
+
           if (sourceRefId && targetRefId) {
             const flowData = {
               id: node.definition.id,
@@ -1238,6 +1252,7 @@ export default {
               waypoint,
               name: node.definition.name,
               conditionExpression: null,
+              color: null,
             };
 
             if (isProcessRequested) {
@@ -1625,6 +1640,12 @@ export default {
         this.players.splice(playerIndex, 1);
       }
     },
+    /**
+     * Update the lasso tool
+     */
+    updateLasso(){
+      this.$refs.selector.updateSelectionBox();
+    },
   },
   created() {
     if (runningInCypressTest()) {

package/src/components/modeler/Selection.vue

@@ -598,8 +598,10 @@ export default {
       shapes.filter(shape => !shapesToNotTranslate.includes(shape.model.get('type')))
         .forEach(shape => {
           if (shape.model.get('type') === 'processmaker.modeler.bpmn.pool') {
-            const children = shape.model.component.getElementsUnderArea(shape.model, this.graph);
-            changed = [...changed, ...this.getContainerProperties(children, changed)];
+            const childrens = shape.model.component.getElementsUnderArea(shape.model, this.graph)
+              .filter((element) => element.component);
+
+            changed = [...changed, ...this.getContainerProperties(childrens, changed)];
           } else {
             const { node } = shape.model.component;
             const defaultData = {
@@ -655,9 +657,9 @@ export default {
       });
       return boundariesChanged;
     },
-    getContainerProperties(children) {
+    getContainerProperties(childrens) {
       const changed = [];
-      children.forEach(model => {
+      childrens.forEach(model => {
         changed.push({
           id: model.component.node.definition.id,
           properties: {

package/src/components/nodes/boundaryEvent/boundaryEvent.vue

@@ -27,6 +27,7 @@ import CrownConfig from '@/components/crown/crownConfig/crownConfig';
 import highlightConfig from '@/mixins/highlightConfig';
 import store from '@/store';
 import { canAddBoundaryEventToTarget } from '@/boundaryEventValidation';
+import { getBoundaryEventData } from '@/components/nodes/boundaryEvent/boundaryEventUtils';
 
 export default {
   components: {
@@ -203,24 +204,9 @@ export default {
       this.invalidTargetElement = targetElement;
     },
     addMultiplayerBoundaryEvent() {
-      const control = {
-        bpmnType: this.node.diagram.$type,
-        id: this.node.diagram.id,
-        type: this.node.type,
-        attachedToRef: this.node.definition.get('attachedToRef'),
-      };
+      const defaultData = getBoundaryEventData(this.node);
 
-      window.ProcessMaker.EventBus.$emit('multiplayer-addBoundaryEvent', {
-        x: this.node.diagram.bounds.x,
-        y: this.node.diagram.bounds.y,
-        height: this.node.diagram.bounds.height,
-        width: this.node.diagram.bounds.width,
-        attachedToRefId: this.node.definition.get('attachedToRef')?.id,
-        control,
-        type: this.node.type,
-        id: this.node.definition.id,
-        color: this.node.definition.get('color'),
-      });
+      window.ProcessMaker.EventBus.$emit('multiplayer-addBoundaryEvent', defaultData);
     },
   },
   async mounted() {
@@ -234,9 +220,9 @@ export default {
     const task = this.getTaskUnderShape();
     this.attachBoundaryEventToTask(task);
     this.updateShapePosition(task);
-    
+
     if (this.node.fromCrown) {
-      this.addMultiplayerBoundaryEvent();  
+      this.addMultiplayerBoundaryEvent();
     }
   },
 };

package/src/components/nodes/boundaryEvent/boundaryEventUtils.js

@@ -0,0 +1,35 @@
+export const getBoundaryEventData = (node) => {
+  const control = {
+    bpmnType: node.diagram.$type,
+    id: node.diagram.id,
+    type: node.type,
+    attachedToRef: node.definition.get('attachedToRef'),
+  };
+
+  const eventDefinition = node.definition.get('eventDefinitions')[0];
+  const eventDefinitionType = Object.keys(eventDefinition).reduce((acc, key) => {
+    if (key.startsWith('time')) {
+      acc[key] = eventDefinition[key];
+    }
+    return acc;
+  }, {});
+
+  const type = Object.keys(eventDefinitionType)[0];
+
+  return {
+    x: node.diagram.bounds.x,
+    y: node.diagram.bounds.y,
+    height: node.diagram.bounds.height,
+    width: node.diagram.bounds.width,
+    attachedToRefId: node.definition.get('attachedToRef')?.id,
+    control,
+    type: node.type,
+    id: node.definition.id,
+    color: node.definition.get('color'),
+    cancelActivity: node.definition.get('cancelActivity'),
+    eventTimerDefinition: {
+      type,
+      body: eventDefinitionType[type]?.body,
+    },
+  };
+};

package/src/components/nodes/boundaryTimerEvent/index.js

@@ -10,6 +10,20 @@ import { defaultDurationTimerEvent } from '@/constants';
 
 export const id = 'processmaker-modeler-boundary-timer-event';
 
+export const setEventTimerDefinition = (moddle, node, type, body) => {
+  const eventDefinition = {
+    [type]: moddle.create('bpmn:Expression', { body }),
+  };
+
+  const eventDefinitions = [
+    moddle.create('bpmn:TimerEventDefinition', eventDefinition),
+  ];
+
+  eventDefinitions[0].id = node.definition.get('eventDefinitions')[0].id;
+
+  return eventDefinitions;
+};
+
 export default merge(cloneDeep(boundaryEventConfig), {
   id,
   component,
@@ -58,17 +72,18 @@ export default merge(cloneDeep(boundaryEventConfig), {
           continue;
         }
 
-        const eventDefinition = {
-          [type]: moddle.create('bpmn:Expression', { body }),
-        };
-
-        const eventDefinitions = [
-          moddle.create('bpmn:TimerEventDefinition', eventDefinition),
-        ];
-
-        eventDefinitions[0].id = node.definition.get('eventDefinitions')[0].id;
+        const eventDefinitions = setEventTimerDefinition(moddle, node, type, body);
 
         setNodeProp(node, 'eventDefinitions', eventDefinitions);
+
+        window.ProcessMaker.EventBus.$emit('multiplayer-updateInspectorProperty', {
+          id: node.definition.id,
+          key: 'eventTimerDefinition',
+          value: {
+            type,
+            body,
+          },
+        });
       } else {
         window.ProcessMaker.EventBus.$emit('multiplayer-updateInspectorProperty', {
           id: node.definition.id , key, value: value[key],

package/src/components/nodes/signalEventDefinition/index.js

@@ -37,6 +37,7 @@ export default {
     }
 
     let signal = definitions.rootElements.find(element => element.id === value.signalRef);
+
     if (!signal && value.signalRef) {
       signal = moddle.create('bpmn:Signal', {
         id: value.signalRef,
@@ -45,5 +46,14 @@ export default {
       definitions.rootElements.push(signal);
     }
     node.definition.get('eventDefinitions')[0].signalRef = signal;
+
+    window.ProcessMaker.EventBus.$emit('multiplayer-updateInspectorProperty', {
+      id: node.definition.id,
+      key: 'signalRef',
+      value: value.signalRef,
+      extras: {
+        signalName: signal?.name,
+      },
+    });
   },
 };

package/src/components/nodes/subProcess/index.js

@@ -5,6 +5,7 @@ import { taskHeight, taskWidth } from '@/components/nodes/task/taskConfig';
 import advancedAccordionConfig from '@/components/inspectors/advancedAccordionConfig';
 import documentationAccordionConfig from '@/components/inspectors/documentationAccordionConfig';
 import defaultNames from '@/components/nodes/task/defaultNames';
+import { loopCharacteristicsHandler } from '@/components/inspectors/LoopCharacteristics';
 
 export const id = 'processmaker-modeler-call-activity';
 
@@ -86,4 +87,5 @@ export default {
       ],
     },
   ],
+  loopCharacteristicsHandler,
 };

package/src/components/nodes/subProcess/subProcess.vue

@@ -41,6 +41,7 @@
 </template>
 
 <script>
+import debounce from 'lodash/debounce';
 import { util } from 'jointjs';
 import portsConfig from '@/mixins/portsConfig';
 import TaskShape from '@/components/nodes/task/shape';
@@ -128,7 +129,7 @@ export default {
     },
   },
   watch: {
-    'node.definition.name'(name) {
+    'node.definition.name': debounce(function(name) {
       const { width } = this.node.diagram.bounds;
       this.shape.attr('label/text', util.breakText(name, { width }));
 
@@ -142,7 +143,7 @@ export default {
         this.shape.resize(width, newHeight);
         this.recalcMarkersAlignment();
       }
-    },
+    }, 300),
     'node.definition.config'(config) {
       store.commit('updateNodeProp', {
         node: this.node,

package/src/components/nodes/task/task.vue

@@ -23,6 +23,7 @@
 </template>
 
 <script>
+import debounce from 'lodash/debounce';
 import { util } from 'jointjs';
 import highlightConfig from '@/mixins/highlightConfig';
 import portsConfig from '@/mixins/portsConfig';
@@ -97,7 +98,7 @@ export default {
     },
   },
   watch: {
-    'node.definition.name'(name) {
+    'node.definition.name': debounce(function(name) {
       const { width } = this.node.diagram.bounds;
       this.shape.attr('label/text', util.breakText(name, { width }));
       const { height } = this.shape.size();
@@ -109,7 +110,7 @@ export default {
         this.shape.resize(width, newHeight);
         this.recalcMarkersAlignment();
       }
-    },
+    }, 300),
     'node.definition.isForCompensation'() {
       setupCompensationMarker(this.node.definition, this.markers, this.$set, this.$delete);
     },
@@ -117,6 +118,7 @@ export default {
       deep: true,
       handler() {
         setupLoopCharacteristicsMarkers(this.node.definition, this.markers, this.$set, this.$delete);
+        this.$emit('definition-changed', this.node.definition);
       },
     },
   },

package/src/components/topRail/multiplayerViewUsers/MultiplayerViewUsers.vue

@@ -1,5 +1,5 @@
 <template>
-  <b-avatar-group class="container" v-if="!isMultiplayer">
+  <b-avatar-group class="container" v-show="isMultiplayer">
     <template v-for="item in filteredPlayers" >
       <Avatar :badgeBackgroundColor="item.color" :imgSrc="item.avatar" :userName="item.name" :key="item.key"/>
     </template>

package/src/mixins/cloneSelection.js

@@ -47,6 +47,7 @@ export default {
             if (node.definition && node.definition.$type !== 'bpmn:BoundaryEvent') {
               clonedNodes.push(clonedElement);
             }  else {
+              clonedElement.fromCrown = true;
               clonedBoundaryEvents.push(clonedElement);
             }
           }
@@ -128,7 +129,7 @@ export default {
         const originalTargetElement = this.getDataInputOutputAssociationTargetRef(originalAssociation.definition);
         const clonedElement = clonedNodes.find(node => node.cloneOf === originalTargetElement.id);
         const clonedDataInput = getOrFindDataInput(this.moddle, clonedElement, srcClone.definition);
-
+        clonedDataInput.$parent.$parent = clonedElement.definition;
         clonedAssociation.definition.set('sourceRef', [srcClone.definition]);
         clonedAssociation.definition.set('targetRef', clonedDataInput);
         clonedElement.definition.set('dataInputAssociations', [clonedAssociation.definition]);

package/src/multiplayer/multiplayer.js

@@ -4,6 +4,8 @@ import { getNodeIdGenerator } from '../NodeIdGenerator';
 import { getDefaultAnchorPoint } from '@/portsUtils';
 import Room from './room';
 import store from '@/store';
+import { setEventTimerDefinition } from '@/components/nodes/boundaryTimerEvent';
+import { getBoundaryEventData } from '@/components/nodes/boundaryEvent/boundaryEventUtils';
 
 export default class Multiplayer {
   clientIO = null;
@@ -199,7 +201,13 @@ export default class Multiplayer {
    */
   syncLocalNodes(clientId){
     // Get the process definition
-    const nodes = this.modeler.nodes.map((node) => this.modeler.multiplayerHook(node, false, true));
+    const nodes = this.modeler.nodes.map((node) => {
+      if (node.definition.$type === 'bpmn:BoundaryEvent') {
+        return getBoundaryEventData(node);
+      }
+
+      return this.modeler.multiplayerHook(node, false, true);
+    });
 
     nodes.forEach((node) => {
       const yMapNested = new Y.Map();
@@ -227,7 +235,7 @@ export default class Multiplayer {
     if (node) {
       return;
     }
-    if (this.modeler.nodeRegistry[value.type] && this.modeler.nodeRegistry[value.type].multiplayerClient) {
+    if (this.modeler.nodeRegistry[value.type]?.multiplayerClient) {
       this.modeler.nodeRegistry[value.type].multiplayerClient(this.modeler, value);
     } else {
       this.modeler.addRemoteNode(value);
@@ -245,7 +253,6 @@ export default class Multiplayer {
   removeNode(data) {
     const index =  this.getIndex(data.definition.id);
     if (index >= 0) {
-      this.removeShape(data);
       this.yArray.delete(index, 1); // delete one element
       // Encode the state as an update and send it to the server
       const stateUpdate = Y.encodeStateAsUpdate(this.yDoc);
@@ -352,34 +359,40 @@ export default class Multiplayer {
     const newPool = this.modeler.getElementByNodeId(data.poolId);
 
     if (this.modeler.flowTypes.includes(data.type)) {
-      // Update the element's waypoints
-      // Get the source and target elements
-      const sourceElem = this.modeler.getElementByNodeId(data.sourceRefId);
-      const targetElem = this.modeler.getElementByNodeId(data.targetRefId);
-
       const { waypoint } = data;
-      const startWaypoint = waypoint.shift();
-      const endWaypoint = waypoint.pop();
-
-      // Update the element's waypoints
-      const newWaypoint = waypoint.map(point => this.modeler.moddle.create('dc:Point', point));
-      element.set('vertices', newWaypoint);
-
-      // Update the element's source anchor
-      element.source(sourceElem, {
-        anchor: () => {
-          return getDefaultAnchorPoint(this.getConnectionPoint(sourceElem, startWaypoint), sourceElem.findView(paper));
-        },
-        connectionPoint: { name: 'boundary' },
-      });
 
-      // Update the element's target anchor
-      element.target(targetElem, {
-        anchor: () => {
-          return getDefaultAnchorPoint(this.getConnectionPoint(targetElem, endWaypoint), targetElem.findView(paper));
-        },
-        connectionPoint: { name: 'boundary' },
-      });
+      if (waypoint) {
+        // Update the element's waypoints
+        // Get the source and target elements
+        const sourceElem = this.modeler.getElementByNodeId(data.sourceRefId);
+        const targetElem = this.modeler.getElementByNodeId(data.targetRefId);
+
+        const startWaypoint = waypoint.shift();
+        const endWaypoint = waypoint.pop();
+
+        // Update the element's waypoints
+        const newWaypoint = waypoint.map(point => this.modeler.moddle.create('dc:Point', point));
+        element.set('vertices', newWaypoint);
+
+        // Update the element's source anchor
+        element.source(sourceElem, {
+          anchor: () => {
+            return getDefaultAnchorPoint(this.getConnectionPoint(sourceElem, startWaypoint), sourceElem.findView(paper));
+          },
+          connectionPoint: { name: 'boundary' },
+        });
+
+        // Update the element's target anchor
+        element.target(targetElem, {
+          anchor: () => {
+            return getDefaultAnchorPoint(this.getConnectionPoint(targetElem, endWaypoint), targetElem.findView(paper));
+          },
+          connectionPoint: { name: 'boundary' },
+        });
+      } else {
+        const node = this.getNodeById(data.id);
+        store.commit('updateNodeProp', { node, key: 'color', value: data.color });
+      }
     } else {
       // Update the element's position attribute
       element.resize(
@@ -405,6 +418,7 @@ export default class Multiplayer {
       if (newPool && element.component.node.pool && element.component.node.pool.component.id !== data.poolId) {
         element.component.node.pool.component.moveElementRemote(element, newPool);
       }
+      this.modeler.updateLasso();
     }
   }
   attachBoundaryEventToNode(element, data) {
@@ -553,8 +567,8 @@ export default class Multiplayer {
         return;
       }
       const keys = Object.keys(data).filter((key) => key !== 'id');
-      const key = keys[0];
-      const value = data[key];
+      let key = keys[0];
+      let value = data[key];
 
       if (key === 'condition') {
         node.definition.get('eventDefinitions')[0].get('condition').body = value;
@@ -586,7 +600,34 @@ export default class Multiplayer {
         }
       }
 
-      if (!['messageRef', 'gatewayDirection', 'condition', 'allowedUsers', 'allowedGroups'].includes(key)) {
+      if (key === 'signalRef') {
+        let signal = this.modeler.definitions.rootElements.find(element => element.id === value);
+
+        if (!signal) {
+          signal = this.modeler.moddle.create('bpmn:Signal', {
+            id: value,
+            name: extras?.signalName || value,
+          });
+          this.modeler.definitions.rootElements.push(signal);
+        }
+
+        node.definition.get('eventDefinitions')[0].signalRef = signal;
+      }
+
+      if (key === 'eventTimerDefinition') {
+        const { type, body } = value;
+
+        const eventDefinitions = setEventTimerDefinition(this.modeler.moddle, node, type, body);
+
+        key = 'eventDefinitions';
+        value = eventDefinitions;
+      }
+
+      const specialProperties = [
+        'messageRef', 'signalRef', 'gatewayDirection', 'condition', 'allowedUsers', 'allowedGroups',
+      ];
+
+      if (!specialProperties.includes(key)) {
         store.commit('updateNodeProp', { node, key, value });
       }
     }