refactor(foundation): make reference within editor-action optional (#325)

* refactor(foundation): change EditorAction api make reference optional

* refactor(Editor): get reference onAction with undefined reference

* test(Editing): add integration test

* fix(lnodetype-wizard): fix getReference with addig LNodeType from template

* refactor(templates): remove reference from create actions

* refactor(communication): remove reference from create actions

* refactor(substation): remove reference from create action

* refactor(templates): remove reference from create action

* refactor(communication mapping): remove reference from create action

* refactor(iedimport): remove reference from create action

Author: JakobVogelsang

src/Editing.ts

@@ -6,6 +6,7 @@ import {
   Delete,
   EditorAction,
   EditorActionEvent,
+  getReference,
   isCreate,
   isDelete,
   isMove,
@@ -17,6 +18,7 @@ import {
   newLogEvent,
   newValidateEvent,
   OpenDocEvent,
+  SCLTag,
   SimpleAction,
   Update,
 } from './foundation.js';
@@ -72,6 +74,12 @@ export function Editing<TBase extends LitElementConstructor>(Base: TBase) {
     private onCreate(action: Create) {
       if (!this.checkCreateValidity(action)) return false;
 
+      if (action.new.reference === undefined)
+        action.new.reference = getReference(
+          action.new.parent,
+          <SCLTag>action.new.element.tagName
+        );
+
       action.new.parent.insertBefore(action.new.element, action.new.reference);
       return true;
     }
@@ -89,6 +97,9 @@ export function Editing<TBase extends LitElementConstructor>(Base: TBase) {
     }
 
     private onDelete(action: Delete) {
+      if (!action.old.reference)
+        action.old.reference = action.old.element.nextSibling;
+
       action.old.element.remove();
       return true;
     }
@@ -138,6 +149,15 @@ export function Editing<TBase extends LitElementConstructor>(Base: TBase) {
     private onMove(action: Move) {
       if (!this.checkMoveValidity(action)) return false;
 
+      if (!action.old.reference)
+        action.old.reference = action.old.element.nextSibling;
+
+      if (action.new.reference === undefined)
+        action.new.reference = getReference(
+          action.new.parent,
+          <SCLTag>action.old.element.tagName
+        );
+
       action.new.parent.insertBefore(action.old.element, action.new.reference);
       return true;
     }

src/editors/Communication.ts

@@ -5,7 +5,6 @@ import {
   newWizardEvent,
   newActionEvent,
   createElement,
-  getReference,
 } from '../foundation.js';
 
 import { selectors, styles } from './communication/foundation.js';
@@ -25,7 +24,6 @@ export default class CommunicationPlugin extends LitElement {
         new: {
           parent: this.doc.documentElement,
           element: createElement(this.doc, 'Communication', {}),
-          reference: getReference(this.doc.documentElement, 'Communication'),
         },
       })
     );

src/editors/Templates.ts

@@ -3,7 +3,6 @@ import { translate } from 'lit-translate';
 
 import {
   createElement,
-  getReference,
   identity,
   newActionEvent,
   newWizardEvent,
@@ -127,10 +126,6 @@ export default class TemplatesPlugin extends LitElement {
           new: {
             parent: this.doc.documentElement,
             element: createElement(this.doc, 'DataTypeTemplates', {}),
-            reference: getReference(
-              this.doc.documentElement,
-              'DataTypeTemplates'
-            ),
           },
         })
       );

src/editors/communication/connectedap-editor.ts

@@ -24,7 +24,6 @@ import {
   getValue,
   createElement,
   ComplexAction,
-  getReference,
 } from '../../foundation.js';
 
 import {
@@ -114,7 +113,6 @@ function createConnectedApAction(parent: Element): WizardActor {
               iedName: value.iedName,
               apName: value.apName,
             }),
-            reference: getReference(parent, 'ConnectedAP'),
           },
         }
     );
@@ -225,7 +223,6 @@ export function editConnectedApAction(parent: Element): WizardActor {
         new: {
           parent: parent,
           element: newAddress,
-          reference: getReference(parent, 'Address'),
         },
       });
 

src/editors/communication/subnetwork-editor.ts

@@ -20,7 +20,6 @@ import {
   patterns,
   compareNames,
   createElement,
-  getReference,
   cloneElement,
 } from '../../foundation.js';
 
@@ -149,7 +148,6 @@ export function createSubNetworkAction(parent: Element): WizardActor {
       new: {
         parent,
         element,
-        reference: getReference(parent, 'SubNetwork'),
       },
     };
 

src/editors/substation/guess-wizard.ts

@@ -8,7 +8,6 @@ import {
   compareNames,
   createElement,
   EditorAction,
-  getReference,
   Wizard,
   WizardActor,
   WizardInput,
@@ -205,7 +204,6 @@ function guessBasedOnCSWI(doc: XMLDocument): WizardActor {
       new: {
         parent: substation,
         element: voltageLevel,
-        reference: getReference(substation, 'VoltageLevel'),
       },
     });
 

src/editors/templates/datype-wizards.ts

@@ -4,13 +4,11 @@ import { get, translate } from 'lit-translate';
 import {
   Create,
   EditorAction,
-  getReference,
   getValue,
   identity,
   newActionEvent,
   newWizardEvent,
   patterns,
-  SCLTag,
   selector,
   Wizard,
   WizardActor,
@@ -167,7 +165,6 @@ function addPredefinedDAType(
       new: {
         parent,
         element,
-        reference: getReference(parent, <SCLTag>element.tagName),
       },
     });
 

src/editors/templates/dotype-wizards.ts

@@ -6,13 +6,11 @@ import {
   Create,
   createElement,
   EditorAction,
-  getReference,
   getValue,
   identity,
   isPublic,
   newActionEvent,
   newWizardEvent,
-  SCLTag,
   selector,
   Wizard,
   WizardActor,
@@ -74,7 +72,6 @@ function createSDoAction(parent: Element): WizardActor {
       new: {
         parent,
         element,
-        reference: getReference(parent, <SCLTag>element.tagName),
       },
     });
 
@@ -216,7 +213,6 @@ function addPredefinedDOType(
       new: {
         parent,
         element,
-        reference: getReference(parent, <SCLTag>element.tagName),
       },
     });
 

src/editors/templates/enumtype-wizard.ts

@@ -5,14 +5,12 @@ import {
   cloneElement,
   createElement,
   EditorAction,
-  getReference,
   getValue,
   identity,
   isPublic,
   newActionEvent,
   newWizardEvent,
   patterns,
-  SCLTag,
   selector,
   Wizard,
   WizardActor,
@@ -57,7 +55,6 @@ function createEnumValAction(parent: Element): WizardActor {
       new: {
         parent,
         element,
-        reference: getReference(parent, 'EnumVal'),
       },
     };
 
@@ -195,7 +192,6 @@ function createAction(parent: Element, templates: XMLDocument): WizardActor {
       new: {
         parent,
         element,
-        reference: getReference(parent, <SCLTag>element.tagName),
       },
     };
 

src/editors/templates/foundation.ts

@@ -5,10 +5,8 @@ import {
   cloneElement,
   Create,
   EditorAction,
-  getReference,
   getValue,
   isPublic,
-  SCLTag,
   WizardActor,
   WizardInput,
 } from '../../foundation.js';
@@ -98,7 +96,6 @@ export function addReferencedDataTypes(
       new: {
         parent,
         element: <Element>adjacent.cloneNode(true),
-        reference: getReference(parent, <SCLTag>adjacent.tagName),
       },
     });
   });