From 2186821e387ad381067a8696c5ce06c3edbe0f88 Mon Sep 17 00:00:00 2001
From: Florin Raducan <florin.raducan@netcentric.biz>
Date: Wed, 6 Nov 2024 17:23:01 +0200
Subject: [PATCH] Fix Module loading

---
 scripts/editor.js |  4 ++--
 scripts/libs.js   | 30 +++++++++++++++---------------
 2 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/scripts/editor.js b/scripts/editor.js
index 5648220..c19f74a 100644
--- a/scripts/editor.js
+++ b/scripts/editor.js
@@ -91,8 +91,8 @@ export default function initEditor(listeners = true) {
             try {
               const fn = window.raqnComponents[componentName];
               const name = fn.name.toLowerCase();
-              const component = await loadModule(`/blocks/${name}/${name}.editor`, { loadCSS: false });
-              const mod = component.js;
+              const component = loadModule(`/blocks/${name}/${name}.editor`, { loadCSS: false });
+              const mod = await component.js;
               if (mod && mod.default) {
                 const dialog = await mod.default();
 
diff --git a/scripts/libs.js b/scripts/libs.js
index 95c7c46..828f593 100644
--- a/scripts/libs.js
+++ b/scripts/libs.js
@@ -420,8 +420,8 @@ export function deepMergeByType(keyPathMethods, origin, ...toMerge) {
   return deepMergeByType({ pathsArrays, currentPath }, origin, ...toMerge);
 }
 
-export async function loadModule(urlWithoutExtension, { loadCSS = true, loadJS = true }) {
-  const modules = { js: Promise.resolve(null), css: Promise.resolve(null) };
+export function loadModule(urlWithoutExtension, { loadCSS = true, loadJS = true }) {
+  const modules = { js: null, css: null };
   if (!urlWithoutExtension) return modules;
 
   if (loadJS) {
@@ -447,13 +447,10 @@ export async function loadModule(urlWithoutExtension, { loadCSS = true, loadJS =
           link.rel = 'stylesheet';
           resolve(link);
         };
-        link.onerror = (error) => reject(error);
+        link.onerror = reject;
         document.head.append(link);
       } else {
-        style.onload = () => {
-          resolve(style);
-        };
-        style.onerror = (error) => reject(error);
+        resolve(style);
       }
     }).catch((error) => {
       // eslint-disable-next-line no-console
@@ -462,23 +459,26 @@ export async function loadModule(urlWithoutExtension, { loadCSS = true, loadJS =
     });
   }
 
-  modules.js = await modules.js;
-  modules.css = await modules.css;
-
   return modules;
 }
 
 export async function loadAndDefine(componentConfig) {
   const { tag, module: { path, loadJS, loadCSS } = {} } = componentConfig;
-  const { js, css } = await loadModule(path, { loadJS, loadCSS });
+  if (window.raqnComponents[tag]) {
+    return { tag, module: window.raqnComponents[tag] };
+  }
+
+  const { js } = loadModule(path, { loadJS, loadCSS });
+
+  const module = await js;
 
-  if (js?.default?.prototype instanceof HTMLElement) {
+  if (module?.default?.prototype instanceof HTMLElement) {
     if (!window.customElements.get(tag)) {
-      window.customElements.define(tag, js.default);
-      window.raqnComponents[tag] = js.default;
+      window.customElements.define(tag, module.default);
+      window.raqnComponents[tag] = module.default;
     }
   }
-  return { tag, module: js, style: css };
+  return { tag, module };
 }
 
 export function mergeUniqueArrays(...arrays) {