wgge 2.0.26 → 2.0.28

package/core/controller/ControllerBase.js

@@ -26,7 +26,7 @@ export default class ControllerBase extends ActivatedTreeNode {
 		this.game = game;
 		this.model = model;
 
-		this.updateActions = null;
+		this.updateActions = [];
 	}
 
 	update(delta) {
@@ -34,16 +34,23 @@ export default class ControllerBase extends ActivatedTreeNode {
 			return;
 		}
 
-		if (this.updateActions) {
-			while (this.updateActions.length > 0) {
-				const action = this.updateActions.shift();
-				action(delta);
-			}
-			this.updateActions = null;
+		if (this.updateActions.length > 0) {
+			this.updateActions.forEach((action) => action(delta));
+			this.updateActions = [];
 		}
 
 		this.updateInternal(delta);
-		this.children.forEach((c) => c.update(delta));
+
+		if (this.children.length > 0) {
+			// this make sure all children are updated even if some of them are removed during processing
+			for (let i = 0, max = this.children.length; i < max; i++) {
+				this.updateActions.push(() => this.children[i].update(delta));
+			}
+			for (let i = 0, max = this.updateActions.length; i < max; i++) {
+				this.updateActions[i]();
+			}
+			this.updateActions = [];
+		}
 	}
 
 	updateInternal(delta) {
@@ -55,9 +62,6 @@ export default class ControllerBase extends ActivatedTreeNode {
 	 * @param action (delta) => any
 	 */
 	runOnUpdate(action) {
-		if (!this.updateActions) {
-			this.updateActions = [];
-		}
 		this.updateActions.push(action);
 	}
 

package/core/renderer/RendererBase.js

@@ -30,8 +30,11 @@ export default class RendererBase extends ActivatedTreeNode {
 		if (!this.isActivated) {
 			return;
 		}
+
 		this.renderInternal();
+
 		this.children.forEach((c) => c.render());
+
 		if (this.isRoot()) {
 			this.model.clean();
 		}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "wgge",
-	"version": "2.0.26",
+	"version": "2.0.28",
 	"description": "Web GL Game Engine",
 	"main": "index.js",
 	"devDependencies": {