Parcourir la source

Update ColladaLoader.js

Fix materials incorrectly defaulting to  THREE.ClampToEdgeWrapping when THREE.RepeatWrapping is expected.
Some dae have a value of 'TRUE' which becomes NaN via parseInt when the desired value is 1
PeanutPower il y a 11 ans
Parent
commit
4031438870
1 fichiers modifiés avec 7 ajouts et 3 suppressions
  1. 7 3
      examples/js/loaders/ColladaLoader.js

+ 7 - 3
examples/js/loaders/ColladaLoader.js

@@ -3088,9 +3088,13 @@ THREE.ColladaLoader = function () {
 
 				case 'wrapU':
 				case 'wrapV':
-
-					this.texOpts[ child.nodeName ] = parseInt( child.textContent );
-					break;
+					
+					// some dae have a value of true which becomes NaN via parseInt			
+			                if (child.textContent.toUpperCase() == 'TRUE'){
+			                     this.texOpts[ child.nodeName ] = 1;
+			                } else {
+			                     this.texOpts[ child.nodeName ] = parseInt( child.textContent );
+			                }
 
 				default:
 					this.texOpts[ child.nodeName ] = child.textContent;