KhronosGroup/OpenCOLLADA — pull request #224 by gaiaclary (2013-09-13 14:45:25+00:00)
Title: Make import of Joint node hierarchies less strict

This is a fix for fixing commit: 1a336bb

The above mentioned commit assumes that nodes of type joint always have an associated sid.
However i could not find any hint that sid is a required attribute for JOINT nodes.
Furthermore the "FBX COLLADA exporter" creates node hierarchies without sid's.
It uses the id attribute instead.

I prefer if the OpenCollada library is more tolerant to Collada files,
especially this particular case can be nicely auto fixed,
since i believe that we safely can use the attribue id
here as scoped id when no sid is specified.

note: I am not sure if the last fallback (use attributeData.name as node sid) is a good idea, so this last fallback can also be removed again if it doesn't make sense here.