Browse Source

GLTFExporter: Fix undefined node name bug in mergeMorphTargetTracks()

Takahiro 4 years ago
parent
commit
72d8db9919
2 changed files with 2 additions and 2 deletions
  1. 1 1
      examples/js/exporters/GLTFExporter.js
  2. 1 1
      examples/jsm/exporters/GLTFExporter.js

+ 1 - 1
examples/js/exporters/GLTFExporter.js

@@ -2282,7 +2282,7 @@ THREE.GLTFExporter.Utils = {
 
 				// We need to take into consideration the intended target node
 				// of our original un-merged morphTarget animation.
-				mergedTrack.name = sourceTrackBinding.nodeName + '.morphTargetInfluences';
+				mergedTrack.name = ( sourceTrackBinding.nodeName || '' ) + '.morphTargetInfluences';
 				mergedTrack.values = values;
 
 				mergedTracks[ sourceTrackNode.uuid ] = mergedTrack;

+ 1 - 1
examples/jsm/exporters/GLTFExporter.js

@@ -2305,7 +2305,7 @@ GLTFExporter.Utils = {
 
 				// We need to take into consideration the intended target node
 				// of our original un-merged morphTarget animation.
-				mergedTrack.name = sourceTrackBinding.nodeName + '.morphTargetInfluences';
+				mergedTrack.name = ( sourceTrackBinding.nodeName || '' ) + '.morphTargetInfluences';
 				mergedTrack.values = values;
 
 				mergedTracks[ sourceTrackNode.uuid ] = mergedTrack;