@odoo/o-spreadsheet 16.0.7 → 16.0.9

package/dist/o-spreadsheet.iife.js

@@ -1024,6 +1024,10 @@
       }
       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.
    */
@@ -1569,12 +1573,77 @@
    * - 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", {
@@ -3201,6 +3270,8 @@
       CommandResult[CommandResult["InvalidSelectionStep"] = 82] = "InvalidSelectionStep";
       CommandResult[CommandResult["DuplicatedChartId"] = 83] = "DuplicatedChartId";
       CommandResult[CommandResult["ChartDoesNotExist"] = 84] = "ChartDoesNotExist";
+      CommandResult[CommandResult["InvalidHeaderIndex"] = 85] = "InvalidHeaderIndex";
+      CommandResult[CommandResult["InvalidQuantity"] = 86] = "InvalidQuantity";
   })(exports.CommandResult || (exports.CommandResult = {}));
 
   var DIRECTION;
@@ -3971,13 +4042,16 @@
               }
           });
       }
+      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() {
-          return this.menuComponentHeight(this.props.menuItems);
+          return this.menuComponentHeight(this.visibleMenuItems);
       }
       get subMenuHeight() {
           return this.menuComponentHeight(this.subMenu.menuItems);
@@ -4013,7 +4087,7 @@
        * and the menu item at a given index.
        */
       subMenuVerticalPosition(position) {
-          const menusAbove = this.props.menuItems.slice(0, position);
+          const menusAbove = this.visibleMenuItems.slice(0, position);
           return this.menuComponentHeight(menusAbove) + this.position.y;
       }
       onExternalClick(ev) {
@@ -5546,6 +5620,11 @@
       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,
@@ -5558,9 +5637,8 @@
       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,
   })
@@ -5634,6 +5712,11 @@
       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,
@@ -5646,9 +5729,8 @@
       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,
   })
@@ -10310,7 +10392,7 @@
       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() {
@@ -17328,7 +17410,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       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);
       },
@@ -19830,14 +19912,31 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
        * */
       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();
@@ -19849,6 +19948,19 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               }
               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() {
@@ -22334,8 +22446,8 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               },
               "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+ ": () => {
@@ -22411,9 +22523,10 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           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() {
@@ -22617,9 +22730,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           this.closeOpenedPopover();
           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)) {
@@ -24243,7 +24354,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       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];
@@ -24277,6 +24388,22 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       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);
@@ -26357,7 +26484,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               }
           }
       }
-      data = JSON.parse(JSON.stringify(data));
+      data = deepCopy(data);
       // apply migrations, if needed
       if ("version" in data) {
           if (data.version < CURRENT_VERSION) {
@@ -27499,6 +27626,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   const exportedCellData = sheet.cells[xc];
                   exportedCellData.value = cell.evaluated.value;
                   exportedCellData.isFormula = cell.isFormula();
+                  if (cell.format !== cell.evaluated.format) {
+                      exportedCellData.computedFormat = cell.evaluated.format;
+                  }
               }
           }
       }
@@ -28942,7 +29072,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       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"
@@ -28951,10 +29081,25 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   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 */
-                      : 65 /* CommandResult.TooManyHiddenElements */;
+                  const hiddenElements = new Set((hiddenGroup || []).flat().concat(cmd.elements));
+                  if (hiddenElements.size >= elements) {
+                      return 65 /* CommandResult.TooManyHiddenElements */;
+                  }
+                  else if (Math.min(...cmd.elements) < 0 || Math.max(...cmd.elements) > elements) {
+                      return 85 /* 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 */;
       }
@@ -29001,6 +29146,10 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           }
           return;
       }
+      canRemoveHeaders(sheetId, dimension, elements) {
+          const visibleHeaders = this.getAllVisibleHeaders(sheetId, dimension);
+          return !includesAll(elements, visibleHeaders);
+      }
       isRowHiddenByUser(sheetId, index) {
           return this.hiddenHeaders[sheetId].ROW[index];
       }
@@ -29045,6 +29194,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           }
           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) {
@@ -29092,6 +29244,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       }
   }
   HeaderVisibilityPlugin.getters = [
+      "canRemoveHeaders",
       "getHiddenColsGroups",
       "getHiddenRowsGroups",
       "isRowHiddenByUser",
@@ -29592,13 +29745,27 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   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 85 /* CommandResult.InvalidHeaderIndex */;
+                  }
+                  else if (cmd.quantity <= 0) {
+                      return 86 /* 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 85 /* CommandResult.InvalidHeaderIndex */;
+                  }
+                  else {
+                      return 0 /* CommandResult.Success */;
+                  }
               }
               case "FREEZE_ROWS": {
                   return this.checkValidations(cmd, this.checkRowFreezeQuantity, this.checkRowFreezeOverlapMerge);
@@ -30083,7 +30250,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       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++) {
@@ -30210,10 +30377,10 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           }
       }
       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) {
@@ -30240,11 +30407,11 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
        */
       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) {
@@ -30274,11 +30441,11 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
        */
       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", {
@@ -30291,7 +30458,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               }
               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", {
@@ -30309,7 +30476,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           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({
@@ -32945,13 +33112,11 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           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 }) {
@@ -33393,13 +33558,33 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       }
       getNextVisibleCellPosition(sheetId, col, row) {
           return {
-              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, indexes) {
           let lastIndex;
@@ -33412,7 +33597,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       }
       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;
               }
@@ -37539,7 +37724,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       }
       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;
       }
@@ -38300,7 +38485,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           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;
@@ -38349,9 +38534,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           this.style.align = this.style.align || (cell === null || cell === void 0 ? void 0 : 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);
@@ -40417,8 +40600,8 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           }
           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 */;
           }
@@ -40493,10 +40676,10 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           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,
           };
       }
@@ -41031,12 +41214,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       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 };
       }
   }
@@ -42136,6 +42316,8 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
            * 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
@@ -42208,7 +42390,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               const command = { type, ...payload };
               const previousStatus = this.status;
               this.status = 2 /* Status.RunningCore */;
-              const handlers = this.isReplayingCommand ? [this.range, ...this.corePlugins] : this.handlers;
+              const handlers = this.isReplayingCommand ? this.coreHandlers : this.handlers;
               this.dispatchToHandlers(handlers, command);
               this.status = previousStatus;
               return DispatchResult.Success;
@@ -42242,6 +42424,8 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           this.uuidGenerator.setIsFastStrategy(true);
           // Initiate stream processor
           this.selection = new SelectionStreamProcessor(this.getters);
+          this.coreHandlers.push(this.range);
+          this.handlers.push(this.range);
           // registering plugins
           for (let Plugin of corePluginRegistry.getAll()) {
               this.setupCorePlugin(Plugin, workbookData);
@@ -42251,6 +42435,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               this.setupUiPlugin(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
@@ -42271,9 +42456,6 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           // by owl, since we do not rely on reactivity
           owl.markRaw(this);
       }
-      get handlers() {
-          return [this.range, ...this.corePlugins, ...this.uiPlugins, this.history];
-      }
       joinSession() {
           this.session.join(this.config.client);
       }
@@ -42292,6 +42474,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               this.getters[name] = plugin[name].bind(plugin);
           }
           this.uiPlugins.push(plugin);
+          this.handlers.push(plugin);
           const layers = Plugin.layers.map((l) => [plugin, l]);
           this.renderers.push(...layers);
           this.renderers.sort((p1, p2) => p1[1] - p2[1]);
@@ -42315,6 +42498,8 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           }
           plugin.import(data);
           this.corePlugins.push(plugin);
+          this.coreHandlers.push(plugin);
+          this.handlers.push(plugin);
       }
       onRemoteRevisionReceived({ commands }) {
           for (let command of commands) {
@@ -42332,7 +42517,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   return;
               }
               this.isReplayingCommand = true;
-              this.dispatchToHandlers([this.range, ...this.corePlugins], command);
+              this.dispatchToHandlers(this.coreHandlers, command);
               this.isReplayingCommand = false;
           }), this.config.transportService, revisionId);
           return session;
@@ -42432,7 +42617,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               }
           }
           data.revisionId = this.session.getRevisionId() || DEFAULT_REVISION_ID;
-          data = JSON.parse(JSON.stringify(data));
+          data = deepCopy(data);
           return data;
       }
       updateMode(mode) {
@@ -42459,7 +42644,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   handler.exportForExcel(data);
               }
           }
-          data = JSON.parse(JSON.stringify(data));
+          data = deepCopy(data);
           return getXLSX(data);
       }
   }
@@ -42593,9 +42778,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '16.0.7';
-  __info__.date = '2023-04-14T12:46:24.076Z';
-  __info__.hash = '2b4831d';
+  __info__.version = '16.0.9';
+  __info__.date = '2023-04-28T12:25:41.572Z';
+  __info__.hash = 'e06ca83';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);