wgge 2.0.29 → 2.0.31

package/core/ActivatedTreeNode.js

@@ -17,6 +17,11 @@ export default class ActivatedTreeNode {
 	 */
 	isActivated;
 
+	/**
+	 * @type boolean
+	 */
+	isRemoved= false;
+
 	/**
 	 *
 	 * @type array
@@ -26,6 +31,11 @@ export default class ActivatedTreeNode {
 	 */
 	autoRegisterEvents = null;
 
+	/**
+	 * @type number
+	 */
+	childrenRemoved = 0;
+
 	constructor() {
 		this.isActivated = false;
 		this.parent = null;
@@ -55,24 +65,19 @@ export default class ActivatedTreeNode {
 	}
 
 	hasChildren() {
-		return this.children.length > 0;
+		return this.children.length > this.childrenRemoved;
 	}
 
 	removeChild(node) {
 		const index = this.children.indexOf(node);
 		if (index >= 0) {
-			this.children.splice(index, 1);
+			this.childrenRemoved++;
 			node.deactivate();
+			node.isRemoved = true;
 			node.parent = null;
 			return node;
 		}
-		for (let i = 0, max = this.children.length; i < max; i++) {
-			const child = this.children[i];
-			const result = child.removeChild(node);
-			if (result) {
-				return result;
-			}
-		}
+		return undefined;
 	}
 
 	removeMyself() {
@@ -84,16 +89,21 @@ export default class ActivatedTreeNode {
 	}
 
 	resetChildren() {
-		this.children.forEach((child) => child.deactivate());
+		for (let i = 0, max = this.children.length; i < max; i++) {
+			this.children[i].deactivate();
+		}
 		this.children = [];
+		this.childrenRemoved = 0;
 	}
 
-	forEach(func) {
-		func(this);
+	updateRemovedChildren() {
+		if (this.childrenRemoved === 0) return;
+		let w = 0;
 		for (let i = 0, max = this.children.length; i < max; i++) {
-			const child = this.children[i];
-			child.forEach(func);
+			if (!this.children[i].isRemoved) this.children[w++] = this.children[i];
 		}
+		this.children.length = w;
+		this.childrenRemoved = 0;
 	}
 
 	activate() {

package/core/controller/ControllerBase.js

@@ -43,17 +43,12 @@ export default class ControllerBase extends ActivatedTreeNode {
 
 		this.updateInternal(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++) {
-				const child = this.children[i];
-				this.updateActions.push(() => child.update(delta));
-			}
-			for (let i = 0, max = this.updateActions.length; i < max; i++) {
-				this.updateActions[i]();
-			}
-			this.updateActions = [];
+		for (let i = 0, max = this.children.length; i < max; i++) {
+			const child = this.children[i];
+			if (!child.isRemoved) child.update(delta);
 		}
+
+		this.updateRemovedChildren();
 	}
 
 	updateInternal(delta) {

package/core/renderer/RendererBase.js

@@ -20,7 +20,6 @@ export default class RendererBase extends ActivatedTreeNode {
 		super();
 		this.game = game;
 		this.model = model;
-
 	}
 
 	render() {
@@ -33,7 +32,12 @@ export default class RendererBase extends ActivatedTreeNode {
 
 		this.renderInternal();
 
-		this.children.forEach((c) => c.render());
+		for (let i = 0, max = this.children.length; i < max; i++) {
+			const child = this.children[i];
+			if (!child.isRemoved) child.render();
+		}
+
+		this.updateRemovedChildren();
 
 		if (this.isRoot()) {
 			this.model.clean();

package/package.json

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