Refactoring foundation functions, more stable.

Signed-off-by: Dennis Labordus <[email protected]>

Author: Dennis Labordus

src/compas-editors/CompasVersions.ts

@@ -17,7 +17,6 @@ import {
   dispatchEventOnOpenScd,
   getOpenScdElement,
   getTypeFromDocName,
-  setDocIdOnOpenScd,
   updateDocumentInOpenSCD
 } from "../compas/foundation.js";
 import {addVersionToCompasWizard} from "../compas/CompasUploadVersion.js";
@@ -284,12 +283,15 @@ function confirmDeleteCompasWizard(docName: string, docId: string): Wizard {
       CompasSclDataService()
         .deleteSclDocument(type, docId)
         .then (() => {
-          setDocIdOnOpenScd('');
-          dispatchEventOnOpenScd(
-            newLogEvent({
-              kind: 'info',
-              title: get('compas.versions.deleteSuccess')
-            }));
+          const openScd = getOpenScdElement();
+          if (openScd !== null) {
+            openScd.docId = '';
+            openScd.dispatchEvent(
+              newLogEvent({
+                kind: 'info',
+                title: get('compas.versions.deleteSuccess')
+              }));
+          }
         })
         .catch(createLogEvent);
 

src/compas/foundation.ts

@@ -33,13 +33,6 @@ export function dispatchEventOnOpenScd(event: Event): void {
   }
 }
 
-export function setDocIdOnOpenScd(docId: string): void {
-  const openScd = getOpenScdElement();
-  if (openScd !== null) {
-    openScd.docId = docId;
-  }
-}
-
 export function updateDocumentInOpenSCD(doc: Document, docName?: string): void {
   const id = (doc.querySelectorAll(':root > Header') ?? []).item(0)?.getAttribute('id') ?? '';