fix(nodes): node added to children after loading are now loaded

Author: ColinEspinas

src/core/Engine.ts

@@ -61,7 +61,7 @@ export class Engine {
   /**
    * Inits the engine by registering default core systems.
    */
-  private init(): void {}
+  private init(): void { }
 
   /**
    * Loads systems/rootNode and starts the update loops.

src/core/InnerNode.ts

@@ -25,13 +25,15 @@ export class InnerNode extends TreeNode {
   }
 
   /**
-   * Adds a child node to this node children.
+   * Adds a child node to this node children and loads it.
    * @param {TreeNode} node Node to add as child.
    * @returns {number} New children array length.
    * @sealed
    */
   public add(node: TreeNode): number {
     this.children.push(node);
+    // Load the added node if added after loading.
+    if (this.isLoaded) node.load();
     return this.children.length;
   }
 
@@ -67,6 +69,7 @@ export class InnerNode extends TreeNode {
     for (let i = 0, len = this.children.length; i !== len; ++i) {
       this.children[i].load();
     }
+    this.isLoaded = true;
   }
 
   /**
@@ -109,14 +112,15 @@ export class InnerNode extends TreeNode {
     for (let i = 0, len = this.children.length; i !== len; ++i) {
       this.children[i].unload();
     }
+    this.isLoaded = false;
   }
 
   /**
    * Called by the node's constructor at instantiation,
    * this function is to be implemented when needed.
    * @virtual
    */
-  protected onCreate(): void {}
+  protected onCreate(): void { }
 
   /**
    * Called by the parent node when loaded,
@@ -126,21 +130,21 @@ export class InnerNode extends TreeNode {
    * function will be called by the engine instead.
    * @virtual
    */
-  protected onLoad(): void {}
+  protected onLoad(): void { }
 
   /**
    * Called by the parent node at each step of the loop,
    * this function is to be implemented when needed.
    * @virtual
    */
-  protected onStep(): void {}
+  protected onStep(): void { }
 
   /**
    * Called by the parent node at each fixed step of the loop,
    * this function is to be implemented when needed.
    * @virtual
    */
-  protected onFixedStep(): void {}
+  protected onFixedStep(): void { }
 
   /**
    * Called by the parent node at unload,
@@ -150,5 +154,5 @@ export class InnerNode extends TreeNode {
    * function will be called by the engine instead.
    * @virtual
    */
-  protected onUnload(): void {}
+  protected onUnload(): void { }
 }

src/core/OuterNode.ts

@@ -42,6 +42,7 @@ export class OuterNode extends TreeNode {
    */
   public load(): void {
     this.onLoad();
+    this.isLoaded = true;
   }
 
   /**
@@ -75,14 +76,15 @@ export class OuterNode extends TreeNode {
    */
   public unload(): void {
     this.onUnload();
+    this.isLoaded = false;
   }
 
   /**
    * Called by the node's constructor at instantiation,
    * this method is to be implemented when needed.
    * @virtual
    */
-  protected onCreate(): void {}
+  protected onCreate(): void { }
 
   /**
    * Called by the parent node when loaded,
@@ -92,21 +94,21 @@ export class OuterNode extends TreeNode {
    * method will be called by the engine instead.
    * @virtual
    */
-  protected onLoad(): void {}
+  protected onLoad(): void { }
 
   /**
    * Called by the parent node at each step of the loop,
    * this method is to be implemented when needed.
    * @virtual
    */
-  protected onStep(): void {}
+  protected onStep(): void { }
 
   /**
    * Called by the parent node at each fixed step of the loop,
    * this method is to be implemented when needed.
    * @virtual
    */
-  protected onFixedStep(): void {}
+  protected onFixedStep(): void { }
 
   /**
    * Called by the parent node at unload,
@@ -116,5 +118,5 @@ export class OuterNode extends TreeNode {
    * method will be called by the engine instead.
    * @virtual
    */
-  protected onUnload(): void {}
+  protected onUnload(): void { }
 }

src/core/TreeNode.ts

@@ -20,6 +20,11 @@ export abstract class TreeNode {
    */
   protected engine: Engine;
 
+  /**
+   * Get wether the node is loaded.
+   */
+  protected isLoaded: boolean;
+
   /**
    * Get parent node.
    */
@@ -36,6 +41,7 @@ export abstract class TreeNode {
       this.engine = parent;
     }
     this.id = nanoid(16);
+    this.isLoaded = false;
   }
 
   /**