fix(vtkpolydata): vtkPolyData MTime consists of point and cell MTimes

Similar to VTK C++, vtkPolyData GetMTime() should consider the MTime of its point and cell data,
points and cells arrays

Author: finetjul

Sources/Common/DataModel/DataSet/index.js

@@ -65,6 +65,14 @@ function vtkDataSet(publicAPI, model) {
       model[fieldName].shallowCopy(other.getReferenceByName(fieldName));
     });
   };
+
+  const superGetMTime = publicAPI.getMTime;
+  publicAPI.getMTime = () =>
+    DATASET_FIELDS.reduce(
+      (mTime, fieldName) =>
+        Math.max(mTime, model[fieldName]?.getMTime() ?? mTime),
+      superGetMTime()
+    );
 }
 
 // ----------------------------------------------------------------------------

Sources/Common/DataModel/PointSet/index.js

@@ -36,6 +36,12 @@ function vtkPointSet(publicAPI, model) {
     model.points = vtkPoints.newInstance();
     model.points.shallowCopy(other.getPoints());
   };
+
+  const superGetMTime = publicAPI.getMTime;
+  publicAPI.getMTime = () => {
+    const mTime = superGetMTime();
+    return Math.max(mTime, model.points?.getMTime() ?? mTime);
+  };
 }
 
 // ----------------------------------------------------------------------------

Sources/Common/DataModel/PolyData/index.js

@@ -58,6 +58,13 @@ function vtkPolyData(publicAPI, model) {
     });
   };
 
+  const superGetMTime = publicAPI.getMTime;
+  publicAPI.getMTime = () =>
+    POLYDATA_FIELDS.reduce(
+      (mTime, type) => Math.max(mTime, model[type]?.getMTime() ?? mTime),
+      superGetMTime()
+    );
+
   publicAPI.buildCells = () => {
     // here are the number of cells we have
     const nVerts = publicAPI.getNumberOfVerts();