fixed recursve loading error

Author: levizwannah

OpenScript.js

@@ -485,11 +485,21 @@ var OpenScript = {
          * @returns
          */
         async emit(event, ...args) {
+            let events = event.split(/\s+/g);
+
+            for(let i = 0; i < events.length; i++){
+                let evt = events[i];
+                if(evt.length < 1) continue;
+                this.#emit(evt, ...args);
+            }
+        }
+
+        async #emit(event, ...args){
             const currentTime = () => new Date().getTime();
 
             this.#logs[event] = this.#logs[event] ?? [];
             this.#logs[event].push({ timestamp: currentTime(), args: args });
-            if (this.#shouldLog) console.log(`fired ${event}: args`, args);
+            if (this.#shouldLog) console.log(`fired ${event}: args: `, args);
 
             return this.#emitter.emit(event, ...args);
         }
@@ -2858,8 +2868,10 @@ var OpenScript = {
          * @param {string} fileName script name without the .js.
          */
         async req(fileName) {
-            let names = fileName.split(/\./);
+            if(!/^[\w\._-]+$/.test(fileName)) throw Error(`OJS-INVALID-FILE: '${fileName}' is an invalid file name`);
 
+            let names = fileName.split(/\./);
+            
             if (OpenScript.AutoLoader.history.has(`${this.dir}.${fileName}`))
                 return OpenScript.AutoLoader.history.get(
                     `${this.dir}.${fileName}`
@@ -2870,6 +2882,7 @@ var OpenScript = {
                     this.version
                 }`
             );
+            
 
             let classes = await response.text();
             let content = classes;

example/ojs-config.js

@@ -3,19 +3,29 @@
  |----------------------------------
 */
 
-/**----------------------------------
+/*----------------------------------
  * 
  * Set the default route path here
  * ----------------------------------
  */
-route.basePath('example'); // === '/'
+route.basePath(''); // === '/'
+
+/*-----------------------------------
+ | Set the global runtime prefix.
+ | This prefix will be appended
+ | to every path before resolution.
+ | So ensure when defining routes,
+ | you have it as the main prefix.
+ |------------------------------------
+*/
+route.runtimePrefix(''); // === ''
 
 /*-----------------------------------
  | set the directories in which we
  | can find the context files
  |-----------------------------------
 */
-ContextProvider.directory = route.baseUrl('contexts');
+ContextProvider.directory = route.baseUrl('example/contexts');
 
 /*-----------------------------------
  | set the version number of the
@@ -32,7 +42,7 @@ ContextProvider.version = '1.0.0';
  | from that directory
  |-----------------------------------
 */
-MediatorManager.directory = route.baseUrl('mediators');
+MediatorManager.directory = route.baseUrl('example/mediators');
 
 /*-----------------------------------
  | Set the version number of the 
@@ -48,7 +58,7 @@ MediatorManager.version = '1.0.0';
  | directory for the loader
  |-----------------------------------
 */
-loader.dir = route.baseUrl('components');
+loader.dir = route.baseUrl('example/components');
 
 /*-----------------------------------
  | set the version number of the
@@ -65,7 +75,7 @@ loader.version = '1.0.0';
  |-----------------------------------
 */
 
-autoload.dir = route.baseUrl('classes');
+autoload.dir = route.baseUrl('example/classes');
 
 /*-----------------------------------
  | set the version number of the
@@ -75,6 +85,25 @@ autoload.dir = route.baseUrl('classes');
 */
 autoload.version = '1.0.0';
 
+
+/*---------------------------------
+ | Get the initial body style so
+ | so that when routes change,
+ | we can reset the body style.
+ | Sometimes, the body element's
+ | style is affected by elements
+ | That are no longer on the DOM.
+ | example, bootstrap offcanvas.
+ | In these cases, you need to
+ | reset the body style when
+ | the routeChanged event is fired
+ | by the router. So you can listen
+ | to it and use this to reset
+ | the style.
+ |----------------------------------
+*/
+var __bodyStyle = document.body.getAttribute("style");
+
 /*--------------------------------
  | Set the logs clearing interval
  | for the broker to remove stale
@@ -97,4 +126,12 @@ broker.TIME_TO_GC = 10000; // 10 secs
  | collector for the broker
  |-------------------------------------------
 */
-broker.removeStaleEvents(); // broker garbage collection started
+broker.removeStaleEvents(); // broker garbage collection started
+
+/*------------------------------------------
+ | When this is set to true, the broker
+ | will display events and their payloads
+ | in the console when they are fired
+ |------------------------------------------
+*/
+broker.withLogs(true); 

ojs-config.js

@@ -3,13 +3,23 @@
  |----------------------------------
 */
 
-/**----------------------------------
+/*----------------------------------
  * 
  * Set the default route path here
  * ----------------------------------
  */
 route.basePath(''); // === '/'
 
+/*-----------------------------------
+ | Set the global runtime prefix.
+ | This prefix will be appended
+ | to every path before resolution.
+ | So ensure when defining routes,
+ | you have it as the main prefix.
+ |------------------------------------
+*/
+route.runtimePrefix(''); // === ''
+
 /*-----------------------------------
  | set the directories in which we
  | can find the context files
@@ -75,6 +85,25 @@ autoload.dir = route.baseUrl('js/classes');
 */
 autoload.version = '1.0.0';
 
+
+/*---------------------------------
+ | Get the initial body style so
+ | so that when routes change,
+ | we can reset the body style.
+ | Sometimes, the body element's
+ | style is affected by elements
+ | That are no longer on the DOM.
+ | example, bootstrap offcanvas.
+ | In these cases, you need to
+ | reset the body style when
+ | the routeChanged event is fired
+ | by the router. So you can listen
+ | to it and use this to reset
+ | the style.
+ |----------------------------------
+*/
+var __bodyStyle = document.body.getAttribute("style");
+
 /*--------------------------------
  | Set the logs clearing interval
  | for the broker to remove stale
@@ -97,4 +126,12 @@ broker.TIME_TO_GC = 10000; // 10 secs
  | collector for the broker
  |-------------------------------------------
 */
-broker.removeStaleEvents(); // broker garbage collection started
+broker.removeStaleEvents(); // broker garbage collection started
+
+/*------------------------------------------
+ | When this is set to true, the broker
+ | will display events and their payloads
+ | in the console when they are fired
+ |------------------------------------------
+*/
+broker.withLogs(true);