chore(moddle-copy): convert to ES6

Author: philippfromme

lib/features/copy-paste/ModdleCopy.js

@@ -12,7 +12,7 @@ import {
 
 import { is } from '../../util/ModelUtil';
 
-var DISALLOWED_PROPERTIES = [
+const DISALLOWED_PROPERTIES = [
   'artifacts',
   'dataInputAssociations',
   'dataOutputAssociations',
@@ -24,7 +24,7 @@ var DISALLOWED_PROPERTIES = [
   'categoryValue'
 ];
 
-var ALLOWED_REFERENCES = [
+const ALLOWED_REFERENCES = [
   'errorRef',
   'escalationRef',
   'messageRef',
@@ -52,24 +52,27 @@ export default function ModdleCopy(eventBus, bpmnFactory, moddle) {
   this._moddle = moddle;
 
   // copy extension elements last
-  eventBus.on('moddleCopy.canCopyProperties', function(context) {
-    var propertyNames = context.propertyNames;
+  eventBus.on('moddleCopy.canCopyProperties', (context) => {
+    const { propertyNames } = context;
 
     if (!propertyNames || !propertyNames.length) {
       return;
     }
 
-    return sortBy(propertyNames, function(propertyName) {
+    return sortBy(propertyNames, (propertyName) => {
       return propertyName === 'extensionElements';
     });
   });
 
   // default check whether property can be copied
-  eventBus.on('moddleCopy.canCopyProperty', function(context) {
-    var parent = context.parent,
-        parentDescriptor = isObject(parent) && parent.$descriptor,
-        propertyName = context.propertyName,
-        property = context.property;
+  eventBus.on('moddleCopy.canCopyProperty', (context) => {
+    const {
+      parent,
+      property,
+      propertyName
+    } = context;
+
+    const parentDescriptor = isObject(parent) && parent.$descriptor;
 
     if (propertyName && ALLOWED_REFERENCES.includes(propertyName)) {
 
@@ -93,8 +96,8 @@ export default function ModdleCopy(eventBus, bpmnFactory, moddle) {
   });
 
   // do NOT allow to copy empty extension elements
-  eventBus.on('moddleCopy.canSetCopiedProperty', function(context) {
-    var property = context.property;
+  eventBus.on('moddleCopy.canSetCopiedProperty', (context) => {
+    const { property } = context;
 
     if (is(property, 'bpmn:ExtensionElements') && (!property.values || !property.values.length)) {
 
@@ -121,15 +124,13 @@ ModdleCopy.$inject = [
  * @return {ModdleElement}
  */
 ModdleCopy.prototype.copyElement = function(sourceElement, targetElement, propertyNames, clone = false) {
-  var self = this;
-
   if (propertyNames && !isArray(propertyNames)) {
     propertyNames = [ propertyNames ];
   }
 
   propertyNames = propertyNames || getPropertyNames(sourceElement.$descriptor);
 
-  var canCopyProperties = this._eventBus.fire('moddleCopy.canCopyProperties', {
+  const canCopyProperties = this._eventBus.fire('moddleCopy.canCopyProperties', {
     propertyNames: propertyNames,
     sourceElement: sourceElement,
     targetElement: targetElement,
@@ -145,20 +146,20 @@ ModdleCopy.prototype.copyElement = function(sourceElement, targetElement, proper
   }
 
   // copy properties
-  forEach(propertyNames, function(propertyName) {
-    var sourceProperty;
+  forEach(propertyNames, (propertyName) => {
+    let sourceProperty;
 
     if (has(sourceElement, propertyName)) {
       sourceProperty = sourceElement.get(propertyName);
     }
 
-    var copiedProperty = self.copyProperty(sourceProperty, targetElement, propertyName, clone);
+    const copiedProperty = this.copyProperty(sourceProperty, targetElement, propertyName, clone);
 
     if (!isDefined(copiedProperty)) {
       return;
     }
 
-    var canSetProperty = self._eventBus.fire('moddleCopy.canSetCopiedProperty', {
+    const canSetProperty = this._eventBus.fire('moddleCopy.canSetCopiedProperty', {
       parent: targetElement,
       property: copiedProperty,
       propertyName: propertyName
@@ -187,10 +188,9 @@ ModdleCopy.prototype.copyElement = function(sourceElement, targetElement, proper
  * @return {any}
  */
 ModdleCopy.prototype.copyProperty = function(property, parent, propertyName, clone = false) {
-  var self = this;
 
   // allow others to copy property
-  var copiedProperty = this._eventBus.fire('moddleCopy.canCopyProperty', {
+  let copiedProperty = this._eventBus.fire('moddleCopy.canCopyProperty', {
     parent: parent,
     property: property,
     propertyName: propertyName,
@@ -210,7 +210,7 @@ ModdleCopy.prototype.copyProperty = function(property, parent, propertyName, clo
     return copiedProperty;
   }
 
-  var propertyDescriptor = this._moddle.getPropertyDescriptor(parent, propertyName);
+  const propertyDescriptor = this._moddle.getPropertyDescriptor(parent, propertyName);
 
   // do NOT copy references
   if (propertyDescriptor.isReference) {
@@ -224,10 +224,10 @@ ModdleCopy.prototype.copyProperty = function(property, parent, propertyName, clo
 
   // copy arrays
   if (isArray(property)) {
-    return reduce(property, function(childProperties, childProperty) {
+    return reduce(property, (childProperties, childProperty) => {
 
       // recursion
-      copiedProperty = self.copyProperty(childProperty, parent, propertyName, clone);
+      const copiedProperty = this.copyProperty(childProperty, parent, propertyName, clone);
 
       // copying might NOT be allowed
       if (copiedProperty) {
@@ -244,12 +244,12 @@ ModdleCopy.prototype.copyProperty = function(property, parent, propertyName, clo
       return;
     }
 
-    copiedProperty = self._bpmnFactory.create(property.$type);
+    copiedProperty = this._bpmnFactory.create(property.$type);
 
     copiedProperty.$parent = parent;
 
     // recursion
-    copiedProperty = self.copyElement(property, copiedProperty, null, clone);
+    copiedProperty = this.copyElement(property, copiedProperty, null, clone);
 
     return copiedProperty;
   }
@@ -276,7 +276,7 @@ ModdleCopy.prototype._copyId = function(id, element, clone = false) {
 // helpers //////////
 
 export function getPropertyNames(descriptor, keepDefaultProperties) {
-  return reduce(descriptor.properties, function(properties, property) {
+  return reduce(descriptor.properties, (properties, property) => {
 
     if (keepDefaultProperties && property.default) {
       return properties;