aframe 1.5.0 THREE.Material: parameter 'glslVersion' has value of undefined.

[vincentfretin]

With aframe 1.5.0, we have a warning "THREE.Material: parameter 'glslVersion' has value of undefined."
I didn't quite follow the threejs changes wth glslVersion. I know @mrxz you participated in the discussion around this in aframe repo, you may now how to remove the warning and keep it compatible with both 1.4.2 and 1.5.0.

[mrxz]

This seems to be an oversight in the changes from aframevr/aframe#5328. Quite surprising given the many iterations, discussion and testing that took place on it. I'd assume this warning would've caught the attention at some point, but alas.

Basically A-Frame now falls back to undefined for glslVersion in case of non-raw shaders (shader.js#L58). However the default value in Three.js is null (ShaderMaterial.js#L53). Fix is straightforward, but needs to happen in A-Frame, will create a PR for it shortly

[vincentfretin]

Thanks you! The warning will be fixed in aframe 1.5.1 then.