@odoo/o-spreadsheet 16.1.8 → 16.1.10

package/dist/o-spreadsheet.cjs.js

@@ -1023,6 +1023,10 @@ function deepEquals(o1, o2) {
     }
     return true;
 }
+/** Check if the given array contains all the values of the other array. */
+function includesAll(arr, values) {
+    return values.every((value) => arr.includes(value));
+}
 /**
  * Return an object with all the keys in the object that have a falsy value removed.
  */
@@ -1662,12 +1666,77 @@ const numberRepresentation = [];
  * - all digit stored in the decimal part of the number
  *
  * The 'maxDecimal' parameter allows to indicate the number of digits to not
- * exceed in the decimal part, in which case digits are rounded
+ * exceed in the decimal part, in which case digits are rounded.
  *
- * Intl.Numberformat is used to properly handle all the roundings.
- * e.g. 1234.7  with format ### (<> maxDecimals=0) should become 1235, not 1234
  **/
 function splitNumber(value, maxDecimals = MAX_DECIMAL_PLACES) {
+    const asString = value.toString();
+    if (asString.includes("e"))
+        return splitNumberIntl(value, maxDecimals);
+    if (Number.isInteger(value)) {
+        return { integerDigits: asString, decimalDigits: undefined };
+    }
+    const indexOfDot = asString.indexOf(".");
+    let integerDigits = asString.substring(0, indexOfDot);
+    let decimalDigits = asString.substring(indexOfDot + 1);
+    if (maxDecimals === 0) {
+        if (Number(decimalDigits[0]) >= 5) {
+            integerDigits = (Number(integerDigits) + 1).toString();
+        }
+        return { integerDigits, decimalDigits: undefined };
+    }
+    if (decimalDigits.length > maxDecimals) {
+        const { integerDigits: roundedIntegerDigits, decimalDigits: roundedDecimalDigits } = limitDecimalDigits(decimalDigits, maxDecimals);
+        decimalDigits = roundedDecimalDigits;
+        if (roundedIntegerDigits !== "0") {
+            integerDigits = (Number(integerDigits) + Number(roundedIntegerDigits)).toString();
+        }
+    }
+    return { integerDigits, decimalDigits: removeTrailingZeroes(decimalDigits || "") };
+}
+/**
+ *  Return the given string minus the trailing "0" characters.
+ *
+ * @param numberString : a string of integers
+ * @returns the numberString, minus the eventual zeroes at the end
+ */
+function removeTrailingZeroes(numberString) {
+    let i = numberString.length - 1;
+    while (i >= 0 && numberString[i] === "0") {
+        i--;
+    }
+    return numberString.slice(0, i + 1) || undefined;
+}
+/**
+ * Limit the size of the decimal part of a number to the given number of digits.
+ */
+function limitDecimalDigits(decimalDigits, maxDecimals) {
+    let integerDigits = "0";
+    let resultDecimalDigits = decimalDigits;
+    // Note : we'd want to simply use number.toFixed() to handle the max digits & rounding,
+    // but it has very strange behaviour. Ex: 12.345.toFixed(2) => "12.35", but 1.345.toFixed(2) => "1.34"
+    let slicedDecimalDigits = decimalDigits.slice(0, maxDecimals);
+    const i = maxDecimals;
+    if (Number(Number(decimalDigits[i]) < 5)) {
+        return { integerDigits, decimalDigits: slicedDecimalDigits };
+    }
+    // round up
+    const slicedRoundedUp = (Number(slicedDecimalDigits) + 1).toString();
+    if (slicedRoundedUp.length > slicedDecimalDigits.length) {
+        integerDigits = (Number(integerDigits) + 1).toString();
+        resultDecimalDigits = undefined;
+    }
+    else {
+        resultDecimalDigits = slicedRoundedUp;
+    }
+    return { integerDigits, decimalDigits: resultDecimalDigits };
+}
+/**
+ * Split numbers into decimal/integer digits using Intl.NumberFormat.
+ * Supports numbers with a lot of digits that are transformed to scientific notation by
+ * number.toString(), but is slow.
+ */
+function splitNumberIntl(value, maxDecimals = MAX_DECIMAL_PLACES) {
     let formatter = numberRepresentation[maxDecimals];
     if (!formatter) {
         formatter = new Intl.NumberFormat("en-US", {
@@ -3559,6 +3628,8 @@ exports.CommandResult = void 0;
     CommandResult[CommandResult["InvalidSelectionStep"] = 83] = "InvalidSelectionStep";
     CommandResult[CommandResult["DuplicatedChartId"] = 84] = "DuplicatedChartId";
     CommandResult[CommandResult["ChartDoesNotExist"] = 85] = "ChartDoesNotExist";
+    CommandResult[CommandResult["InvalidHeaderIndex"] = 86] = "InvalidHeaderIndex";
+    CommandResult[CommandResult["InvalidQuantity"] = 87] = "InvalidQuantity";
 })(exports.CommandResult || (exports.CommandResult = {}));
 
 var DIRECTION;
@@ -5906,13 +5977,16 @@ class Menu extends owl.Component {
             }
         });
     }
+    get visibleMenuItems() {
+        return this.props.menuItems.filter((x) => x.isVisible(this.env));
+    }
     get subMenuPosition() {
         const position = Object.assign({}, this.subMenu.position);
         position.y -= this.subMenu.scrollOffset || 0;
         return position;
     }
     get menuHeight() {
-        const menuItems = this.props.menuItems;
+        const menuItems = this.visibleMenuItems;
         let menuItemsHeight = this.getMenuItemsHeight(menuItems);
         // We don't display separator at the end of a menu
         if (menuItems[menuItems.length - 1].separator) {
@@ -5954,7 +6028,7 @@ class Menu extends owl.Component {
      * and the menu item at a given index.
      */
     subMenuVerticalPosition(menuIndex) {
-        const menusAbove = this.props.menuItems.slice(0, menuIndex);
+        const menusAbove = this.visibleMenuItems.slice(0, menuIndex);
         return this.position.y + this.getMenuItemsHeight(menusAbove) + MENU_VERTICAL_PADDING;
     }
     onExternalClick(ev) {
@@ -6126,7 +6200,7 @@ class ChartFigure extends owl.Component {
     openContextMenu(position) {
         const registry = this.getMenuItemRegistry();
         this.menuState.isOpen = true;
-        this.menuState.menuItems = registry.getAll().filter((x) => x.isVisible(this.env));
+        this.menuState.menuItems = registry.getAll();
         this.menuState.position = position;
     }
     get chartComponent() {
@@ -7257,6 +7331,11 @@ colMenuRegistry
     name: REMOVE_COLUMNS_NAME,
     sequence: 90,
     action: REMOVE_COLUMNS_ACTION,
+    isVisible: (env) => {
+        const sheetId = env.model.getters.getActiveSheetId();
+        const selectedCols = env.model.getters.getElementsFromSelection("COL");
+        return env.model.getters.canRemoveHeaders(sheetId, "COL", selectedCols);
+    },
 })
     .add("clear_column", {
     name: DELETE_CONTENT_COLUMNS_NAME,
@@ -7269,9 +7348,8 @@ colMenuRegistry
     action: HIDE_COLUMNS_ACTION,
     isVisible: (env) => {
         const sheetId = env.model.getters.getActiveSheetId();
-        const hiddenCols = env.model.getters.getHiddenColsGroups(sheetId).flat();
-        return (env.model.getters.getNumberCols(sheetId) >
-            hiddenCols.length + env.model.getters.getElementsFromSelection("COL").length);
+        const selectedCols = env.model.getters.getElementsFromSelection("COL");
+        return env.model.getters.canRemoveHeaders(sheetId, "COL", selectedCols);
     },
     separator: true,
 })
@@ -7345,6 +7423,11 @@ rowMenuRegistry
     name: REMOVE_ROWS_NAME,
     sequence: 70,
     action: REMOVE_ROWS_ACTION,
+    isVisible: (env) => {
+        const sheetId = env.model.getters.getActiveSheetId();
+        const selectedRows = env.model.getters.getElementsFromSelection("ROW");
+        return env.model.getters.canRemoveHeaders(sheetId, "ROW", selectedRows);
+    },
 })
     .add("clear_row", {
     name: DELETE_CONTENT_ROWS_NAME,
@@ -7357,9 +7440,8 @@ rowMenuRegistry
     action: HIDE_ROWS_ACTION,
     isVisible: (env) => {
         const sheetId = env.model.getters.getActiveSheetId();
-        const hiddenRows = env.model.getters.getHiddenRowsGroups(sheetId).flat();
-        return (env.model.getters.getNumberRows(sheetId) >
-            hiddenRows.length + env.model.getters.getElementsFromSelection("ROW").length);
+        const selectedRows = env.model.getters.getElementsFromSelection("ROW");
+        return env.model.getters.canRemoveHeaders(sheetId, "ROW", selectedRows);
     },
     separator: true,
 })
@@ -14417,7 +14499,7 @@ const POW = {
       base (number) ${_lt("The number to raise to the exponent power.")}
       exponent (number) ${_lt("The exponent to raise base to.")}
     `),
-    returns: ["BOOLEAN"],
+    returns: ["NUMBER"],
     compute: function (base, exponent) {
         return POWER.compute(base, exponent);
     },
@@ -16932,14 +17014,31 @@ class ContentEditableHelper {
      * */
     getCurrentSelection() {
         let { startElement, endElement, startSelectionOffset, endSelectionOffset } = this.getStartAndEndSelection();
-        let startSizeBefore = this.findSizeBeforeElement(startElement);
-        let endSizeBefore = this.findSizeBeforeElement(endElement);
+        let startSizeBefore = this.findSelectionIndex(startElement, startSelectionOffset);
+        let endSizeBefore = this.findSelectionIndex(endElement, endSelectionOffset);
         return {
-            start: startSizeBefore + startSelectionOffset,
-            end: endSizeBefore + endSelectionOffset,
+            start: startSizeBefore,
+            end: endSizeBefore,
         };
     }
-    findSizeBeforeElement(nodeToFind) {
+    /**
+     * Computes the text 'index' inside this.el based on the currently selected node and its offset.
+     * The selected node is either a Text node or an Element node.
+     *
+     * case 1 -Text node:
+     * the offset is the number of characters from the start of the node. We have to add this offset to the
+     * content length of all previous nodes.
+     *
+     * case 2 - Element node:
+     * the offset is the number of child nodes before the selected node. We have to add the content length of
+     * all the bnodes prior to the selected node as well as the content of the child node before the offset.
+     *
+     * See the MDN documentation for more details.
+     * https://developer.mozilla.org/en-US/docs/Web/API/Range/startOffset
+     * https://developer.mozilla.org/en-US/docs/Web/API/Range/endOffset
+     *
+     */
+    findSelectionIndex(nodeToFind, nodeOffset) {
         let it = this.iterateChildren(this.el);
         let usedCharacters = 0;
         let current = it.next();
@@ -16951,6 +17050,19 @@ class ContentEditableHelper {
             }
             current = it.next();
         }
+        if (current.value === nodeToFind) {
+            if (!current.value.hasChildNodes()) {
+                usedCharacters += nodeOffset;
+            }
+            else {
+                const children = [...current.value.childNodes].slice(0, nodeOffset);
+                for (const child of children) {
+                    if (!child.textContent)
+                        continue;
+                    usedCharacters += child.textContent.length;
+                }
+            }
+        }
         return usedCharacters;
     }
     getStartAndEndSelection() {
@@ -23542,8 +23654,8 @@ class Grid extends owl.Component {
             },
             "CTRL+END": () => {
                 const sheetId = this.env.model.getters.getActiveSheetId();
-                const col = this.env.model.getters.findVisibleHeader(sheetId, "COL", range(0, this.env.model.getters.getNumberCols(sheetId)).reverse());
-                const row = this.env.model.getters.findVisibleHeader(sheetId, "ROW", range(0, this.env.model.getters.getNumberRows(sheetId)).reverse());
+                const col = this.env.model.getters.findVisibleHeader(sheetId, "COL", this.env.model.getters.getNumberCols(sheetId) - 1, 0);
+                const row = this.env.model.getters.findVisibleHeader(sheetId, "ROW", this.env.model.getters.getNumberRows(sheetId) - 1, 0);
                 this.env.model.selection.selectCell(col, row);
             },
             "SHIFT+ ": () => {
@@ -23623,9 +23735,10 @@ class Grid extends owl.Component {
         this.focus();
     }
     focus() {
+        var _a;
         if (!this.env.model.getters.getSelectedFigureId() &&
             this.env.model.getters.getEditionMode() === "inactive") {
-            this.hiddenInput.el.focus();
+            (_a = this.hiddenInput.el) === null || _a === void 0 ? void 0 : _a.focus();
         }
     }
     get gridEl() {
@@ -23826,9 +23939,7 @@ class Grid extends owl.Component {
         }
         this.menuState.isOpen = true;
         this.menuState.position = { x, y };
-        this.menuState.menuItems = registries$1[type]
-            .getAll()
-            .filter((item) => !item.isVisible || item.isVisible(this.env));
+        this.menuState.menuItems = registries$1[type].getAll();
     }
     copy(cut, ev) {
         if (!this.gridEl.contains(document.activeElement)) {
@@ -25082,7 +25193,7 @@ function extractStyle(cell, data) {
     if (cell.style) {
         style = data.styles[cell.style];
     }
-    const format = cell.format ? data.formats[cell.format] : undefined;
+    const format = extractFormat(cell, data);
     const exportedBorder = {};
     if (cell.border) {
         const border = data.borders[cell.border];
@@ -25116,6 +25227,22 @@ function extractStyle(cell, data) {
     styles.font["italic"] = !!(style === null || style === void 0 ? void 0 : style.italic) || undefined;
     return styles;
 }
+function extractFormat(cell, data) {
+    if (cell.format) {
+        return data.formats[cell.format];
+    }
+    if (cell.isFormula) {
+        const tokens = tokenize(cell.content || "");
+        const functions = functionRegistry.content;
+        const isExported = tokens
+            .filter((tk) => tk.type === "FUNCTION")
+            .every((tk) => functions[tk.value.toUpperCase()].isExported);
+        if (!isExported) {
+            return cell.computedFormat;
+        }
+    }
+    return undefined;
+}
 function normalizeStyle(construct, styles) {
     const { id: fontId } = pushElement(styles["font"], construct.fonts);
     const { id: fillId } = pushElement(styles["fill"], construct.fills);
@@ -27197,7 +27324,7 @@ function load(data, verboseImport) {
             }
         }
     }
-    data = JSON.parse(JSON.stringify(data));
+    data = deepCopy(data);
     // apply migrations, if needed
     if ("version" in data) {
         if (data.version < CURRENT_VERSION) {
@@ -29857,7 +29984,7 @@ class HeaderVisibilityPlugin extends CorePlugin {
     allowDispatch(cmd) {
         switch (cmd.type) {
             case "HIDE_COLUMNS_ROWS": {
-                if (!this.hiddenHeaders[cmd.sheetId]) {
+                if (!this.getters.tryGetSheet(cmd.sheetId)) {
                     return 27 /* CommandResult.InvalidSheetId */;
                 }
                 const hiddenGroup = cmd.dimension === "COL"
@@ -29866,10 +29993,25 @@ class HeaderVisibilityPlugin extends CorePlugin {
                 const elements = cmd.dimension === "COL"
                     ? this.getters.getNumberCols(cmd.sheetId)
                     : this.getters.getNumberRows(cmd.sheetId);
-                return (hiddenGroup || []).flat().concat(cmd.elements).length < elements
-                    ? 0 /* CommandResult.Success */
-                    : 66 /* CommandResult.TooManyHiddenElements */;
+                const hiddenElements = new Set((hiddenGroup || []).flat().concat(cmd.elements));
+                if (hiddenElements.size >= elements) {
+                    return 66 /* CommandResult.TooManyHiddenElements */;
+                }
+                else if (Math.min(...cmd.elements) < 0 || Math.max(...cmd.elements) > elements) {
+                    return 86 /* CommandResult.InvalidHeaderIndex */;
+                }
+                else {
+                    return 0 /* CommandResult.Success */;
+                }
             }
+            case "REMOVE_COLUMNS_ROWS":
+                if (!this.getters.tryGetSheet(cmd.sheetId)) {
+                    return 27 /* CommandResult.InvalidSheetId */;
+                }
+                if (!this.canRemoveHeaders(cmd.sheetId, cmd.dimension, cmd.elements)) {
+                    return 8 /* CommandResult.NotEnoughElements */;
+                }
+                return 0 /* CommandResult.Success */;
         }
         return 0 /* CommandResult.Success */;
     }
@@ -29916,6 +30058,10 @@ class HeaderVisibilityPlugin extends CorePlugin {
         }
         return;
     }
+    canRemoveHeaders(sheetId, dimension, elements) {
+        const visibleHeaders = this.getAllVisibleHeaders(sheetId, dimension);
+        return !includesAll(elements, visibleHeaders);
+    }
     isRowHiddenByUser(sheetId, index) {
         return this.hiddenHeaders[sheetId].ROW[index];
     }
@@ -29960,6 +30106,9 @@ class HeaderVisibilityPlugin extends CorePlugin {
         }
         return consecutiveIndexes;
     }
+    getAllVisibleHeaders(sheetId, dimension) {
+        return range(0, this.hiddenHeaders[sheetId][dimension].length).filter((i) => !this.hiddenHeaders[sheetId][dimension][i]);
+    }
     import(data) {
         var _a, _b;
         for (let sheet of data.sheets) {
@@ -30007,6 +30156,7 @@ class HeaderVisibilityPlugin extends CorePlugin {
     }
 }
 HeaderVisibilityPlugin.getters = [
+    "canRemoveHeaders",
     "getHiddenColsGroups",
     "getHiddenRowsGroups",
     "isRowHiddenByUser",
@@ -31016,13 +31166,27 @@ class SheetPlugin extends CorePlugin {
                 return this.orderedSheetIds.length > 1
                     ? 0 /* CommandResult.Success */
                     : 9 /* CommandResult.NotEnoughSheets */;
+            case "ADD_COLUMNS_ROWS":
+                const elements = cmd.dimension === "COL"
+                    ? this.getNumberCols(cmd.sheetId)
+                    : this.getNumberRows(cmd.sheetId);
+                if (cmd.base < 0 || cmd.base > elements) {
+                    return 86 /* CommandResult.InvalidHeaderIndex */;
+                }
+                else if (cmd.quantity <= 0) {
+                    return 87 /* CommandResult.InvalidQuantity */;
+                }
+                return 0 /* CommandResult.Success */;
             case "REMOVE_COLUMNS_ROWS": {
-                const length = cmd.dimension === "COL"
+                const elements = cmd.dimension === "COL"
                     ? this.getNumberCols(cmd.sheetId)
                     : this.getNumberRows(cmd.sheetId);
-                return length > cmd.elements.length
-                    ? 0 /* CommandResult.Success */
-                    : 8 /* CommandResult.NotEnoughElements */;
+                if (Math.min(...cmd.elements) < 0 || Math.max(...cmd.elements) > elements) {
+                    return 86 /* CommandResult.InvalidHeaderIndex */;
+                }
+                else {
+                    return 0 /* CommandResult.Success */;
+                }
             }
             case "FREEZE_ROWS": {
                 return this.checkValidations(cmd, this.checkRowFreezeQuantity, this.checkRowFreezeOverlapMerge);
@@ -31494,7 +31658,7 @@ class SheetPlugin extends CorePlugin {
     duplicateSheet(fromId, toId) {
         const sheet = this.getSheet(fromId);
         const toName = this.getDuplicateSheetName(sheet.name);
-        const newSheet = JSON.parse(JSON.stringify(sheet));
+        const newSheet = deepCopy(sheet);
         newSheet.id = toId;
         newSheet.name = toName;
         for (let col = 0; col <= newSheet.numberOfCols; col++) {
@@ -31621,10 +31785,10 @@ class SheetPlugin extends CorePlugin {
         }
     }
     moveCellOnColumnsDeletion(sheet, deletedColumn) {
-        for (let [index, row] of Object.entries(sheet.rows)) {
-            const rowIndex = parseInt(index, 10);
+        for (let rowIndex = 0; rowIndex < sheet.rows.length; rowIndex++) {
+            const row = sheet.rows[rowIndex];
             for (let i in row.cells) {
-                const colIndex = parseInt(i, 10);
+                const colIndex = Number(i);
                 const cellId = row.cells[i];
                 if (cellId) {
                     if (colIndex === deletedColumn) {
@@ -31651,11 +31815,11 @@ class SheetPlugin extends CorePlugin {
      */
     moveCellsOnAddition(sheet, addedElement, quantity, dimension) {
         const commands = [];
-        for (const [index, row] of Object.entries(sheet.rows)) {
-            const rowIndex = parseInt(index, 10);
+        for (let rowIndex = 0; rowIndex < sheet.rows.length; rowIndex++) {
+            const row = sheet.rows[rowIndex];
             if (dimension !== "rows" || rowIndex >= addedElement) {
                 for (let i in row.cells) {
-                    const colIndex = parseInt(i, 10);
+                    const colIndex = Number(i);
                     const cellId = row.cells[i];
                     if (cellId) {
                         if (dimension === "rows" || colIndex >= addedElement) {
@@ -31685,11 +31849,11 @@ class SheetPlugin extends CorePlugin {
      */
     moveCellOnRowsDeletion(sheet, deleteFromRow, deleteToRow) {
         const numberRows = deleteToRow - deleteFromRow + 1;
-        for (let [index, row] of Object.entries(sheet.rows)) {
-            const rowIndex = parseInt(index, 10);
+        for (let rowIndex = 0; rowIndex < sheet.rows.length; rowIndex++) {
+            const row = sheet.rows[rowIndex];
             if (rowIndex >= deleteFromRow && rowIndex <= deleteToRow) {
                 for (let i in row.cells) {
-                    const colIndex = parseInt(i, 10);
+                    const colIndex = Number(i);
                     const cellId = row.cells[i];
                     if (cellId) {
                         this.dispatch("CLEAR_CELL", {
@@ -31702,7 +31866,7 @@ class SheetPlugin extends CorePlugin {
             }
             if (rowIndex > deleteToRow) {
                 for (let i in row.cells) {
-                    const colIndex = parseInt(i, 10);
+                    const colIndex = Number(i);
                     const cellId = row.cells[i];
                     if (cellId) {
                         this.dispatch("UPDATE_CELL_POSITION", {
@@ -31720,7 +31884,7 @@ class SheetPlugin extends CorePlugin {
         const rows = [];
         const cellsQueue = sheet.rows.map((row) => row.cells);
         for (let i in sheet.rows) {
-            if (parseInt(i, 10) === index) {
+            if (Number(i) === index) {
                 continue;
             }
             rows.push({
@@ -32272,8 +32436,12 @@ class EvaluationPlugin extends UIPlugin {
                 const cell = this.getters.getCell(position);
                 if (cell) {
                     const exportedCellData = sheet.cells[xc];
-                    exportedCellData.value = this.getEvaluatedCell(position).value;
+                    const evaluatedCell = this.getEvaluatedCell(position);
+                    exportedCellData.value = evaluatedCell.value;
                     exportedCellData.isFormula = cell.isFormula && !this.isBadExpression(cell.content);
+                    if (cell.format !== evaluatedCell.format) {
+                        exportedCellData.computedFormat = evaluatedCell.format;
+                    }
                 }
             }
         }
@@ -32863,13 +33031,11 @@ class FilterEvaluationPlugin extends UIPlugin {
         return Boolean(id && ((_b = (_a = this.filterValues[sheetId]) === null || _a === void 0 ? void 0 : _a[id]) === null || _b === void 0 ? void 0 : _b.length));
     }
     intersectZoneWithViewport(sheetId, zone) {
-        const colsRange = range(zone.left, zone.right + 1);
-        const rowsRange = range(zone.top, zone.bottom + 1);
         return {
-            left: this.getters.findVisibleHeader(sheetId, "COL", colsRange),
-            right: this.getters.findVisibleHeader(sheetId, "COL", colsRange.reverse()),
-            top: this.getters.findVisibleHeader(sheetId, "ROW", rowsRange),
-            bottom: this.getters.findVisibleHeader(sheetId, "ROW", rowsRange.reverse()),
+            left: this.getters.findVisibleHeader(sheetId, "COL", zone.left, zone.right),
+            right: this.getters.findVisibleHeader(sheetId, "COL", zone.right, zone.left),
+            top: this.getters.findVisibleHeader(sheetId, "ROW", zone.top, zone.bottom),
+            bottom: this.getters.findVisibleHeader(sheetId, "ROW", zone.bottom, zone.top),
         };
     }
     updateFilter({ col, row, values, sheetId }) {
@@ -34975,13 +35141,33 @@ class HeaderVisibilityUIPlugin extends UIPlugin {
     getNextVisibleCellPosition({ sheetId, col, row }) {
         return {
             sheetId,
-            col: this.findVisibleHeader(sheetId, "COL", range(col, this.getters.getNumberCols(sheetId))),
-            row: this.findVisibleHeader(sheetId, "ROW", range(row, this.getters.getNumberRows(sheetId))),
+            col: this.findVisibleHeader(sheetId, "COL", col, this.getters.getNumberCols(sheetId) - 1),
+            row: this.findVisibleHeader(sheetId, "ROW", row, this.getters.getNumberRows(sheetId) - 1),
         };
     }
-    findVisibleHeader(sheetId, dimension, indexes) {
-        return indexes.find((index) => this.getters.doesHeaderExist(sheetId, dimension, index) &&
-            !this.isHeaderHidden(sheetId, dimension, index));
+    /**
+     * Find the first visible header in the range [`from` => `to`].
+     *
+     * Both `from` and `to` are inclusive.
+     */
+    findVisibleHeader(sheetId, dimension, from, to) {
+        if (from <= to) {
+            for (let i = from; i <= to; i++) {
+                if (this.getters.doesHeaderExist(sheetId, dimension, i) &&
+                    !this.isHeaderHidden(sheetId, dimension, i)) {
+                    return i;
+                }
+            }
+        }
+        if (from > to) {
+            for (let i = from; i >= to; i--) {
+                if (this.getters.doesHeaderExist(sheetId, dimension, i) &&
+                    !this.isHeaderHidden(sheetId, dimension, i)) {
+                    return i;
+                }
+            }
+        }
+        return undefined;
     }
     findLastVisibleColRowIndex(sheetId, dimension, { last, first }) {
         const lastVisibleIndex = range(last, first, -1).find((index) => !this.isHeaderHidden(sheetId, dimension, index));
@@ -34989,7 +35175,7 @@ class HeaderVisibilityUIPlugin extends UIPlugin {
     }
     findFirstVisibleColRowIndex(sheetId, dimension) {
         const numberOfHeaders = this.getters.getNumberHeaders(sheetId, dimension);
-        for (let i = 0; i < numberOfHeaders - 1; i++) {
+        for (let i = 0; i < numberOfHeaders; i++) {
             if (dimension === "COL" && !this.isColHidden(sheetId, i)) {
                 return i;
             }
@@ -39216,7 +39402,7 @@ class BottomBar extends owl.Component {
     }
     openContextMenu(x, y, registry, menuId) {
         this.menuState.isOpen = true;
-        this.menuState.menuItems = registry.getAll().filter((x) => x.isVisible(this.env));
+        this.menuState.menuItems = registry.getAll();
         this.menuState.position = { x, y };
         this.menuState.menuId = menuId;
     }
@@ -40023,7 +40209,7 @@ class TopBar extends owl.Component {
         const { left, top, height } = ev.target.getBoundingClientRect();
         this.state.menuState.isOpen = true;
         this.state.menuState.position = { x: left, y: top + height };
-        this.state.menuState.menuItems = getMenuChildren(menu, this.env).filter((item) => !item.isVisible || item.isVisible(this.env));
+        this.state.menuState.menuItems = getMenuChildren(menu, this.env);
         this.state.menuState.parentMenu = menu;
         this.isSelectingMenu = true;
         this.openedEl = ev.target;
@@ -40066,9 +40252,7 @@ class TopBar extends owl.Component {
         this.style.align = this.style.align || cell.defaultAlign;
         this.fillColor = this.style.fillColor || "#ffffff";
         this.textColor = this.style.textColor || "#000000";
-        this.menus = topbarMenuRegistry
-            .getAll()
-            .filter((item) => !item.isVisible || item.isVisible(this.env));
+        this.menus = topbarMenuRegistry.getAll();
     }
     getMenuName(menu) {
         return getMenuName(menu, this.env);
@@ -41782,8 +41966,8 @@ class SelectionStreamProcessor {
         }
         const { left, right, top, bottom } = zone;
         const sheetId = this.getters.getActiveSheetId();
-        const refCol = this.getters.findVisibleHeader(sheetId, "COL", range(left, right + 1));
-        const refRow = this.getters.findVisibleHeader(sheetId, "ROW", range(top, bottom + 1));
+        const refCol = this.getters.findVisibleHeader(sheetId, "COL", left, right);
+        const refRow = this.getters.findVisibleHeader(sheetId, "ROW", top, bottom);
         if (refRow === undefined || refCol === undefined) {
             return 17 /* CommandResult.SelectionOutOfBound */;
         }
@@ -41858,10 +42042,10 @@ class SelectionStreamProcessor {
         const { col: anchorCol, row: anchorRow } = anchor.cell;
         return {
             col: this.getters.isColHidden(sheetId, anchorCol)
-                ? this.getters.findVisibleHeader(sheetId, "COL", range(left, right + 1)) || anchorCol
+                ? this.getters.findVisibleHeader(sheetId, "COL", left, right) || anchorCol
                 : anchorCol,
             row: this.getters.isRowHidden(sheetId, anchorRow)
-                ? this.getters.findVisibleHeader(sheetId, "ROW", range(top, bottom + 1)) || anchorRow
+                ? this.getters.findVisibleHeader(sheetId, "ROW", top, bottom) || anchorRow
                 : anchorRow,
         };
     }
@@ -42435,12 +42619,9 @@ function addFormula(cell) {
     else {
         // Shouldn't we always output the value then ?
         const value = cell.value;
-        // what if value = 0? Is this condition correct?
-        if (value) {
-            const type = getCellType(value);
-            attrs.push(["t", type]);
-            node = escapeXml /*xml*/ `<v>${value}</v>`;
-        }
+        const type = getCellType(value);
+        attrs.push(["t", type]);
+        node = escapeXml /*xml*/ `<v>${value}</v>`;
         return { attrs, node };
     }
 }
@@ -43542,6 +43723,8 @@ class Model extends EventBus {
          * Internal status of the model. Important for command handling coordination
          */
         this.status = 0 /* Status.Ready */;
+        this.handlers = [];
+        this.coreHandlers = [];
         /**
          * The dispatch method is the only entry point to manipulate data in the model.
          * This is through this method that commands are dispatched most of the time
@@ -43646,6 +43829,8 @@ class Model extends EventBus {
         this.uuidGenerator.setIsFastStrategy(true);
         // Initiate stream processor
         this.selection = new SelectionStreamProcessor(this.getters);
+        this.coreHandlers.push(this.range);
+        this.handlers.push(this.range);
         this.corePluginConfig = this.setupCorePluginConfig();
         this.uiPluginConfig = this.setupUiPluginConfig();
         // registering plugins
@@ -43654,15 +43839,23 @@ class Model extends EventBus {
         }
         Object.assign(this.getters, this.coreGetters);
         for (let Plugin of statefulUIPluginRegistry.getAll()) {
-            this.statefulUIPlugins.push(this.setupUiPlugin(Plugin));
+            const plugin = this.setupUiPlugin(Plugin);
+            this.statefulUIPlugins.push(plugin);
+            this.handlers.push(plugin);
         }
         for (let Plugin of coreViewsPluginRegistry.getAll()) {
-            this.coreViewsPlugins.push(this.setupUiPlugin(Plugin));
+            const plugin = this.setupUiPlugin(Plugin);
+            this.coreViewsPlugins.push(plugin);
+            this.handlers.push(plugin);
+            this.coreHandlers.push(plugin);
         }
         for (let Plugin of featurePluginRegistry.getAll()) {
-            this.featurePlugins.push(this.setupUiPlugin(Plugin));
+            const plugin = this.setupUiPlugin(Plugin);
+            this.featurePlugins.push(plugin);
+            this.handlers.push(plugin);
         }
         this.uuidGenerator.setIsFastStrategy(false);
+        this.handlers.push(this.history);
         // starting plugins
         this.dispatch("START");
         // Model should be the last permanent subscriber in the list since he should render
@@ -43684,12 +43877,6 @@ class Model extends EventBus {
         // by owl, since we do not rely on reactivity
         owl.markRaw(this);
     }
-    get handlers() {
-        return [this.range, ...this.corePlugins, ...this.allUIPlugins, this.history];
-    }
-    get allUIPlugins() {
-        return [...this.statefulUIPlugins, ...this.coreViewsPlugins, ...this.featurePlugins];
-    }
     joinSession() {
         this.session.join(this.config.client);
     }
@@ -43731,6 +43918,8 @@ class Model extends EventBus {
         }
         plugin.import(data);
         this.corePlugins.push(plugin);
+        this.coreHandlers.push(plugin);
+        this.handlers.push(plugin);
     }
     onRemoteRevisionReceived({ commands }) {
         for (let command of commands) {
@@ -43873,7 +44062,7 @@ class Model extends EventBus {
             }
         }
         data.revisionId = this.session.getRevisionId() || DEFAULT_REVISION_ID;
-        data = JSON.parse(JSON.stringify(data));
+        data = deepCopy(data);
         return data;
     }
     updateMode(mode) {
@@ -43901,7 +44090,7 @@ class Model extends EventBus {
                 handler.exportForExcel(data);
             }
         }
-        data = JSON.parse(JSON.stringify(data));
+        data = deepCopy(data);
         return getXLSX(data);
     }
     garbageCollectExternalResources() {
@@ -44042,6 +44231,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = '16.1.8';
-__info__.date = '2023-04-14T12:46:41.355Z';
-__info__.hash = '4399770';
+__info__.version = '16.1.10';
+__info__.date = '2023-04-28T12:24:14.323Z';
+__info__.hash = '9632927';