@odoo/o-spreadsheet 16.1.8 → 16.1.10

package/dist/o-spreadsheet.iife.js

@@ -1020,6 +1020,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.
    */
@@ -1659,12 +1663,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", {
@@ -3556,6 +3625,8 @@
       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;
@@ -5903,13 +5974,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() {
-          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) {
@@ -5951,7 +6025,7 @@
        * 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) {
@@ -6123,7 +6197,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() {
@@ -7254,6 +7328,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,
@@ -7266,9 +7345,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,
   })
@@ -7342,6 +7420,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,
@@ -7354,9 +7437,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,
   })
@@ -14414,7 +14496,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);
       },
@@ -16929,14 +17011,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();
@@ -16948,6 +17047,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() {
@@ -23539,8 +23651,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+ ": () => {
@@ -23620,9 +23732,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() {
@@ -23823,9 +23936,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           }
           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)) {
@@ -25079,7 +25190,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];
@@ -25113,6 +25224,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);
@@ -27194,7 +27321,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) {
@@ -29854,7 +29981,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"
@@ -29863,10 +29990,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 */
-                      : 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 */;
       }
@@ -29913,6 +30055,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];
       }
@@ -29957,6 +30103,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) {
@@ -30004,6 +30153,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       }
   }
   HeaderVisibilityPlugin.getters = [
+      "canRemoveHeaders",
       "getHiddenColsGroups",
       "getHiddenRowsGroups",
       "isRowHiddenByUser",
@@ -31013,13 +31163,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 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);
@@ -31491,7 +31655,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++) {
@@ -31618,10 +31782,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) {
@@ -31648,11 +31812,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) {
@@ -31682,11 +31846,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", {
@@ -31699,7 +31863,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", {
@@ -31717,7 +31881,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({
@@ -32269,8 +32433,12 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   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;
+                      }
                   }
               }
           }
@@ -32860,13 +33028,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 }) {
@@ -34972,13 +35138,33 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       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));
@@ -34986,7 +35172,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;
               }
@@ -39213,7 +39399,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;
       }
@@ -40020,7 +40206,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;
@@ -40063,9 +40249,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           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);
@@ -41779,8 +41963,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 */;
           }
@@ -41855,10 +42039,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,
           };
       }
@@ -42432,12 +42616,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 };
       }
   }
@@ -43539,6 +43720,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
@@ -43643,6 +43826,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);
           this.corePluginConfig = this.setupCorePluginConfig();
           this.uiPluginConfig = this.setupUiPluginConfig();
           // registering plugins
@@ -43651,15 +43836,23 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           }
           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
@@ -43681,12 +43874,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.allUIPlugins, this.history];
-      }
-      get allUIPlugins() {
-          return [...this.statefulUIPlugins, ...this.coreViewsPlugins, ...this.featurePlugins];
-      }
       joinSession() {
           this.session.join(this.config.client);
       }
@@ -43728,6 +43915,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) {
@@ -43870,7 +44059,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) {
@@ -43898,7 +44087,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);
       }
       garbageCollectExternalResources() {
@@ -44041,9 +44230,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __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';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);