fix hasSkin default to true

shrekshao · Aug 28, 2017 · 85492f3 · 85492f3
1 parent 9cb9c47
commit 85492f3
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/examples/webgl2-renderer.html b/examples/webgl2-renderer.html
@@ -730,7 +730,7 @@
 
             var hasIndices = true;
 
-            var hasSkin = true;
+            var hasSkin = false;
 
             // var nodeMatrix = new Array(glTF.nodes.length);
             // for(i = 0, len = nodeMatrix.length; i < len; i++) {
@@ -752,7 +752,9 @@
 
                 if (hasSkin) {
                     // @tmp: assume material only use base color
-                    baseColor = primitive.material.pbrMetallicRoughness.baseColorFactor;
+                    if ( primitive.material.pbrMetallicRoughness.baseColorFactor ) {
+                        baseColor = primitive.material.pbrMetallicRoughness.baseColorFactor;
+                    }
                 } else {
 
                     if (primitive.material !== null) {
@@ -844,7 +846,7 @@
                 }
                 // mat4.mul(matrix, parentModelMatrix, node.matrix);
 
-
+                hasSkin = false;
                 if (node.skin !== null) {
                     // mesh node with skin
                     hasSkin = true;