@onerjs/core 9.0.2 → 9.0.4

package/Meshes/abstractMesh.pure.js

@@ -1537,6 +1537,9 @@ let AbstractMesh = (() => {
                 if (!data) {
                     return null;
                 }
+                const transform = !!((options.applyMorph && this.morphTargetManager) ||
+                    (options.applySkeleton && this.skeleton) ||
+                    (options.applyBakedVertexAnimation && this.bakedVertexAnimationManager?.isEnabled));
                 if (cache) {
                     if (cache._outputData) {
                         cache._outputData.set(data);
@@ -1546,9 +1549,7 @@ let AbstractMesh = (() => {
                     }
                     data = cache._outputData;
                 }
-                else if ((options.applyMorph && this.morphTargetManager) ||
-                    (options.applySkeleton && this.skeleton) ||
-                    (options.applyBakedVertexAnimation && this.bakedVertexAnimationManager?.isEnabled)) {
+                else if (transform) {
                     data = data.slice();
                 }
                 if (options.applyMorph && this.morphTargetManager) {
@@ -1577,7 +1578,14 @@ let AbstractMesh = (() => {
                         _a._ApplySkeleton(data, kind, skeletonMatrices, matricesIndicesData, matricesWeightsData, matricesIndicesExtraData, matricesWeightsExtraData);
                     }
                 }
-                if (options.updatePositionsArray !== false && kind === VertexBuffer.PositionKind) {
+                if (options.updatePositionsArray !== false && kind === VertexBuffer.PositionKind && !transform) {
+                    // untransformed, these are the geometry's positions: picking and collisions use the geometry's points, made when they
+                    // need them and shared by its meshes, so the mesh keeps no Vector3 per vertex of its own. A refresh means the positions
+                    // may have changed in place (updateVerticesDataDirectly, VertexBuffer.update), so those points are made again when needed
+                    this._internalAbstractMeshDataInfo._positions = null;
+                    this.geometry?._resetPointsArrayCache();
+                }
+                else if (options.updatePositionsArray !== false && kind === VertexBuffer.PositionKind) {
                     const positions = this._internalAbstractMeshDataInfo._positions || [];
                     const previousLength = positions.length;
                     positions.length = data.length / 3;