handsontable 14.3.0 → 14.4.0-next-f62dd73-20240521

package/core.mjs

@@ -1,5 +1,12 @@
 import "core-js/modules/es.error.cause.js";
 import "core-js/modules/es.array.push.js";
+import "core-js/modules/esnext.set.difference.v2.js";
+import "core-js/modules/esnext.set.intersection.v2.js";
+import "core-js/modules/esnext.set.is-disjoint-from.v2.js";
+import "core-js/modules/esnext.set.is-subset-of.v2.js";
+import "core-js/modules/esnext.set.is-superset-of.v2.js";
+import "core-js/modules/esnext.set.symmetric-difference.v2.js";
+import "core-js/modules/esnext.set.union.v2.js";
 import "core-js/modules/web.immediate.js";
 import { addClass, empty, observeVisibilityChangeOnce, removeClass } from "./helpers/dom/element.mjs";
 import { isFunction } from "./helpers/function.mjs";
@@ -325,7 +332,7 @@ export default function Core(rootElement, userSettings) {
       hiddenIndexesChanged
     } = _ref;
     if (hiddenIndexesChanged) {
-      this.selection.refresh();
+      this.selection.commit();
     }
   };
   this.columnIndexMapper.addLocalHook('cacheUpdated', onIndexMapperCacheUpdate);
@@ -360,7 +367,11 @@ export default function Core(rootElement, userSettings) {
     } else {
       removeClass(this.rootElement, ['ht__selection--rows', 'ht__selection--columns']);
     }
-    this._refreshBorders(null);
+    if (selection.getSelectionSource() !== 'shift') {
+      editorManager.closeEditor(null);
+    }
+    instance.view.render();
+    editorManager.prepareEditor();
   });
   this.selection.addLocalHook('beforeSetFocus', cellCoords => {
     this.runHooks('beforeSelectionFocusSet', cellCoords.row, cellCoords.col);
@@ -371,7 +382,9 @@ export default function Core(rootElement, userSettings) {
     if (!preventScrolling.isTouched() || preventScrolling.isTouched() && !preventScrolling.value) {
       viewportScroller.scrollTo(cellCoords);
     }
-    this._refreshBorders(null);
+    editorManager.closeEditor();
+    instance.view.render();
+    editorManager.prepareEditor();
   });
   this.selection.addLocalHook('afterSelectionFinished', cellRanges => {
     const selectionLayerLevel = cellRanges.length - 1;
@@ -389,8 +402,8 @@ export default function Core(rootElement, userSettings) {
     }
   });
   this.selection.addLocalHook('afterDeselect', () => {
-    editorManager.destroyEditor();
-    this._refreshBorders();
+    editorManager.closeEditor();
+    instance.view.render();
     removeClass(this.rootElement, ['ht__selection--rows', 'ht__selection--columns']);
     this.runHooks('afterDeselect');
   });
@@ -545,39 +558,7 @@ export default function Core(rootElement, userSettings) {
             source,
             mode: insertRowMode
           });
-          if (rowDelta) {
-            const currentSelectedRange = selection.selectedRange.current();
-            const currentFromRange = currentSelectedRange === null || currentSelectedRange === void 0 ? void 0 : currentSelectedRange.from;
-            const currentFromRow = currentFromRange === null || currentFromRange === void 0 ? void 0 : currentFromRange.row;
-            const startVisualRowIndex = instance.toVisualRow(startRowPhysicalIndex);
-            if (selection.isSelectedByCorner()) {
-              selection.selectAll(true, true, {
-                disableHeadersHighlight: true
-              });
-            } else if (isDefined(currentFromRow) && currentFromRow >= startVisualRowIndex) {
-              // Moving the selection (if it exists) downward – it should be applied to the "old" row.
-              // TODO: The logic here should be handled by selection module.
-              const {
-                row: currentToRow,
-                col: currentToColumn
-              } = currentSelectedRange.to;
-              let currentFromColumn = currentFromRange.col;
-
-              // Workaround: headers are not stored inside selection.
-              if (selection.isSelectedByRowHeader()) {
-                currentFromColumn = -1;
-              }
-
-              // Remove from the stack the last added selection as that selection below will be
-              // replaced by new transformed selection.
-              selection.getSelectedRange().pop();
-              // I can't use transforms as they don't work in negative indexes.
-              selection.setRangeStartOnly(instance._createCellCoords(currentFromRow + rowDelta, currentFromColumn), true);
-              selection.setRangeEnd(instance._createCellCoords(currentToRow + rowDelta, currentToColumn)); // will call render() internally
-            } else {
-              instance._refreshBorders(); // it will call render and prepare methods
-            }
-          }
+          selection.shiftRows(instance.toVisualRow(startRowPhysicalIndex), rowDelta);
           break;
         case 'insert_col_start':
         case 'insert_col_end':
@@ -600,38 +581,7 @@ export default function Core(rootElement, userSettings) {
               spliceArray.length += colDelta; // inserts empty (undefined) elements at the end of an array
               Array.prototype.splice.apply(tableMeta.colHeaders, spliceArray); // inserts empty (undefined) elements into the colHeader array
             }
-            const currentSelectedRange = selection.selectedRange.current();
-            const currentFromRange = currentSelectedRange === null || currentSelectedRange === void 0 ? void 0 : currentSelectedRange.from;
-            const currentFromColumn = currentFromRange === null || currentFromRange === void 0 ? void 0 : currentFromRange.col;
-            const startVisualColumnIndex = instance.toVisualColumn(startColumnPhysicalIndex);
-            if (selection.isSelectedByCorner()) {
-              selection.selectAll(true, true, {
-                disableHeadersHighlight: true
-              });
-            } else if (isDefined(currentFromColumn) && currentFromColumn >= startVisualColumnIndex) {
-              // Moving the selection (if it exists) rightward – it should be applied to the "old" column.
-              // TODO: The logic here should be handled by selection module.
-              const {
-                row: currentToRow,
-                col: currentToColumn
-              } = currentSelectedRange.to;
-              let currentFromRow = currentFromRange.row;
-
-              // Workaround: headers are not stored inside selection.
-              if (selection.isSelectedByColumnHeader()) {
-                currentFromRow = -1;
-              }
-
-              // Remove from the stack the last added selection as that selection below will be
-              // replaced by new transformed selection.
-              selection.getSelectedRange().pop();
-
-              // I can't use transforms as they don't work in negative indexes.
-              selection.setRangeStartOnly(instance._createCellCoords(currentFromRow, currentFromColumn + colDelta), true);
-              selection.setRangeEnd(instance._createCellCoords(currentToRow, currentToColumn + colDelta)); // will call render() internally
-            } else {
-              instance._refreshBorders(); // it will call render and prepare methods
-            }
+            selection.shiftColumns(instance.toVisualColumn(startColumnPhysicalIndex), colDelta);
           }
           break;
         case 'remove_row':
@@ -656,7 +606,22 @@ export default function Core(rootElement, userSettings) {
               if (!wasRemoved) {
                 return;
               }
+              if (selection.isSelected()) {
+                const {
+                  row
+                } = instance.getSelectedRangeLast().highlight;
+                if (row >= groupIndex && row <= groupIndex + groupAmount - 1) {
+                  editorManager.closeEditor(true);
+                }
+              }
               const totalRows = instance.countRows();
+              if (totalRows === 0) {
+                selection.deselect();
+              } else if (source === 'ContextMenu.removeRow') {
+                selection.refresh();
+              } else {
+                selection.shiftRows(groupIndex, -groupAmount);
+              }
               const fixedRowsTop = tableMeta.fixedRowsTop;
               if (fixedRowsTop >= calcIndex + 1) {
                 tableMeta.fixedRowsTop -= Math.min(groupAmount, fixedRowsTop - calcIndex);
@@ -673,8 +638,6 @@ export default function Core(rootElement, userSettings) {
           } else {
             removeRow([[index, amount]]);
           }
-          grid.adjustRowsAndCols();
-          instance._refreshBorders(); // it will call render and prepare methods
           break;
         case 'remove_col':
           const removeCol = indexes => {
@@ -699,6 +662,22 @@ export default function Core(rootElement, userSettings) {
               if (!wasRemoved) {
                 return;
               }
+              if (selection.isSelected()) {
+                const {
+                  col
+                } = instance.getSelectedRangeLast().highlight;
+                if (col >= groupIndex && col <= groupIndex + groupAmount - 1) {
+                  editorManager.closeEditor(true);
+                }
+              }
+              const totalColumns = instance.countCols();
+              if (totalColumns === 0) {
+                selection.deselect();
+              } else if (source === 'ContextMenu.removeColumn') {
+                selection.refresh();
+              } else {
+                selection.shiftColumns(groupIndex, -groupAmount);
+              }
               const fixedColumnsStart = tableMeta.fixedColumnsStart;
               if (fixedColumnsStart >= calcIndex + 1) {
                 tableMeta.fixedColumnsStart -= Math.min(groupAmount, fixedColumnsStart - calcIndex);
@@ -717,13 +696,11 @@ export default function Core(rootElement, userSettings) {
           } else {
             removeCol([[index, amount]]);
           }
-          grid.adjustRowsAndCols();
-          instance._refreshBorders(); // it will call render and prepare methods
-
           break;
         default:
           throw new Error(`There is no such action "${action}"`);
       }
+      instance.view.render();
       if (!keepEmptyRows) {
         grid.adjustRowsAndCols(); // makes sure that we did not add rows that will be removed in next refresh
       }
@@ -738,9 +715,6 @@ export default function Core(rootElement, userSettings) {
       const minSpareRows = tableMeta.minSpareRows;
       const minCols = tableMeta.minCols;
       const minSpareCols = tableMeta.minSpareCols;
-      if (instance.countRows() === 0 && instance.countCols() === 0) {
-        selection.deselect();
-      }
       if (minRows) {
         // should I add empty rows to data source to meet minRows?
         const nrOfRows = instance.countRows();
@@ -799,55 +773,6 @@ export default function Core(rootElement, userSettings) {
           });
         }
       }
-      if (selection.isSelected()) {
-        const rowCount = instance.countRows();
-        const colCount = instance.countCols();
-        arrayEach(selection.selectedRange, range => {
-          let selectionChanged = false;
-          let fromRow = range.from.row;
-          let fromCol = range.from.col;
-          let toRow = range.to.row;
-          let toCol = range.to.col;
-
-          // if selection is outside, move selection to last row
-          if (fromRow > rowCount - 1) {
-            fromRow = rowCount - 1;
-            selectionChanged = true;
-            if (toRow > fromRow) {
-              toRow = fromRow;
-            }
-          } else if (toRow > rowCount - 1) {
-            toRow = rowCount - 1;
-            selectionChanged = true;
-            if (fromRow > toRow) {
-              fromRow = toRow;
-            }
-          }
-          // if selection is outside, move selection to last row
-          if (fromCol > colCount - 1) {
-            fromCol = colCount - 1;
-            selectionChanged = true;
-            if (toCol > fromCol) {
-              toCol = fromCol;
-            }
-          } else if (toCol > colCount - 1) {
-            toCol = colCount - 1;
-            selectionChanged = true;
-            if (fromCol > toCol) {
-              fromCol = toCol;
-            }
-          }
-          if (selectionChanged) {
-            if (fromCol < 0) {
-              instance.selectRows(fromRow, toRow, fromCol);
-            } else if (fromRow < 0) {
-              instance.selectColumns(fromCol, toCol, fromRow);
-            } else {
-              instance.selectCell(fromRow, fromCol, toRow, toCol);
-            }
-          }
-        });
-      }
       if (instance.view) {
         instance.view.adjustElementsSize();
       }
@@ -1192,8 +1117,18 @@ export default function Core(rootElement, userSettings) {
     };
     for (let i = changes.length - 1; i >= 0; i--) {
       const [row, prop,, newValue] = changes[i];
-      const col = datamap.propToCol(prop);
-      const cellProperties = instance.getCellMeta(row, col);
+      const visualCol = datamap.propToCol(prop);
+      let cellProperties;
+      if (Number.isInteger(visualCol)) {
+        cellProperties = instance.getCellMeta(row, visualCol);
+      } else {
+        // If there's no requested visual column, we can use the table meta as the cell properties when retrieving
+        // the cell validator.
+        cellProperties = {
+          ...Object.getPrototypeOf(tableMeta),
+          ...tableMeta
+        };
+      }
       if (cellProperties.type === 'numeric' && typeof newValue === 'string' && isNumericLike(newValue)) {
         changes[i][3] = getParsedNumber(newValue);
       }
@@ -1284,9 +1219,9 @@ export default function Core(rootElement, userSettings) {
     instance.forceFullRender = true; // used when data was changed
     grid.adjustRowsAndCols();
     instance.runHooks('beforeChangeRender', changes, source);
-    editorManager.lockEditor();
-    instance._refreshBorders(null);
-    editorManager.unlockEditor();
+    editorManager.closeEditor();
+    instance.view.render();
+    editorManager.prepareEditor();
     instance.view.adjustElementsSize();
     instance.runHooks('afterChange', changes, source || 'edit');
     const activeEditor = instance.getActiveEditor();
@@ -1497,6 +1432,9 @@ export default function Core(rootElement, userSettings) {
     for (i = 0, ilen = input.length; i < ilen; i++) {
       changes.push([input[i][0], input[i][1], dataSource.getAtCell(this.toPhysicalRow(input[i][0]), input[i][1]), input[i][2]]);
     }
+
+    // TODO: I don't think `prop` should be used as `changeSource` here, but removing it would be a breaking change.
+    // We should remove it with the next major release.
     if (!changeSource && typeof row === 'object') {
       changeSource = prop;
     }
@@ -1563,7 +1501,11 @@ export default function Core(rootElement, userSettings) {
   this.destroyEditor = function () {
     let revertOriginal = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : false;
     let prepareEditorIfNeeded = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : true;
-    instance._refreshBorders(revertOriginal, prepareEditorIfNeeded);
+    editorManager.closeEditor(revertOriginal);
+    instance.view.render();
+    if (prepareEditorIfNeeded && selection.isSelected()) {
+      editorManager.prepareEditor();
+    }
   };
 
   /**
@@ -1826,7 +1768,7 @@ export default function Core(rootElement, userSettings) {
       if (this.renderCall) {
         this.render();
       } else {
-        this._refreshBorders(null);
+        instance.view.render();
       }
     }
   };
@@ -1847,9 +1789,7 @@ export default function Core(rootElement, userSettings) {
       this.forceFullRender = true; // used when data was changed
 
       if (!this.isRenderSuspended()) {
-        editorManager.lockEditor();
-        this._refreshBorders(null);
-        editorManager.unlockEditor();
+        instance.view.render();
       }
     }
   };
@@ -2116,6 +2056,7 @@ export default function Core(rootElement, userSettings) {
       instance.columnIndexMapper.fitToLength(this.getInitialColumnCount());
       instance.rowIndexMapper.fitToLength(this.countSourceRows());
       grid.adjustRowsAndCols();
+      selection.refresh();
     }, {
       hotInstance: instance,
       dataMap: datamap,
@@ -2156,6 +2097,7 @@ export default function Core(rootElement, userSettings) {
       metaManager.clearCellsCache();
       instance.initIndexMappers();
       grid.adjustRowsAndCols();
+      selection.refresh();
       if (firstRun) {
         firstRun = [null, 'loadData'];
       }
@@ -2443,10 +2385,8 @@ export default function Core(rootElement, userSettings) {
     grid.adjustRowsAndCols();
     if (instance.view && !firstRun) {
       instance.forceFullRender = true; // used when data was changed
-      editorManager.lockEditor();
-      instance._refreshBorders(null);
+      instance.view.render();
       instance.view._wt.wtOverlays.adjustElementsSize();
-      editorManager.unlockEditor();
     }
     if (!init && instance.view && (currentHeight === '' || height === '' || height === undefined) && currentHeight !== height) {
       instance.view._wt.wtOverlays.updateMainScrollableElements();
@@ -2598,7 +2538,7 @@ export default function Core(rootElement, userSettings) {
       }
       renderableRowIndex = this.rowIndexMapper.getRenderableFromVisualIndex(row);
     }
-    if (renderableRowIndex === null || renderableColumnIndex === null) {
+    if (renderableRowIndex === null || renderableColumnIndex === null || renderableRowIndex === undefined || renderableColumnIndex === undefined) {
       return null;
     }
     return instance.view.getCellAtCoords(instance._createCellCoords(renderableRowIndex, renderableColumnIndex), topmost);
@@ -4344,13 +4284,17 @@ export default function Core(rootElement, userSettings) {
    * @see Hooks#add
    * @param {string} key Hook name (see {@link Hooks}).
    * @param {Function|Array} callback Function or array of functions.
+   * @param {number} [orderIndex] Order index of the callback.
+   *                              If > 0, the callback will be added after the others, for example, with an index of 1, the callback will be added before the ones with an index of 2, 3, etc., but after the ones with an index of 0 and lower.
+   *                              If < 0, the callback will be added before the others, for example, with an index of -1, the callback will be added after the ones with an index of -2, -3, etc., but before the ones with an index of 0 and higher.
+   *                              If 0 or no order index is provided, the callback will be added between the "negative" and "positive" indexes.
    * @example
    * ```js
    * hot.addHook('beforeInit', myCallback);
    * ```
    */
-  this.addHook = function (key, callback) {
-    Hooks.getSingleton().add(key, callback, instance);
+  this.addHook = function (key, callback, orderIndex) {
+    Hooks.getSingleton().add(key, callback, instance, orderIndex);
   };
 
   /**
@@ -4381,13 +4325,17 @@ export default function Core(rootElement, userSettings) {
    * @see Hooks#once
    * @param {string} key Hook name (see {@link Hooks}).
    * @param {Function|Array} callback Function or array of functions.
+   * @param {number} [orderIndex] Order index of the callback.
+   *                              If > 0, the callback will be added after the others, for example, with an index of 1, the callback will be added before the ones with an index of 2, 3, etc., but after the ones with an index of 0 and lower.
+   *                              If < 0, the callback will be added before the others, for example, with an index of -1, the callback will be added after the ones with an index of -2, -3, etc., but before the ones with an index of 0 and higher.
+   *                              If 0 or no order index is provided, the callback will be added between the "negative" and "positive" indexes.
    * @example
    * ```js
    * hot.addHookOnce('beforeInit', myCallback);
    * ```
    */
-  this.addHookOnce = function (key, callback) {
-    Hooks.getSingleton().once(key, callback, instance);
+  this.addHookOnce = function (key, callback, orderIndex) {
+    Hooks.getSingleton().once(key, callback, instance, orderIndex);
   };
 
   /**
@@ -4524,23 +4472,6 @@ export default function Core(rootElement, userSettings) {
     });
   };
 
-  /**
-   * Refresh selection borders. This is temporary method relic after selection rewrite.
-   *
-   * @private
-   * @param {boolean} [revertOriginal=false] If `true`, the previous value will be restored. Otherwise, the edited value will be saved.
-   * @param {boolean} [prepareEditorIfNeeded=true] If `true` the editor under the selected cell will be prepared to open.
-   */
-  this._refreshBorders = function () {
-    let revertOriginal = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : false;
-    let prepareEditorIfNeeded = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : true;
-    editorManager.destroyEditor(revertOriginal);
-    instance.view.render();
-    if (prepareEditorIfNeeded && selection.isSelected()) {
-      editorManager.prepareEditor();
-    }
-  };
-
   /**
    * Gets the instance of the EditorManager.
    *

package/dataMap/dataMap.js

@@ -671,17 +671,20 @@ class DataMap {
     dataRow = isNaN(modifiedRowData) ? modifiedRowData : dataRow;
     //
 
+    const {
+      dataDotNotation
+    } = this.hot.getSettings();
     let value = null;
 
     // try to get value under property `prop` (includes dot)
     if (dataRow && dataRow.hasOwnProperty && (0, _object.hasOwnProperty)(dataRow, prop)) {
       value = dataRow[prop];
-    } else if (typeof prop === 'string' && prop.indexOf('.') > -1) {
-      const sliced = prop.split('.');
+    } else if (dataDotNotation && typeof prop === 'string' && prop.indexOf('.') > -1) {
       let out = dataRow;
       if (!out) {
         return null;
       }
+      const sliced = prop.split('.');
       for (let i = 0, ilen = sliced.length; i < ilen; i++) {
         out = out[sliced[i]];
         if (typeof out === 'undefined') {
@@ -739,15 +742,18 @@ class DataMap {
         newValue = valueHolder.value;
       }
     }
+    const {
+      dataDotNotation
+    } = this.hot.getSettings();
 
     // try to set value under property `prop` (includes dot)
     if (dataRow && dataRow.hasOwnProperty && (0, _object.hasOwnProperty)(dataRow, prop)) {
       dataRow[prop] = newValue;
-    } else if (typeof prop === 'string' && prop.indexOf('.') > -1) {
-      const sliced = prop.split('.');
+    } else if (dataDotNotation && typeof prop === 'string' && prop.indexOf('.') > -1) {
       let out = dataRow;
       let i = 0;
       let ilen;
+      const sliced = prop.split('.');
       for (i = 0, ilen = sliced.length - 1; i < ilen; i++) {
         if (typeof out[sliced[i]] === 'undefined') {
           out[sliced[i]] = {};

package/dataMap/dataMap.mjs

@@ -667,17 +667,20 @@ class DataMap {
     dataRow = isNaN(modifiedRowData) ? modifiedRowData : dataRow;
     //
 
+    const {
+      dataDotNotation
+    } = this.hot.getSettings();
     let value = null;
 
     // try to get value under property `prop` (includes dot)
     if (dataRow && dataRow.hasOwnProperty && hasOwnProperty(dataRow, prop)) {
       value = dataRow[prop];
-    } else if (typeof prop === 'string' && prop.indexOf('.') > -1) {
-      const sliced = prop.split('.');
+    } else if (dataDotNotation && typeof prop === 'string' && prop.indexOf('.') > -1) {
       let out = dataRow;
       if (!out) {
         return null;
       }
+      const sliced = prop.split('.');
       for (let i = 0, ilen = sliced.length; i < ilen; i++) {
         out = out[sliced[i]];
         if (typeof out === 'undefined') {
@@ -735,15 +738,18 @@ class DataMap {
         newValue = valueHolder.value;
       }
     }
+    const {
+      dataDotNotation
+    } = this.hot.getSettings();
 
     // try to set value under property `prop` (includes dot)
     if (dataRow && dataRow.hasOwnProperty && hasOwnProperty(dataRow, prop)) {
       dataRow[prop] = newValue;
-    } else if (typeof prop === 'string' && prop.indexOf('.') > -1) {
-      const sliced = prop.split('.');
+    } else if (dataDotNotation && typeof prop === 'string' && prop.indexOf('.') > -1) {
       let out = dataRow;
       let i = 0;
       let ilen;
+      const sliced = prop.split('.');
       for (i = 0, ilen = sliced.length - 1; i < ilen; i++) {
         if (typeof out[sliced[i]] === 'undefined') {
           out[sliced[i]] = {};

package/dataMap/dataSource.js

@@ -110,6 +110,9 @@ class DataSource {
   getAtRow(row, startColumn, endColumn) {
     let toArray = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : false;
     const getAllProps = startColumn === undefined && endColumn === undefined;
+    const {
+      dataDotNotation
+    } = this.hot.getSettings();
     let dataRow = null;
     let newDataRow = null;
     dataRow = this.modifyRowData(row);
@@ -140,14 +143,21 @@ class DataSource {
             const cellValue = this.getAtPhysicalCell(row, prop, dataRow);
             if (toArray) {
               newDataRow.push(cellValue);
-            } else {
+            } else if (dataDotNotation) {
               (0, _object.setProperty)(newDataRow, prop, cellValue);
+            } else {
+              newDataRow[prop] = cellValue;
             }
           }
         });
       } else {
         (0, _object.objectEach)(dataRow, (value, prop) => {
-          (0, _object.setProperty)(newDataRow, prop, this.getAtPhysicalCell(row, prop, dataRow));
+          const cellValue = this.getAtPhysicalCell(row, prop, dataRow);
+          if (dataDotNotation) {
+            (0, _object.setProperty)(newDataRow, prop, cellValue);
+          } else {
+            newDataRow[prop] = cellValue;
+          }
         });
       }
     }
@@ -194,7 +204,10 @@ class DataSource {
     let result = null;
     if (dataRow) {
       if (typeof column === 'string') {
-        result = (0, _object.getProperty)(dataRow, column);
+        const {
+          dataDotNotation
+        } = this.hot.getSettings();
+        result = dataDotNotation ? (0, _object.getProperty)(dataRow, column) : dataRow[column];
       } else if (typeof column === 'function') {
         result = column(dataRow);
       } else {

package/dataMap/dataSource.mjs

@@ -107,6 +107,9 @@ class DataSource {
   getAtRow(row, startColumn, endColumn) {
     let toArray = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : false;
     const getAllProps = startColumn === undefined && endColumn === undefined;
+    const {
+      dataDotNotation
+    } = this.hot.getSettings();
     let dataRow = null;
     let newDataRow = null;
     dataRow = this.modifyRowData(row);
@@ -137,14 +140,21 @@ class DataSource {
             const cellValue = this.getAtPhysicalCell(row, prop, dataRow);
             if (toArray) {
               newDataRow.push(cellValue);
-            } else {
+            } else if (dataDotNotation) {
               setProperty(newDataRow, prop, cellValue);
+            } else {
+              newDataRow[prop] = cellValue;
             }
           }
         });
       } else {
         objectEach(dataRow, (value, prop) => {
-          setProperty(newDataRow, prop, this.getAtPhysicalCell(row, prop, dataRow));
+          const cellValue = this.getAtPhysicalCell(row, prop, dataRow);
+          if (dataDotNotation) {
+            setProperty(newDataRow, prop, cellValue);
+          } else {
+            newDataRow[prop] = cellValue;
+          }
         });
       }
     }
@@ -191,7 +201,10 @@ class DataSource {
     let result = null;
     if (dataRow) {
       if (typeof column === 'string') {
-        result = getProperty(dataRow, column);
+        const {
+          dataDotNotation
+        } = this.hot.getSettings();
+        result = dataDotNotation ? getProperty(dataRow, column) : dataRow[column];
       } else if (typeof column === 'function') {
         result = column(dataRow);
       } else {

package/dataMap/metaManager/lazyFactoryMap.js

@@ -3,6 +3,13 @@
 exports.__esModule = true;
 require("core-js/modules/es.error.cause.js");
 require("core-js/modules/es.array.push.js");
+require("core-js/modules/esnext.set.difference.v2.js");
+require("core-js/modules/esnext.set.intersection.v2.js");
+require("core-js/modules/esnext.set.is-disjoint-from.v2.js");
+require("core-js/modules/esnext.set.is-subset-of.v2.js");
+require("core-js/modules/esnext.set.is-superset-of.v2.js");
+require("core-js/modules/esnext.set.symmetric-difference.v2.js");
+require("core-js/modules/esnext.set.union.v2.js");
 var _array = require("../../helpers/array");
 var _utils = require("./utils");
 function _defineProperty(obj, key, value) { key = _toPropertyKey(key); if (key in obj) { Object.defineProperty(obj, key, { value: value, enumerable: true, configurable: true, writable: true }); } else { obj[key] = value; } return obj; }

package/dataMap/metaManager/lazyFactoryMap.mjs

@@ -1,5 +1,12 @@
 import "core-js/modules/es.error.cause.js";
 import "core-js/modules/es.array.push.js";
+import "core-js/modules/esnext.set.difference.v2.js";
+import "core-js/modules/esnext.set.intersection.v2.js";
+import "core-js/modules/esnext.set.is-disjoint-from.v2.js";
+import "core-js/modules/esnext.set.is-subset-of.v2.js";
+import "core-js/modules/esnext.set.is-superset-of.v2.js";
+import "core-js/modules/esnext.set.symmetric-difference.v2.js";
+import "core-js/modules/esnext.set.union.v2.js";
 function _defineProperty(obj, key, value) { key = _toPropertyKey(key); if (key in obj) { Object.defineProperty(obj, key, { value: value, enumerable: true, configurable: true, writable: true }); } else { obj[key] = value; } return obj; }
 function _toPropertyKey(t) { var i = _toPrimitive(t, "string"); return "symbol" == typeof i ? i : i + ""; }
 function _toPrimitive(t, r) { if ("object" != typeof t || !t) return t; var e = t[Symbol.toPrimitive]; if (void 0 !== e) { var i = e.call(t, r || "default"); if ("object" != typeof i) return i; throw new TypeError("@@toPrimitive must return a primitive value."); } return ("string" === r ? String : Number)(t); }