BufferGeometryUtils: Remove use of old Material.morphTargets property

[willstott101]

Related issue: #23951

Description

In #22169 the Material.morphTargets boolean property was removed, but BufferGeometryUtils.computeMorphedAttributes was left using it.

This means computeMorphedAttributes is not currently applying morph targets to morphed meshes.

[willstott101]

Would it be helpful to open a dedicated issue for this? Does that help with the changelog or anything like that?

[Mugen87]

Not necessary. A PR for tracking the changes is sufficient.

[takahirox]

The change looks good to me. And the second argument material can be removed from _calculateMorphedAttributeData() because it will no longer be used?

three.js/examples/js/utils/BufferGeometryUtils.js

Line 794 in 804c3c1

function _calculateMorphedAttributeData( object, material, attribute, morphAttribute, morphTargetsRelative, a, b, c, modifiedAttributeArray ) {

[willstott101]

Yep right you were - I realised I had edited the JS file not the JSM file, so force pushed. I hope that's not too irritating

[takahirox]

I haven't tested yet because no example uses the method, but looks good to me. Thanks for the update.

[mrdoob]

Thanks!