ColladaLoader: instance node names now match their associated library nodes name

[shelbyspeegle]

Fixes #13365

[mrdoob]

Thanks!

[thedayofcondor]

Hi,
just updated my project from r87 to r100 and the updated ColladaLoader does not seem to load my model correctly anymore . My model is generated with SketchUp 2018.

In my opinion the original behavious was correct, allowing an instance to have a separate name from the library node (this is the intended behaviour, having a separate "name" property in the node).

As a suggestion, what about allowing a node to have a "name" and an "instanceName" (or a "name" and a "libraryName")?

[mrdoob]

@thedayofcondor It'll be better to create a new issue for this. Please, provide a example file if possible.

[thedayofcondor]

@mrdoob thank you, I created #15658