[FIX] SceneModelTransform: parentTransformId parameter doesn't work about xeokit-sdk

[lzoia-bc]

I tried the latest version 2.6.0 and the parentTransformId doesn't work still.
I suppose the error now is inside SceneModel.createTransform when the transform is created. I see that the parent transform is sent to the SceneModelTransform constructor using the parameter parentTransform, while the SceneModelTransform constructor code is reading cfg.parent.
I think this bug is still present since is not possible to use the parentTransformId feature yet

[xeolabs]

I haven't made a test yet, but seemed an obvious thing to patch - try: https://github.com/xeokit/xeokit-sdk/releases/tag/v2.6.1

[lzoia-bc]

I just tried and I'm getting the following error

TypeError: childTransform._setWorldMatrixDirty is not a function
    at SceneModelTransform._addChildTransform (xeokit-sdk.es.js:80390:1)
    at new SceneModelTransform (xeokit-sdk.es.js:80383:1)
    at SceneModel.createTransform (xeokit-sdk.es.js:83363:1)

[xeolabs]

I can't test right now, but _setWorldMatrixDirty() should probably should be _transformDirty(), if you'd like to try modifying that and testing against your code.

[lzoia-bc]

I just tried with 2.6.2 and I'm getting the following error

TypeError: childTransform._setAABBDirty is not a function
    at SceneModelTransform._addChildTransform (xeokit-sdk.es.js:80391:1)
    at new SceneModelTransform (xeokit-sdk.es.js:80383:1)
    at SceneModel.createTransform (xeokit-sdk.es.js:83363:1)

[lzoia-bc]

I modified locally childTransform._setAABBDirty(); with childTransform._setSubtreeAABBsDirty(this); and it seems to work fine against my code

[lzoia-bc]

I created this PR with the above change #1469

[lzoia-bc]

Re-opened PR #1471

[lzoia-bc]

I tried with 2.6.6 and it's working fine. Maybe it could be worth to update the docs and tell that the parentTransformId is available starting from 2.6.6