handsontable 0.0.0-next-3be5fb5-20230427 → 0.0.0-next-aad186f-20230428

package/plugins/formulas/formulas.mjs

@@ -76,10 +76,10 @@ import { getEngineSettingsWithOverrides, haveEngineSettingsChanged } from "./eng
 import { isArrayOfArrays } from "../../helpers/data.mjs";
 import { toUpperCaseFirst } from "../../helpers/string.mjs";
 import Hooks from "../../pluginHooks.mjs";
+import IndexSyncer from "./indexSyncer/index.mjs";
 export var PLUGIN_KEY = 'formulas';
 export var SETTING_KEYS = ['maxRows', 'maxColumns', 'language'];
 export var PLUGIN_PRIORITY = 260;
-var ROW_MOVE_UNDO_REDO_NAME = 'row_move';
 Hooks.getSingleton().register('afterNamedExpressionAdded');
 Hooks.getSingleton().register('afterNamedExpressionRemoved');
 Hooks.getSingleton().register('afterSheetAdded');
@@ -185,6 +185,24 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
      * @type {string|null}
      */
     _defineProperty(_assertThisInitialized(_this), "sheetName", null);
+    /**
+     * Index synchronizer responsible for manipulating with some general options related to indexes synchronization.
+     *
+     * @type {IndexSyncer|null}
+     */
+    _defineProperty(_assertThisInitialized(_this), "indexSyncer", null);
+    /**
+     * Index synchronizer responsible for syncing the order of HOT and HF's data for the axis of the rows.
+     *
+     * @type {AxisSyncer|null}
+     */
+    _defineProperty(_assertThisInitialized(_this), "rowAxisSyncer", null);
+    /**
+     * Index synchronizer responsible for syncing the order of HOT and HF's data for the axis of the columns.
+     *
+     * @type {AxisSyncer|null}
+     */
+    _defineProperty(_assertThisInitialized(_this), "columnAxisSyncer", null);
     return _this;
   }
   _createClass(Formulas, [{
@@ -292,6 +310,41 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
       this.addHook('afterRemoveCol', function () {
         return _this8.onAfterRemoveCol.apply(_this8, arguments);
       });
+      this.indexSyncer = new IndexSyncer(this.hot.rowIndexMapper, this.hot.columnIndexMapper, function (postponedAction) {
+        _this8.hot.addHookOnce('init', function () {
+          // Engine is initialized after executing callback to `afterLoadData` hook. Thus, some actions on indexes should
+          // be postponed.
+          postponedAction();
+        });
+      });
+      this.rowAxisSyncer = this.indexSyncer.getForAxis('row');
+      this.columnAxisSyncer = this.indexSyncer.getForAxis('column');
+      this.hot.addHook('afterRowSequenceChange', this.rowAxisSyncer.getIndexesChangeSyncMethod());
+      this.hot.addHook('afterColumnSequenceChange', this.columnAxisSyncer.getIndexesChangeSyncMethod());
+      this.hot.addHook('beforeRowMove', function (movedRows, finalIndex, _, movePossible) {
+        _this8.rowAxisSyncer.storeMovesInformation(movedRows, finalIndex, movePossible);
+      });
+      this.hot.addHook('beforeColumnMove', function (movedColumns, finalIndex, _, movePossible) {
+        _this8.columnAxisSyncer.storeMovesInformation(movedColumns, finalIndex, movePossible);
+      });
+      this.hot.addHook('afterRowMove', function (movedRows, finalIndex, dropIndex, movePossible, orderChanged) {
+        _this8.rowAxisSyncer.calculateAndSyncMoves(movePossible, orderChanged);
+      });
+      this.hot.addHook('afterColumnMove', function (movedColumns, finalIndex, dropIndex, movePossible, orderChanged) {
+        _this8.columnAxisSyncer.calculateAndSyncMoves(movePossible, orderChanged);
+      });
+      this.hot.addHook('beforeColumnFreeze', function (column, freezePerformed) {
+        _this8.columnAxisSyncer.storeMovesInformation([column], _this8.hot.getSettings().fixedColumnsStart, freezePerformed);
+      });
+      this.hot.addHook('afterColumnFreeze', function (_, freezePerformed) {
+        _this8.columnAxisSyncer.calculateAndSyncMoves(freezePerformed, freezePerformed);
+      });
+      this.hot.addHook('beforeColumnUnfreeze', function (column, unfreezePerformed) {
+        _this8.columnAxisSyncer.storeMovesInformation([column], _this8.hot.getSettings().fixedColumnsStart - 1, unfreezePerformed);
+      });
+      this.hot.addHook('afterColumnUnfreeze', function (_, unfreezePerformed) {
+        _this8.columnAxisSyncer.calculateAndSyncMoves(unfreezePerformed, unfreezePerformed);
+      });
 
       // TODO: Actions related to overwriting dates from HOT format to HF default format are done as callback to this
       // hook, because some hooks, such as `afterLoadData` doesn't have information about composed cell properties.
@@ -301,22 +354,22 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
       });
 
       // Handling undo actions on data just using HyperFormula's UndoRedo mechanism
-      this.addHook('beforeUndo', function (action) {
-        // TODO: Move action isn't handled by HyperFormula.
-        if ((action === null || action === void 0 ? void 0 : action.actionType) === ROW_MOVE_UNDO_REDO_NAME) {
-          return;
-        }
+      this.addHook('beforeUndo', function () {
+        _this8.indexSyncer.setPerformUndo(true);
         _this8.engine.undo();
       });
 
       // Handling redo actions on data just using HyperFormula's UndoRedo mechanism
-      this.addHook('beforeRedo', function (action) {
-        // TODO: Move action isn't handled by HyperFormula.
-        if ((action === null || action === void 0 ? void 0 : action.actionType) === ROW_MOVE_UNDO_REDO_NAME) {
-          return;
-        }
+      this.addHook('beforeRedo', function () {
+        _this8.indexSyncer.setPerformRedo(true);
         _this8.engine.redo();
       });
+      this.addHook('afterUndo', function () {
+        _this8.indexSyncer.setPerformUndo(false);
+      });
+      this.addHook('afterUndo', function () {
+        _this8.indexSyncer.setPerformRedo(false);
+      });
       this.addHook('afterDetachChild', function () {
         return _this8.onAfterDetachChild.apply(_this8, arguments);
       });
@@ -401,63 +454,6 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
       _get(_getPrototypeOf(Formulas.prototype), "destroy", this).call(this);
     }
 
-    /**
-     * Helper function for `toPhysicalRowPosition` and `toPhysicalColumnPosition`.
-     *
-     * @private
-     * @param {number} visualIndex Visual entry index.
-     * @param {number} physicalIndex Physical entry index.
-     * @param {number} entriesCount Visual entries count.
-     * @param {number} sourceEntriesCount Source entries count.
-     * @param {boolean} contained `true` if it should return only indexes within boundaries of the table (basically
-     * `toPhysical` alias.
-     * @returns {*}
-     */
-  }, {
-    key: "getPhysicalIndexPosition",
-    value: function getPhysicalIndexPosition(visualIndex, physicalIndex, entriesCount, sourceEntriesCount, contained) {
-      if (!contained) {
-        if (visualIndex >= entriesCount) {
-          return sourceEntriesCount + (visualIndex - entriesCount);
-        }
-      }
-      return physicalIndex;
-    }
-
-    /**
-     * Returns the physical row index. The difference between this and Core's `toPhysical` is that it doesn't return
-     * `null` on rows with indexes higher than the number of rows.
-     *
-     * @private
-     * @param {number} row Visual row index.
-     * @param {boolean} [contained] `true` if it should return only indexes within boundaries of the table (basically
-     * `toPhysical` alias.
-     * @returns {number} The physical row index.
-     */
-  }, {
-    key: "toPhysicalRowPosition",
-    value: function toPhysicalRowPosition(row) {
-      var contained = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : false;
-      return this.getPhysicalIndexPosition(row, this.hot.toPhysicalRow(row), this.hot.countRows(), this.hot.countSourceRows(), contained);
-    }
-
-    /**
-     * Returns the physical column index. The difference between this and Core's `toPhysical` is that it doesn't return
-     * `null` on columns with indexes higher than the number of columns.
-     *
-     * @private
-     * @param {number} column Visual column index.
-     * @param {boolean} [contained] `true` if it should return only indexes within boundaries of the table (basically
-     * `toPhysical` alias.
-     * @returns {number} The physical column index.
-     */
-  }, {
-    key: "toPhysicalColumnPosition",
-    value: function toPhysicalColumnPosition(column) {
-      var contained = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : false;
-      return this.getPhysicalIndexPosition(column, this.hot.toPhysicalColumn(column), this.hot.countCols(), this.hot.countSourceCols(), contained);
-    }
-
     /**
      * Add a sheet to the shared HyperFormula instance.
      *
@@ -528,8 +524,8 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
       if (physicalRow !== null && physicalColumn !== null) {
         return this.engine.getCellType({
           sheet: sheet,
-          row: physicalRow,
-          col: physicalColumn
+          row: this.rowAxisSyncer.getHfIndexFromVisualIndex(row),
+          col: this.columnAxisSyncer.getHfIndexFromVisualIndex(column)
         });
       } else {
         // Should return `EMPTY` when out of bounds (according to the test cases).
@@ -549,15 +545,10 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
     key: "isFormulaCellType",
     value: function isFormulaCellType(row, column) {
       var sheet = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : this.sheetId;
-      var physicalRow = this.hot.toPhysicalRow(row);
-      var physicalColumn = this.hot.toPhysicalColumn(column);
-      if (physicalRow === null || physicalColumn === null) {
-        return false;
-      }
       return this.engine.doesCellHaveFormula({
         sheet: sheet,
-        row: physicalRow,
-        col: physicalColumn
+        row: this.rowAxisSyncer.getHfIndexFromVisualIndex(row),
+        col: this.columnAxisSyncer.getHfIndexFromVisualIndex(column)
       });
     }
 
@@ -622,11 +613,9 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
         var _ref8 = (_change$address3 = change.address) !== null && _change$address3 !== void 0 ? _change$address3 : {},
           row = _ref8.row,
           col = _ref8.col;
-        var visualRow = isDefined(row) ? _this12.hot.toVisualRow(row) : null;
-        var visualColumn = isDefined(col) ? _this12.hot.toVisualColumn(col) : null;
 
         // Don't try to validate cells outside of the visual part of the table.
-        if (visualRow === null || visualColumn === null) {
+        if (isDefined(row) === false || isDefined(col) === false || row >= _this12.hot.countRows() || col >= _this12.hot.countCols()) {
           return;
         }
 
@@ -645,7 +634,7 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
           }
 
           // It will just re-render certain cell when necessary.
-          boundHot.validateCell(boundHot.getDataAtCell(visualRow, visualColumn), boundHot.getCellMeta(visualRow, visualColumn), function () {});
+          boundHot.validateCell(boundHot.getDataAtCell(row, col), boundHot.getCellMeta(row, col), function () {});
         }
       });
     }
@@ -663,8 +652,8 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
     key: "syncChangeWithEngine",
     value: function syncChangeWithEngine(row, column, newValue) {
       var address = {
-        row: this.toPhysicalRowPosition(row),
-        col: this.toPhysicalColumnPosition(column),
+        row: this.rowAxisSyncer.getHfIndexFromVisualIndex(row),
+        col: this.columnAxisSyncer.getHfIndexFromVisualIndex(column),
         sheet: this.sheetId
       };
       if (!this.engine.isItPossibleToSetCellContents(address)) {
@@ -700,8 +689,8 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
       var visualColumn = this.hot.propToCol(prop);
       if (this.isFormulaCellType(visualRow, visualColumn)) {
         var address = {
-          row: this.hot.toPhysicalRow(visualRow),
-          col: this.hot.toPhysicalColumn(visualColumn),
+          row: this.rowAxisSyncer.getHfIndexFromVisualIndex(visualRow),
+          col: this.columnAxisSyncer.getHfIndexFromVisualIndex(visualColumn),
           sheet: this.sheetId
         };
         var cellMeta = this.hot.getCellMeta(visualRow, visualColumn);
@@ -855,6 +844,7 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
         if (this.engine.isItPossibleToReplaceSheetContent(this.sheetId, sourceDataArray)) {
           _classPrivateFieldSet(this, _internalOperationPending, true);
           var dependentCells = this.engine.setSheetContent(this.sheetId, sourceDataArray);
+          this.indexSyncer.setupSyncEndpoint(this.engine, this.sheetId);
           this.renderDependentSheets(dependentCells);
           _classPrivateFieldSet(this, _internalOperationPending, false);
         }
@@ -867,24 +857,27 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
      * `modifyData` hook callback.
      *
      * @private
-     * @param {number} row Physical row height.
-     * @param {number} column Physical column index.
+     * @param {number} physicalRow Physical row index.
+     * @param {number} visualColumn Visual column index.
      * @param {object} valueHolder Object which contains original value which can be modified by overwriting `.value`
      *   property.
      * @param {string} ioMode String which indicates for what operation hook is fired (`get` or `set`).
      */
   }, {
     key: "onModifyData",
-    value: function onModifyData(row, column, valueHolder, ioMode) {
+    value: function onModifyData(physicalRow, visualColumn, valueHolder, ioMode) {
       if (ioMode !== 'get' || _classPrivateFieldGet(this, _internalOperationPending) || this.sheetName === null || !this.engine.doesSheetExist(this.sheetName)) {
         return;
       }
-      var visualRow = this.hot.toVisualRow(row);
+      var visualRow = this.hot.toVisualRow(physicalRow);
+      if (visualRow === null || visualColumn === null) {
+        return;
+      }
 
       // `column` is here as visual index because of inconsistencies related to hook execution in `src/dataMap`.
-      var isFormulaCellType = this.isFormulaCellType(visualRow, column);
+      var isFormulaCellType = this.isFormulaCellType(visualRow, visualColumn);
       if (!isFormulaCellType) {
-        var cellType = this.getCellType(visualRow, column);
+        var cellType = this.getCellType(visualRow, visualColumn);
         if (cellType !== 'ARRAY') {
           if (isEscapedFormulaExpression(valueHolder.value)) {
             valueHolder.value = unescapeFormulaExpression(valueHolder.value);
@@ -892,15 +885,13 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
           return;
         }
       }
-
-      // `toPhysicalColumn` is here because of inconsistencies related to hook execution in `DataMap`.
       var address = {
-        row: row,
-        col: this.toPhysicalColumnPosition(column),
+        row: this.rowAxisSyncer.getHfIndexFromVisualIndex(visualRow),
+        col: this.columnAxisSyncer.getHfIndexFromVisualIndex(visualColumn),
         sheet: this.sheetId
       };
       var cellValue = this.engine.getCellValue(address); // Date as an integer (Excel like date).
-      var cellMeta = this.hot.getCellMeta(row, column);
+      var cellMeta = this.hot.getCellMeta(visualRow, visualColumn);
       if (cellMeta.type === 'date' && isNumeric(cellValue)) {
         cellValue = getDateFromExcelDate(cellValue, cellMeta.dateFormat);
       }
@@ -928,6 +919,9 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
       }
       var visualRow = this.hot.toVisualRow(row);
       var visualColumn = this.hot.propToCol(columnOrProp);
+      if (visualRow === null || visualColumn === null) {
+        return;
+      }
 
       // `column` is here as visual index because of inconsistencies related to hook execution in `src/dataMap`.
       var isFormulaCellType = this.isFormulaCellType(visualRow, visualColumn);
@@ -947,9 +941,8 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
         return;
       }
       var address = {
-        row: row,
-        // Workaround for inconsistencies in `src/dataSource.js`
-        col: this.toPhysicalColumnPosition(visualColumn),
+        row: this.rowAxisSyncer.getHfIndexFromVisualIndex(visualRow),
+        col: this.columnAxisSyncer.getHfIndexFromVisualIndex(visualColumn),
         sheet: this.sheetId
       };
       valueHolder.value = this.engine.getCellSerialized(address);
@@ -975,21 +968,21 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
       var dependentCells = this.engine.batch(function () {
         changes.forEach(function (_ref9) {
           var _ref10 = _slicedToArray(_ref9, 4),
-            row = _ref10[0],
+            visualRow = _ref10[0],
             prop = _ref10[1],
             newValue = _ref10[3];
-          var column = _this15.hot.propToCol(prop);
-          var physicalRow = _this15.hot.toPhysicalRow(row);
-          var physicalColumn = _this15.hot.toPhysicalColumn(column);
+          var visualColumn = _this15.hot.propToCol(prop);
+          var physicalRow = _this15.hot.toPhysicalRow(visualRow);
+          var physicalColumn = _this15.hot.toPhysicalColumn(visualColumn);
           var address = {
-            row: physicalRow,
-            col: physicalColumn,
+            row: _this15.rowAxisSyncer.getHfIndexFromVisualIndex(visualRow),
+            col: _this15.columnAxisSyncer.getHfIndexFromVisualIndex(visualColumn),
             sheet: _this15.sheetId
           };
           if (physicalRow !== null && physicalColumn !== null) {
-            _this15.syncChangeWithEngine(row, column, newValue);
+            _this15.syncChangeWithEngine(visualRow, visualColumn, newValue);
           } else {
-            outOfBoundsChanges.push([row, column, newValue]);
+            outOfBoundsChanges.push([visualRow, visualColumn, newValue]);
           }
           changedCells.push({
             address: address
@@ -1035,16 +1028,16 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
       var changedCells = [];
       changes.forEach(function (_ref13) {
         var _ref14 = _slicedToArray(_ref13, 4),
-          row = _ref14[0],
+          visualRow = _ref14[0],
           prop = _ref14[1],
           newValue = _ref14[3];
-        var column = _this16.hot.propToCol(prop);
-        if (!isNumeric(column)) {
+        var visualColumn = _this16.hot.propToCol(prop);
+        if (!isNumeric(visualColumn)) {
           return;
         }
         var address = {
-          row: row,
-          col: _this16.toPhysicalColumnPosition(column),
+          row: _this16.rowAxisSyncer.getHfIndexFromVisualIndex(visualRow),
+          col: _this16.columnAxisSyncer.getHfIndexFromVisualIndex(visualColumn),
           sheet: _this16.sheetId
         };
         if (!_this16.engine.isItPossibleToSetCellContents(address)) {
@@ -1064,14 +1057,19 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
      * `beforeCreateRow` hook callback.
      *
      * @private
-     * @param {number} row Represents the visual index of first newly created row in the data source array.
+     * @param {number} visualRow Represents the visual index of first newly created row in the data source array.
      * @param {number} amount Number of newly created rows in the data source array.
      * @returns {*|boolean} If false is returned the action is canceled.
      */
   }, {
     key: "onBeforeCreateRow",
-    value: function onBeforeCreateRow(row, amount) {
-      if (this.sheetId === null || !this.engine.doesSheetExist(this.sheetName) || !this.engine.isItPossibleToAddRows(this.sheetId, [this.toPhysicalRowPosition(row), amount])) {
+    value: function onBeforeCreateRow(visualRow, amount) {
+      var hfRowIndex = this.rowAxisSyncer.getHfIndexFromVisualIndex(visualRow);
+      if (visualRow >= this.hot.countRows()) {
+        hfRowIndex = visualRow; // Row beyond the table boundaries.
+      }
+
+      if (this.sheetId === null || !this.engine.doesSheetExist(this.sheetName) || !this.engine.isItPossibleToAddRows(this.sheetId, [hfRowIndex, amount])) {
         return false;
       }
     }
@@ -1080,14 +1078,19 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
      * `beforeCreateCol` hook callback.
      *
      * @private
-     * @param {number} col Represents the visual index of first newly created column in the data source.
+     * @param {number} visualColumn Represents the visual index of first newly created column in the data source.
      * @param {number} amount Number of newly created columns in the data source.
      * @returns {*|boolean} If false is returned the action is canceled.
      */
   }, {
     key: "onBeforeCreateCol",
-    value: function onBeforeCreateCol(col, amount) {
-      if (this.sheetId === null || !this.engine.doesSheetExist(this.sheetName) || !this.engine.isItPossibleToAddColumns(this.sheetId, [this.toPhysicalColumnPosition(col), amount])) {
+    value: function onBeforeCreateCol(visualColumn, amount) {
+      var hfColumnIndex = this.columnAxisSyncer.getHfIndexFromVisualIndex(visualColumn);
+      if (visualColumn >= this.hot.countCols()) {
+        hfColumnIndex = visualColumn; // Column beyond the table boundaries.
+      }
+
+      if (this.sheetId === null || !this.engine.doesSheetExist(this.sheetName) || !this.engine.isItPossibleToAddColumns(this.sheetId, [hfColumnIndex, amount])) {
         return false;
       }
     }
@@ -1105,8 +1108,9 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
     key: "onBeforeRemoveRow",
     value: function onBeforeRemoveRow(row, amount, physicalRows) {
       var _this17 = this;
-      var possible = physicalRows.every(function (physicalRow) {
-        return _this17.engine.isItPossibleToRemoveRows(_this17.sheetId, [physicalRow, 1]);
+      var hfRows = this.rowAxisSyncer.setRemovedHfIndexes(physicalRows);
+      var possible = hfRows.every(function (hfRow) {
+        return _this17.engine.isItPossibleToRemoveRows(_this17.sheetId, [hfRow, 1]);
       });
       return possible === false ? false : void 0;
     }
@@ -1124,8 +1128,9 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
     key: "onBeforeRemoveCol",
     value: function onBeforeRemoveCol(col, amount, physicalColumns) {
       var _this18 = this;
-      var possible = physicalColumns.every(function (physicalColumn) {
-        return _this18.engine.isItPossibleToRemoveColumns(_this18.sheetId, [physicalColumn, 1]);
+      var hfColumns = this.columnAxisSyncer.setRemovedHfIndexes(physicalColumns);
+      var possible = hfColumns.every(function (hfColumn) {
+        return _this18.engine.isItPossibleToRemoveColumns(_this18.sheetId, [hfColumn, 1]);
       });
       return possible === false ? false : void 0;
     }
@@ -1134,18 +1139,18 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
      * `afterCreateRow` hook callback.
      *
      * @private
-     * @param {number} row Represents the visual index of first newly created row in the data source array.
+     * @param {number} visualRow Represents the visual index of first newly created row in the data source array.
      * @param {number} amount Number of newly created rows in the data source array.
      * @param {string} [source] String that identifies source of hook call
      *                          ([list of all available sources]{@link https://handsontable.com/docs/javascript-data-grid/events-and-hooks/#handsontable-hooks}).
      */
   }, {
     key: "onAfterCreateRow",
-    value: function onAfterCreateRow(row, amount, source) {
+    value: function onAfterCreateRow(visualRow, amount, source) {
       if (isBlockedSource(source)) {
         return;
       }
-      var changes = this.engine.addRows(this.sheetId, [this.toPhysicalRowPosition(row), amount]);
+      var changes = this.engine.addRows(this.sheetId, [this.rowAxisSyncer.getHfIndexFromVisualIndex(visualRow), amount]);
       this.renderDependentSheets(changes);
     }
 
@@ -1153,18 +1158,18 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
      * `afterCreateCol` hook callback.
      *
      * @private
-     * @param {number} col Represents the visual index of first newly created column in the data source.
+     * @param {number} visualColumn Represents the visual index of first newly created column in the data source.
      * @param {number} amount Number of newly created columns in the data source.
      * @param {string} [source] String that identifies source of hook call
      *                          ([list of all available sources]{@link https://handsontable.com/docs/javascript-data-grid/events-and-hooks/#handsontable-hooks}).
      */
   }, {
     key: "onAfterCreateCol",
-    value: function onAfterCreateCol(col, amount, source) {
+    value: function onAfterCreateCol(visualColumn, amount, source) {
       if (isBlockedSource(source)) {
         return;
       }
-      var changes = this.engine.addColumns(this.sheetId, [this.toPhysicalColumnPosition(col), amount]);
+      var changes = this.engine.addColumns(this.sheetId, [this.columnAxisSyncer.getHfIndexFromVisualIndex(visualColumn), amount]);
       this.renderDependentSheets(changes);
     }
 
@@ -1185,10 +1190,10 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
       if (isBlockedSource(source)) {
         return;
       }
-      var descendingPhysicalRows = physicalRows.sort().reverse();
+      var descendingHfRows = this.rowAxisSyncer.getRemovedHfIndexes().sort().reverse();
       var changes = this.engine.batch(function () {
-        descendingPhysicalRows.forEach(function (physicalRow) {
-          _this19.engine.removeRows(_this19.sheetId, [physicalRow, 1]);
+        descendingHfRows.forEach(function (hfRow) {
+          _this19.engine.removeRows(_this19.sheetId, [hfRow, 1]);
         });
       });
       this.renderDependentSheets(changes);
@@ -1211,10 +1216,10 @@ export var Formulas = /*#__PURE__*/function (_BasePlugin) {
       if (isBlockedSource(source)) {
         return;
       }
-      var descendingPhysicalColumns = physicalColumns.sort().reverse();
+      var descendingHfColumns = this.columnAxisSyncer.getRemovedHfIndexes().sort().reverse();
       var changes = this.engine.batch(function () {
-        descendingPhysicalColumns.forEach(function (physicalColumn) {
-          _this20.engine.removeColumns(_this20.sheetId, [physicalColumn, 1]);
+        descendingHfColumns.forEach(function (hfColumn) {
+          _this20.engine.removeColumns(_this20.sheetId, [hfColumn, 1]);
         });
       });
       this.renderDependentSheets(changes);