Fixed branch recompile bug: #297

xeolabs · Oct 11, 2013 · 9f81ccb · 9f81ccb
1 parent 475e224
commit 9f81ccb
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/api/latest/scenejs.js b/api/latest/scenejs.js
@@ -7364,11 +7364,9 @@ SceneJS.Node.prototype._compileNodes = function () {
         child.branchDirty = child.branchDirty || this.branchDirty; // Compile subnodes if scene branch dirty
 
         if (child.dirty || child.branchDirty || this._engine.sceneDirty) {  // Compile nodes that are flagged dirty
-
+            child._compile();
             child.dirty = false;
             child.branchDirty = false;
-
-            child._compile();
         }
     }
 
@@ -7378,6 +7376,7 @@ SceneJS.Node.prototype._compileNodes = function () {
 };
 
 
+
 /**
  * Destroys this node. It is marked for destruction; when the next scene traversal begins (or the current one ends)
  * it will be destroyed and removed from it's parent.
@@ -10542,7 +10541,7 @@ new (function () {
     SceneJS.Name.prototype._compile = function () {
 
         this._engine.display.name = coreStack[stackLen++] = this._core;
-        
+
         // (Re)build name path
         var path = [];
         var name;

diff --git a/src/core/scene/node.js b/src/core/scene/node.js
@@ -1309,11 +1309,9 @@ SceneJS.Node.prototype._compileNodes = function () {
         child.branchDirty = child.branchDirty || this.branchDirty; // Compile subnodes if scene branch dirty
 
         if (child.dirty || child.branchDirty || this._engine.sceneDirty) {  // Compile nodes that are flagged dirty
-
+            child._compile();
             child.dirty = false;
             child.branchDirty = false;
-
-            child._compile();
         }
     }
 
@@ -1323,6 +1321,7 @@ SceneJS.Node.prototype._compileNodes = function () {
 };
 
 
+
 /**
  * Destroys this node. It is marked for destruction; when the next scene traversal begins (or the current one ends)
  * it will be destroyed and removed from it's parent.