Merge branch 'main' into Fetch_BTComponents

Author: pascalwilbrink

CHANGELOG.md

@@ -2,6 +2,20 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+## [0.33.0](https://github.com/openscd/open-scd/compare/v0.32.0...v0.33.0) (2023-06-22)
+
+
+### Features
+
+* **openscd:** Move progress indicator beneath plugin tabs ([#1181](https://github.com/openscd/open-scd/issues/1181)) ([a7a7081](https://github.com/openscd/open-scd/commits/a7a7081a32ee14998d0ead75ffb89ba2726631f8)), closes [#1178](https://github.com/openscd/open-scd/issues/1178)
+
+
+### Bug Fixes
+
+* **editing:** use editCount property for change propagation ([#1233](https://github.com/openscd/open-scd/issues/1233)) ([548f63b](https://github.com/openscd/open-scd/commits/548f63b5b35dac6772230004e6ca7859cf3337b8))
+* escaped symbols in regex patterns ([#1266](https://github.com/openscd/open-scd/issues/1266)) ([de2dd0d](https://github.com/openscd/open-scd/commits/de2dd0dc2351c8feb6a70da62e43640334df571b))
+* goose subscription reflects state incorrectly ([#1261](https://github.com/openscd/open-scd/issues/1261)) ([4440bff](https://github.com/openscd/open-scd/commits/4440bff63c0ebe816a9fdc25f8af6da1ca6645f4))
+
 ## [0.31.0](https://github.com/openscd/open-scd/compare/v0.30.0...v0.31.0) (2023-05-02)
 
 

manifest.json

@@ -40,5 +40,5 @@
       "purpose": "maskable"
     }
   ],
-  "version": "0.31.0"
+  "version": "0.33.0"
 }

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "open-scd",
-  "version": "0.31.0",
+  "version": "0.33.0",
   "repository": "https://github.com/openscd/open-scd.git",
   "description": "A bottom-up substation configuration designer for projects described using SCL `IEC 61850-6` Edition 2 or greater.",
   "keywords": [
@@ -17,7 +17,6 @@
   "module": "open-scd.js",
   "type": "module",
   "dependencies": {
-    "@material/mwc-circular-progress-four-color": "0.22.1",
     "@material/mwc-dialog": "0.22.1",
     "@material/mwc-drawer": "0.22.1",
     "@material/mwc-fab": "0.22.1",

package.json

@@ -1,15 +1,15 @@
 export const officialPlugins = [
   {
-    name: 'Substation',
-    src: '/src/editors/Substation.js',
-    icon: 'margin',
+    name: 'IED',
+    src: '/src/editors/IED.js',
+    icon: 'developer_board',
     default: true,
     kind: 'editor',
   },
   {
-    name: 'IED',
-    src: '/src/editors/IED.js',
-    icon: 'developer_board',
+    name: 'Substation',
+    src: '/src/editors/Substation.js',
+    icon: 'margin',
     default: true,
     kind: 'editor',
   },
@@ -24,14 +24,14 @@ export const officialPlugins = [
     name: 'Subscriber Message Binding (GOOSE)',
     src: '/src/editors/GooseSubscriberMessageBinding.js',
     icon: 'link',
-    default: true,
+    default: false,
     kind: 'editor',
   },
   {
     name: 'Subscriber Data Binding (GOOSE)',
     src: '/src/editors/GooseSubscriberDataBinding.js',
     icon: 'link',
-    default: true,
+    default: false,
     kind: 'editor',
   },
   {
@@ -45,14 +45,14 @@ export const officialPlugins = [
     name: 'Subscriber Message Binding (SMV)',
     src: '/src/editors/SMVSubscriberMessageBinding.js',
     icon: 'link',
-    default: true,
+    default: false,
     kind: 'editor',
   },
   {
     name: 'Subscriber Data Binding (SMV)',
     src: '/src/editors/SMVSubscriberDataBinding.js',
     icon: 'link',
-    default: true,
+    default: false,
     kind: 'editor',
   },
   {
@@ -168,10 +168,10 @@ export const officialPlugins = [
     position: 'top',
   },
   {
-    name: 'Validate using OCL',
+    name: '[WIP] Validate using OCL',
     src: '/src/validators/CompasValidateSchema.js',
     icon: 'rule_folder',
-    default: true,
+    default: false,
     kind: 'validator',
   },
   {
@@ -329,7 +329,7 @@ export const officialPlugins = [
   },
   {
     name: 'Autogen Substation',
-    src: '/src/editors/substation/autogen-substation/autogen-substation.js',
+    src: '/src/compas-editors/autogen-substation.js',
     icon: 'playlist_add_circle',
     default: true,
     kind: 'menu',

public/js/plugins.js

@@ -424,16 +424,6 @@ export function Editing<TBase extends LitElementConstructor>(Base: TBase) {
 
       if (!this.doc) return;
 
-      const newDoc = document.implementation.createDocument(
-        this.doc.lookupNamespaceURI(''),
-        this.doc.documentElement.tagName,
-        this.doc.doctype
-      );
-
-      // change the document object reference to enable reactive updates
-      newDoc.documentElement.replaceWith(this.doc.documentElement);
-      this.doc = newDoc;
-
       await this.updateComplete;
       this.dispatchEvent(newValidateEvent());
     }

src/Editing.ts

@@ -246,6 +246,7 @@ export function Hosting<
     renderMenuItem(me: MenuItem | 'divider'): TemplateResult {
       if (me === 'divider')
         return html`<li divider padded role="separator"></li>`;
+      if (me.actionItem) return html``;
       return html`
         <mwc-list-item
           class="${me.kind}"
@@ -294,10 +295,12 @@ export function Hosting<
             wrapFocus
             @action=${(ae: CustomEvent<ActionDetail>) => {
               //FIXME: dirty hack to be fixed in open-scd-core
-              //       if clause not neccassary when oscd... compenents in open-scd not list
+              //       if clause not necessary when oscd... components in open-scd not list
               if (ae.target instanceof List)
                 (<MenuItem>(
-                  this.menu.filter(item => item !== 'divider')[ae.detail.index]
+                  this.menu.filter(
+                    item => item !== 'divider' && !item.actionItem
+                  )[ae.detail.index]
                 ))?.action?.(ae);
             }}
           >

src/Hosting.ts

@@ -63,7 +63,7 @@ function getPluginName(src: string): string {
  * A mixin adding a `history` property to any `LitElement`, in which
  * incoming [[`LogEvent`]]s are logged.
  *
- * For [[`EditorAction`]] entries, also sets `currentAction` to the index of
+ * For [[`EditorAction`]] entries, also sets `editCount` to the index of
  * the committed action, allowing the user to go to `previousAction` with
  * `undo()` if `canUndo` and to go to `nextAction` with `redo()` if `canRedo`.
  *
@@ -79,7 +79,7 @@ export function Logging<TBase extends LitElementConstructor>(Base: TBase) {
     history: LogEntry[] = [];
     /** Index of the last [[`EditorAction`]] applied. */
     @property({ type: Number })
-    currentAction = -1;
+    editCount = -1;
     @property()
     diagnoses = new Map<string, IssueDetail[]>();
     @internalProperty()
@@ -93,7 +93,7 @@ export function Logging<TBase extends LitElementConstructor>(Base: TBase) {
     @query('#issue') issueUI!: Snackbar;
 
     get canUndo(): boolean {
-      return this.currentAction >= 0;
+      return this.editCount >= 0;
     }
     get canRedo(): boolean {
       return this.nextAction >= 0;
@@ -102,15 +102,15 @@ export function Logging<TBase extends LitElementConstructor>(Base: TBase) {
     get previousAction(): number {
       if (!this.canUndo) return -1;
       return this.history
-        .slice(0, this.currentAction)
+        .slice(0, this.editCount)
         .map(entry => (entry.kind == 'action' ? true : false))
         .lastIndexOf(true);
     }
     get nextAction(): number {
       let index = this.history
-        .slice(this.currentAction + 1)
+        .slice(this.editCount + 1)
         .findIndex(entry => entry.kind == 'action');
-      if (index >= 0) index += this.currentAction + 1;
+      if (index >= 0) index += this.editCount + 1;
       return index;
     }
 
@@ -129,25 +129,25 @@ export function Logging<TBase extends LitElementConstructor>(Base: TBase) {
       if (!this.canUndo) return false;
       this.dispatchEvent(
         newActionEvent(
-          invert((<CommitEntry>this.history[this.currentAction]).action)
+          invert((<CommitEntry>this.history[this.editCount]).action)
         )
       );
-      this.currentAction = this.previousAction;
+      this.editCount = this.previousAction;
       return true;
     }
     redo(): boolean {
       if (!this.canRedo) return false;
       this.dispatchEvent(
         newActionEvent((<CommitEntry>this.history[this.nextAction]).action)
       );
-      this.currentAction = this.nextAction;
+      this.editCount = this.nextAction;
       return true;
     }
 
     private onLog(le: LogEvent): void {
       if (le.detail.kind === 'reset') {
         this.history = [];
-        this.currentAction = -1;
+        this.editCount = -1;
         return;
       }
 
@@ -160,7 +160,7 @@ export function Logging<TBase extends LitElementConstructor>(Base: TBase) {
         if (entry.action.derived) return;
         entry.action.derived = true;
         if (this.nextAction !== -1) this.history.splice(this.nextAction);
-        this.currentAction = this.history.length;
+        this.editCount = this.history.length;
       }
 
       this.history.push(entry);
@@ -210,7 +210,7 @@ export function Logging<TBase extends LitElementConstructor>(Base: TBase) {
           class="${entry.kind}"
           graphic="icon"
           ?twoline=${!!entry.message}
-          ?activated=${this.currentAction == history.length - index - 1}
+          ?activated=${this.editCount == history.length - index - 1}
         >
           <span>
             <!-- FIXME: replace tt with mwc-chip asap -->