fix(abstractmapper, selectionnode): fix refactoring

fix refactoring

Author: joannacirillo

Sources/Common/DataModel/SelectionNode/index.js

@@ -1,6 +1,17 @@
 import macro from 'vtk.js/Sources/macros';
 import Constants from 'vtk.js/Sources/Common/DataModel/SelectionNode/Constants';
 
+// ----------------------------------------------------------------------------
+// vtkSelectionNode methods
+// ----------------------------------------------------------------------------
+
+function vtkSelectionNode(publicAPI, model) {
+  // Set our className
+  model.classHierarchy.push('vtkSelectionNode');
+
+  publicAPI.getBounds = () => model.points.getBounds();
+}
+
 // ----------------------------------------------------------------------------
 // Object factory
 // ----------------------------------------------------------------------------
@@ -14,17 +25,6 @@ function defaultValues(initialValues) {
   };
 }
 
-// ----------------------------------------------------------------------------
-// vtkSelectionNode methods
-// ----------------------------------------------------------------------------
-
-function vtkSelectionNode(publicAPI, model) {
-  // Set our className
-  model.classHierarchy.push('vtkSelectionNode');
-
-  publicAPI.getBounds = () => model.points.getBounds();
-}
-
 // ----------------------------------------------------------------------------
 
 export function extend(publicAPI, model, initialValues = {}) {

Sources/Rendering/Core/AbstractMapper/index.js

@@ -22,8 +22,7 @@ function vtkAbstractMapper(publicAPI, model) {
     return false;
   };
 
-  publicAPI.getNumberOfClippingPlanes = () =>
-    model.clippingPlanes ? model.clippingPlanes.length : 0;
+  publicAPI.getNumberOfClippingPlanes = () => model.clippingPlanes.length;
 
   publicAPI.removeAllClippingPlanes = () => {
     model.clippingPlanes.length = 0;
@@ -42,7 +41,8 @@ function vtkAbstractMapper(publicAPI, model) {
   publicAPI.getClippingPlanes = () => model.clippingPlanes;
 
   publicAPI.setClippingPlanes = (planes) => {
-    if (!planes || planes.length === 0) {
+    // Instanciation time
+    if (model.clippingPlanes === undefined) {
       model.clippingPlanes = [];
       return;
     }