FBXLoader2 Fixes - NormalsIndex & Material Mapping

[Kyle-Larson]

Fixed more FBXLoader2 bugs

• Fixed bug where NormalIndex was referenced as NormalsIndex.
• Fixed bug where geometry without any assigned material had material mapping information listed as "NoMappingInformation"

[mrdoob]

Thanks!

[mrdoob]

@Kyle-Larson Here are some more FBX that fail to load... 😇

http://files.davidoreilly.com/downloads/EXTW_UnityRigs.zip

[Kyle-Larson]

@mrdoob All of these FBX files are binary exported, which we don't currently support.

[mrdoob]

Oh, I see. Is there a way to detect that and warn the user?

Right now this is all the user gets:

[Kyle-Larson]

Perchance did I use the wrong files? I tested the files in the FBX folder. Even if there wasn't a check for binary vs. ASCII, the loader wouldn't be getting as far as to report an error like that.

[mrdoob]

Oh... The check must be in the load()? The editor uses the parse() directly.

[Kyle-Larson]

Yeah, you're right on that account. Might make more sense to put the check in parse, since that's the function that has to do the actual parsing. I'll make that quick change and make a new PR.