@odoo/o-spreadsheet 17.4.2 → 17.4.4

package/dist/o-spreadsheet.iife.js

@@ -2,9 +2,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.4.2
- * @date 2024-07-24T10:26:00.237Z
- * @hash f5ede5b
+ * @version 17.4.4
+ * @date 2024-08-09T12:24:07.798Z
+ * @hash a43e855
  */
 
 (function (exports, owl) {
@@ -2766,13 +2766,16 @@
       }
       return new RegExp("^" + exp + "$", "i");
   });
-  function evaluatePredicate(value = "", criterion) {
+  function evaluatePredicate(value = "", criterion, locale) {
       const { operator, operand } = criterion;
       if (operand === undefined || value === null || operand === null) {
           return false;
       }
       if (typeof operand === "number" && operator === "=") {
-          return value.toString() === operand.toString();
+          if (typeof value === "string" && (isNumber(value, locale) || isDateTime(value, locale))) {
+              return toNumber(value, locale) === operand;
+          }
+          return value === operand;
       }
       if (operator === "<>" || operator === "=") {
           let result;
@@ -2854,7 +2857,7 @@
               for (let k = 0; k < countArg - 1; k += 2) {
                   const criteriaValue = toMatrix(args[k])[i][j].value;
                   const criterion = predicates[k / 2];
-                  validatedPredicates = evaluatePredicate(criteriaValue ?? undefined, criterion);
+                  validatedPredicates = evaluatePredicate(criteriaValue ?? undefined, criterion, locale);
                   if (!validatedPredicates) {
                       break;
                   }
@@ -6545,6 +6548,9 @@
       const groupValueString = typeof groupValue === "boolean"
           ? toString(groupValue).toLocaleLowerCase()
           : toString(groupValue);
+      if (groupValueString === "null") {
+          return null;
+      }
       if (!pivotNormalizationValueRegistry.contains(dimension.type)) {
           throw new EvaluationError(_t("Field %(field)s is not supported because of its type (%(type)s)", {
               field: dimension.displayName,
@@ -6565,6 +6571,9 @@
       return pivotTimeAdapter(granularity).normalizeFunctionValue(value);
   }
   function toFunctionPivotValue(value, dimension) {
+      if (value === null) {
+          return `"null"`;
+      }
       if (!pivotToFunctionValueRegistry.contains(dimension.type)) {
           return `"${value}"`;
       }
@@ -10568,6 +10577,18 @@ stores.inject(MyMetaStore, storeInstance);
       useLeftAxis ||= !useRightAxis;
       return { useLeftAxis, useRightAxis };
   }
+  function formatTickValue(localeFormat) {
+      return (value) => {
+          value = Number(value);
+          if (isNaN(value))
+              return value;
+          const { locale, format } = localeFormat;
+          return formatValue(value, {
+              locale,
+              format: !format && Math.abs(value) >= 1000 ? "#,##" : format,
+          });
+      };
+  }
 
   /** This is a chartJS plugin that will draw the values of each data next to the point/bar/pie slice */
   const chartShowValuesPlugin = {
@@ -19116,6 +19137,7 @@ stores.inject(MyMetaStore, storeInstance);
           const pivot = this.getters.getPivot(pivotId);
           const coreDefinition = this.getters.getPivotCoreDefinition(pivotId);
           addPivotDependencies(this, coreDefinition);
+          pivot.init({ reload: pivot.needsReevaluation });
           const error = pivot.assertIsValid({ throwOnError: false });
           if (error) {
               return error;
@@ -19144,6 +19166,7 @@ stores.inject(MyMetaStore, storeInstance);
           const pivot = this.getters.getPivot(_pivotId);
           const coreDefinition = this.getters.getPivotCoreDefinition(_pivotId);
           addPivotDependencies(this, coreDefinition);
+          pivot.init({ reload: pivot.needsReevaluation });
           const error = pivot.assertIsValid({ throwOnError: false });
           if (error) {
               return error;
@@ -20361,14 +20384,6 @@ stores.inject(MyMetaStore, storeInstance);
   }
   const notAvailableError = new NotAvailableError(_t("Array arguments to [[FUNCTION_NAME]] are of different size."));
   function createComputeFunction(descr, functionName) {
-      function runtimeCompute(...args) {
-          try {
-              return vectorizedCompute.apply(this, args);
-          }
-          catch (e) {
-              return handleError(e, functionName);
-          }
-      }
       function vectorizedCompute(...args) {
           let countVectorizableCol = 1;
           let countVectorizableRow = 1;
@@ -20409,13 +20424,13 @@ stores.inject(MyMetaStore, storeInstance);
                   }
               }
               if (!isMatrix(arg) && argDefinition.acceptMatrixOnly) {
-                  throw new BadExpressionError(_t("Function [[FUNCTION_NAME]] expects the parameter '%s' to be reference to a cell or range.", (i + 1).toString()));
+                  throw new BadExpressionError(_t("Function %s expects the parameter '%s' to be reference to a cell or range.", functionName, (i + 1).toString()));
               }
           }
           //#endregion
           if (countVectorizableCol === 1 && countVectorizableRow === 1) {
               // either this function is not vectorized or it ends up with a 1x1 dimension
-              return computeFunctionToObject.apply(this, args);
+              return errorHandlingCompute.apply(this, args);
           }
           const getArgOffset = (i, j) => args.map((arg, index) => {
               switch (vectorArgsType?.[index]) {
@@ -20433,7 +20448,7 @@ stores.inject(MyMetaStore, storeInstance);
               if (col > vectorizableColLimit - 1 || row > vectorizableRowLimit - 1) {
                   return notAvailableError;
               }
-              const singleCellComputeResult = computeFunctionToObject.apply(this, getArgOffset(col, row));
+              const singleCellComputeResult = errorHandlingCompute.apply(this, getArgOffset(col, row));
               // In the case where the user tries to vectorize arguments of an array formula, we will get an
               // array for every combination of the vectorized arguments, which will lead to a 3D matrix and
               // we won't be able to return the values.
@@ -20448,6 +20463,14 @@ stores.inject(MyMetaStore, storeInstance);
                   : singleCellComputeResult;
           });
       }
+      function errorHandlingCompute(...args) {
+          try {
+              return computeFunctionToObject.apply(this, args);
+          }
+          catch (e) {
+              return handleError(e, functionName);
+          }
+      }
       function computeFunctionToObject(...args) {
           const result = descr.compute.apply(this, args);
           if (!isMatrix(result)) {
@@ -20463,7 +20486,7 @@ stores.inject(MyMetaStore, storeInstance);
           }
           return matrixMap(result, (row) => ({ value: row }));
       }
-      return runtimeCompute;
+      return vectorizedCompute;
   }
   function handleError(e, functionName) {
       // the error could be an user error (instance of EvaluationError)
@@ -20780,6 +20803,78 @@ stores.inject(MyMetaStore, storeInstance);
       return isMacOS() ? ev.metaKey : ev.ctrlKey;
   }
 
+  /**
+   * Return the o-spreadsheet element position relative
+   * to the browser viewport.
+   */
+  function useSpreadsheetRect() {
+      const position = owl.useState({ x: 0, y: 0, width: 0, height: 0 });
+      let spreadsheetElement = null;
+      function updatePosition() {
+          if (!spreadsheetElement) {
+              spreadsheetElement = document.querySelector(".o-spreadsheet");
+          }
+          if (spreadsheetElement) {
+              const { top, left, width, height } = spreadsheetElement.getBoundingClientRect();
+              position.x = left;
+              position.y = top;
+              position.width = width;
+              position.height = height;
+          }
+      }
+      owl.onMounted(updatePosition);
+      owl.onPatched(updatePosition);
+      return position;
+  }
+  /**
+   * Return the component (or ref's component) BoundingRect, relative
+   * to the upper left corner of the screen (<body> element).
+   *
+   * Note: when used with a <Portal/> component, it will
+   * return the portal position, not the teleported position.
+   */
+  function useAbsoluteBoundingRect(ref) {
+      const rect = owl.useState({ x: 0, y: 0, width: 0, height: 0 });
+      function updateElRect() {
+          const el = ref.el;
+          if (el === null) {
+              return;
+          }
+          const { top, left, width, height } = el.getBoundingClientRect();
+          rect.x = left;
+          rect.y = top;
+          rect.width = width;
+          rect.height = height;
+      }
+      owl.onMounted(updateElRect);
+      owl.onPatched(updateElRect);
+      return rect;
+  }
+  /**
+   * Get the rectangle inside which a popover should stay when being displayed.
+   * It's the value defined in `env.getPopoverContainerRect`, or the Rect of the "o-spreadsheet"
+   * element by default.
+   *
+   * Coordinates are expressed expressed as absolute DOM position.
+   */
+  function usePopoverContainer() {
+      const container = owl.useState({ x: 0, y: 0, width: 0, height: 0 });
+      const component = owl.useComponent();
+      const spreadsheetRect = useSpreadsheetRect();
+      function updateRect() {
+          const env = component.env;
+          const newRect = "getPopoverContainerRect" in env ? env.getPopoverContainerRect() : spreadsheetRect;
+          container.x = newRect.x;
+          container.y = newRect.y;
+          container.width = newRect.width;
+          container.height = newRect.height;
+      }
+      updateRect();
+      owl.onMounted(updateRect);
+      owl.onPatched(updateRect);
+      return container;
+  }
+
   const arrowMap = {
       ArrowDown: "down",
       ArrowLeft: "left",
@@ -21369,7 +21464,9 @@ stores.inject(MyMetaStore, storeInstance);
           argToFocus: 0,
       });
       compositionActive = false;
+      spreadsheetRect = useSpreadsheetRect();
       get assistantStyle() {
+          const composerRect = this.composerRef.el.getBoundingClientRect();
           const assistantStyle = {};
           assistantStyle["min-width"] = `${this.props.rect?.width || ASSISTANT_WIDTH}px`;
           const proposals = this.autoCompleteState.provider?.proposals;
@@ -21396,6 +21493,9 @@ stores.inject(MyMetaStore, storeInstance);
           }
           else if (this.props.delimitation) {
               assistantStyle["max-height"] = `${this.props.delimitation.height}px`;
+              if (composerRect.left + ASSISTANT_WIDTH > this.spreadsheetRect.width) {
+                  assistantStyle.right = `0px`;
+              }
           }
           return cssPropertiesToCss(assistantStyle);
       }
@@ -23607,23 +23707,13 @@ stores.inject(MyMetaStore, storeInstance);
           padding: { left: 20, right: 20, top: chart.title ? 10 : 25, bottom: 10 },
       };
       config.options.indexAxis = chart.horizontal ? "y" : "x";
-      const formatCallback = (value) => {
-          value = Number(value);
-          if (isNaN(value))
-              return value;
-          const { locale, format } = localeFormat;
-          return formatValue(value, {
-              locale,
-              format: !format && Math.abs(value) >= 1000 ? "#,##" : format,
-          });
-      };
       config.options.scales = {};
       const labelsAxis = { ticks: { padding: 5, color: fontColor } };
       const valuesAxis = {
           beginAtZero: true, // the origin of the y axis is always zero
           ticks: {
               color: fontColor,
-              callback: formatCallback,
+              callback: formatTickValue(localeFormat),
           },
       };
       const xAxis = chart.horizontal ? valuesAxis : labelsAxis;
@@ -23660,7 +23750,7 @@ stores.inject(MyMetaStore, storeInstance);
           showValues: chart.showValues,
           background: chart.background,
           horizontal: chart.horizontal,
-          callback: formatCallback,
+          callback: formatTickValue(localeFormat),
       };
       return config;
   }
@@ -23939,21 +24029,11 @@ stores.inject(MyMetaStore, storeInstance);
               title: getChartAxisTitleRuntime(chart.axesDesign?.x),
           },
       };
-      const formatCallback = (value) => {
-          value = Number(value);
-          if (isNaN(value))
-              return value;
-          const { locale, format } = options;
-          return formatValue(value, {
-              locale,
-              format: !format && Math.abs(value) >= 1000 ? "#,##" : format,
-          });
-      };
       const yAxis = {
           beginAtZero: true, // the origin of the y axis is always zero
           ticks: {
               color: fontColor,
-              callback: formatCallback,
+              callback: formatTickValue(options),
           },
       };
       const { useLeftAxis, useRightAxis } = getDefinedAxis(chart.getDefinition());
@@ -23984,7 +24064,7 @@ stores.inject(MyMetaStore, storeInstance);
       config.options.plugins.chartShowValuesPlugin = {
           showValues: chart.showValues,
           background: chart.background,
-          callback: formatCallback,
+          callback: formatTickValue(options),
       };
       return config;
   }
@@ -24259,30 +24339,18 @@ stores.inject(MyMetaStore, storeInstance);
               title: getChartAxisTitleRuntime(chart.axesDesign?.x),
           },
       };
-      const formatCallback = (format) => {
-          return (value) => {
-              value = Number(value);
-              if (isNaN(value))
-                  return value;
-              const { locale } = localeFormat;
-              return formatValue(value, {
-                  locale,
-                  format: !format && Math.abs(value) >= 1000 ? "#,##" : format,
-              });
-          };
-      };
       const leftVerticalAxis = {
           beginAtZero: true, // the origin of the y axis is always zero
           ticks: {
               color: fontColor,
-              callback: formatCallback(mainDataSetFormat),
+              callback: formatTickValue({ format: mainDataSetFormat, locale }),
           },
       };
       const rightVerticalAxis = {
           beginAtZero: true, // the origin of the y axis is always zero
           ticks: {
               color: fontColor,
-              callback: formatCallback(lineDataSetsFormat),
+              callback: formatTickValue({ format: lineDataSetsFormat, locale }),
           },
       };
       const definition = chart.getDefinition();
@@ -24307,7 +24375,7 @@ stores.inject(MyMetaStore, storeInstance);
       config.options.plugins.chartShowValuesPlugin = {
           showValues: chart.showValues,
           background: chart.background,
-          callback: formatCallback(mainDataSetFormat),
+          callback: formatTickValue({ format: mainDataSetFormat, locale }),
       };
       const colors = new ColorGenerator();
       for (let [index, { label, data }] of dataSetsValues.entries()) {
@@ -24839,7 +24907,10 @@ stores.inject(MyMetaStore, storeInstance);
           const yLabelStr = formatValue(yLabel, { format: toolTipFormat, locale });
           return xLabel ? `${xLabel}: ${yLabelStr} (${percentage}%)` : `${yLabelStr} (${percentage}%)`;
       };
-      config.options.plugins.chartShowValuesPlugin = { showValues: chart.showValues };
+      config.options.plugins.chartShowValuesPlugin = {
+          showValues: chart.showValues,
+          callback: formatTickValue(localeFormat),
+      };
       return config;
   }
   function getPieColors(colors, dataSetsValues) {
@@ -25423,6 +25494,7 @@ stores.inject(MyMetaStore, storeInstance);
       config.options.plugins.chartShowValuesPlugin = {
           showValues: chart.showValues,
           background: chart.background,
+          callback: formatTickValue(localeFormat),
       };
       return config;
   }
@@ -26238,78 +26310,6 @@ stores.inject(MyMetaStore, storeInstance);
       };
   }
 
-  /**
-   * Return the o-spreadsheet element position relative
-   * to the browser viewport.
-   */
-  function useSpreadsheetRect() {
-      const position = owl.useState({ x: 0, y: 0, width: 0, height: 0 });
-      let spreadsheetElement = null;
-      function updatePosition() {
-          if (!spreadsheetElement) {
-              spreadsheetElement = document.querySelector(".o-spreadsheet");
-          }
-          if (spreadsheetElement) {
-              const { top, left, width, height } = spreadsheetElement.getBoundingClientRect();
-              position.x = left;
-              position.y = top;
-              position.width = width;
-              position.height = height;
-          }
-      }
-      owl.onMounted(updatePosition);
-      owl.onPatched(updatePosition);
-      return position;
-  }
-  /**
-   * Return the component (or ref's component) BoundingRect, relative
-   * to the upper left corner of the screen (<body> element).
-   *
-   * Note: when used with a <Portal/> component, it will
-   * return the portal position, not the teleported position.
-   */
-  function useAbsoluteBoundingRect(ref) {
-      const rect = owl.useState({ x: 0, y: 0, width: 0, height: 0 });
-      function updateElRect() {
-          const el = ref.el;
-          if (el === null) {
-              return;
-          }
-          const { top, left, width, height } = el.getBoundingClientRect();
-          rect.x = left;
-          rect.y = top;
-          rect.width = width;
-          rect.height = height;
-      }
-      owl.onMounted(updateElRect);
-      owl.onPatched(updateElRect);
-      return rect;
-  }
-  /**
-   * Get the rectangle inside which a popover should stay when being displayed.
-   * It's the value defined in `env.getPopoverContainerRect`, or the Rect of the "o-spreadsheet"
-   * element by default.
-   *
-   * Coordinates are expressed expressed as absolute DOM position.
-   */
-  function usePopoverContainer() {
-      const container = owl.useState({ x: 0, y: 0, width: 0, height: 0 });
-      const component = owl.useComponent();
-      const spreadsheetRect = useSpreadsheetRect();
-      function updateRect() {
-          const env = component.env;
-          const newRect = "getPopoverContainerRect" in env ? env.getPopoverContainerRect() : spreadsheetRect;
-          container.x = newRect.x;
-          container.y = newRect.y;
-          container.width = newRect.width;
-          container.height = newRect.height;
-      }
-      updateRect();
-      owl.onMounted(updateRect);
-      owl.onPatched(updateRect);
-      return container;
-  }
-
   css /* scss */ `
   .o-popover {
     position: absolute;
@@ -35808,6 +35808,14 @@ stores.inject(MyMetaStore, storeInstance);
     .pivot-dim-operator-label {
       min-width: 120px;
     }
+
+    &.pivot-dimension-invalid {
+      background-color: #ffdddd;
+      border-color: red !important;
+      select {
+        background-color: #ffdddd;
+      }
+    }
   }
 `;
   class PivotDimension extends owl.Component {
@@ -47215,12 +47223,13 @@ stores.inject(MyMetaStore, storeInstance);
                   this.clearBorders(cmd.sheetId, cmd.target);
                   break;
               case "REMOVE_COLUMNS_ROWS":
-                  for (let el of [...cmd.elements].sort((a, b) => b - a)) {
+                  const elements = [...cmd.elements].sort((a, b) => b - a);
+                  for (const group of groupConsecutive(elements)) {
                       if (cmd.dimension === "COL") {
-                          this.shiftBordersHorizontally(cmd.sheetId, el + 1, -1);
+                          this.shiftBordersHorizontally(cmd.sheetId, group[group.length - 1] + 1, -group.length);
                       }
                       else {
-                          this.shiftBordersVertically(cmd.sheetId, el + 1, -1);
+                          this.shiftBordersVertically(cmd.sheetId, group[group.length - 1] + 1, -group.length);
                       }
                   }
                   break;
@@ -50390,7 +50399,7 @@ stores.inject(MyMetaStore, storeInstance);
        * @param sheetXC the string description of a range, in the form SheetName!XC:XC
        */
       getRangeFromSheetXC(defaultSheetId, sheetXC) {
-          if (!rangeReference.test(sheetXC)) {
+          if (!rangeReference.test(sheetXC) || !this.getters.tryGetSheet(defaultSheetId)) {
               return new RangeImpl({
                   sheetId: "",
                   zone: { left: -1, top: -1, right: -1, bottom: -1 },
@@ -51281,12 +51290,7 @@ stores.inject(MyMetaStore, storeInstance);
                           });
                       }
                       if (colIndex > deletedColumn) {
-                          this.dispatch("UPDATE_CELL_POSITION", {
-                              sheetId: sheet.id,
-                              cellId: cellId,
-                              col: colIndex - 1,
-                              row: rowIndex,
-                          });
+                          this.setNewPosition(cellId, sheet.id, colIndex - 1, rowIndex);
                       }
                   }
               }
@@ -51296,7 +51300,7 @@ stores.inject(MyMetaStore, storeInstance);
        * Move the cells after a column or rows insertion
        */
       moveCellsOnAddition(sheet, addedElement, quantity, dimension) {
-          const commands = [];
+          const updates = [];
           for (let rowIndex = 0; rowIndex < sheet.rows.length; rowIndex++) {
               const row = sheet.rows[rowIndex];
               if (dimension !== "rows" || rowIndex >= addedElement) {
@@ -51305,20 +51309,20 @@ stores.inject(MyMetaStore, storeInstance);
                       const cellId = row.cells[i];
                       if (cellId) {
                           if (dimension === "rows" || colIndex >= addedElement) {
-                              commands.push({
-                                  type: "UPDATE_CELL_POSITION",
+                              updates.push({
                                   sheetId: sheet.id,
                                   cellId: cellId,
                                   col: colIndex + (dimension === "columns" ? quantity : 0),
                                   row: rowIndex + (dimension === "rows" ? quantity : 0),
+                                  type: "UPDATE_CELL_POSITION",
                               });
                           }
                       }
                   }
               }
           }
-          for (let cmd of commands.reverse()) {
-              this.dispatch(cmd.type, cmd);
+          for (let update of updates.reverse()) {
+              this.updateCellPosition(update);
           }
       }
       /**
@@ -51351,12 +51355,7 @@ stores.inject(MyMetaStore, storeInstance);
                       const colIndex = Number(i);
                       const cellId = row.cells[i];
                       if (cellId) {
-                          this.dispatch("UPDATE_CELL_POSITION", {
-                              sheetId: sheet.id,
-                              cellId: cellId,
-                              col: colIndex,
-                              row: rowIndex - numberRows,
-                          });
+                          this.setNewPosition(cellId, sheet.id, colIndex, rowIndex - numberRows);
                       }
                   }
               }
@@ -54053,18 +54052,23 @@ stores.inject(MyMetaStore, storeInstance);
           this.evaluate(this.getAllCells());
           console.info("evaluate all cells", performance.now() - start, "ms");
       }
-      evaluateFormula(sheetId, formulaString) {
-          const compiledFormula = compile(formulaString);
-          const ranges = compiledFormula.dependencies.map((xc) => this.getters.getRangeFromSheetXC(sheetId, xc));
-          this.updateCompilationParameters();
-          const result = updateEvalContextAndExecute({ ...compiledFormula, dependencies: ranges }, this.compilationParams, sheetId);
-          if (isMatrix(result)) {
-              return matrixMap(result, (cell) => cell.value);
+      evaluateFormulaResult(sheetId, formulaString) {
+          try {
+              const compiledFormula = compile(formulaString);
+              const ranges = compiledFormula.dependencies.map((xc) => this.getters.getRangeFromSheetXC(sheetId, xc));
+              this.updateCompilationParameters();
+              const result = updateEvalContextAndExecute({ ...compiledFormula, dependencies: ranges }, this.compilationParams, sheetId);
+              if (isMatrix(result)) {
+                  return result;
+              }
+              if (result.value === null) {
+                  return { value: 0, format: result.format };
+              }
+              return result;
           }
-          if (result.value === null) {
-              return 0;
+          catch (error) {
+              return handleError(error, "");
           }
-          return result.value;
       }
       getAllCells() {
           const positions = this.createEmptyPositionSet();
@@ -54122,6 +54126,9 @@ stores.inject(MyMetaStore, storeInstance);
           if (!this.blockedArrayFormulas.has(position)) {
               this.invalidateSpreading(position);
           }
+          if (this.spreadingRelations.isArrayFormula(position)) {
+              this.spreadingRelations.removeNode(position);
+          }
           const cell = this.getters.getCell(position);
           if (cell === undefined) {
               return EMPTY_CELL;
@@ -54164,7 +54171,6 @@ stores.inject(MyMetaStore, storeInstance);
           this.assertSheetHasEnoughSpaceToSpreadFormulaResult(formulaPosition, formulaReturn);
           const nbColumns = formulaReturn.length;
           const nbRows = formulaReturn[0].length;
-          this.spreadingRelations.removeNode(formulaPosition);
           forEachSpreadPositionInMatrix(nbColumns, nbRows, this.updateSpreadRelation(formulaPosition));
           this.assertNoMergedCellsInSpreadZone(formulaPosition, formulaReturn);
           forEachSpreadPositionInMatrix(nbColumns, nbRows, this.checkCollision(formulaPosition));
@@ -54425,6 +54431,7 @@ stores.inject(MyMetaStore, storeInstance);
   class EvaluationPlugin extends UIPlugin {
       static getters = [
           "evaluateFormula",
+          "evaluateFormulaResult",
           "getCorrespondingFormulaCell",
           "getRangeFormattedValues",
           "getRangeValues",
@@ -54484,12 +54491,14 @@ stores.inject(MyMetaStore, storeInstance);
       // Getters
       // ---------------------------------------------------------------------------
       evaluateFormula(sheetId, formulaString) {
-          try {
-              return this.evaluator.evaluateFormula(sheetId, formulaString);
-          }
-          catch (error) {
-              return error.value || CellErrorType.GenericError;
+          const result = this.evaluateFormulaResult(sheetId, formulaString);
+          if (isMatrix(result)) {
+              return matrixMap(result, (cell) => cell.value);
           }
+          return result.value;
+      }
+      evaluateFormulaResult(sheetId, formulaString) {
+          return this.evaluator.evaluateFormulaResult(sheetId, formulaString);
       }
       /**
        * Return the value of each cell in the range as they are displayed in the grid.
@@ -55872,28 +55881,33 @@ stores.inject(MyMetaStore, storeInstance);
               const pivotRow = position.row - mainPosition.row;
               return pivotCells[pivotCol][pivotRow];
           }
-          if (functionName === "PIVOT.HEADER" && args.at(-2) === "measure") {
-              const domain = pivot.parseArgsToPivotDomain(args.slice(1, -2).map((value) => ({ value })));
+          try {
+              if (functionName === "PIVOT.HEADER" && args.at(-2) === "measure") {
+                  const domain = pivot.parseArgsToPivotDomain(args.slice(1, -2).map((value) => ({ value })));
+                  return {
+                      type: "MEASURE_HEADER",
+                      domain,
+                      measure: args.at(-1)?.toString() || "",
+                  };
+              }
+              else if (functionName === "PIVOT.HEADER") {
+                  const domain = pivot.parseArgsToPivotDomain(args.slice(1).map((value) => ({ value })));
+                  return {
+                      type: "HEADER",
+                      domain,
+                  };
+              }
+              const [measure, ...domainArgs] = args.slice(1);
+              const domain = pivot.parseArgsToPivotDomain(domainArgs.map((value) => ({ value })));
               return {
-                  type: "MEASURE_HEADER",
+                  type: "VALUE",
                   domain,
-                  measure: args.at(-1)?.toString() || "",
+                  measure: measure?.toString() || "",
               };
           }
-          else if (functionName === "PIVOT.HEADER") {
-              const domain = pivot.parseArgsToPivotDomain(args.slice(1).map((value) => ({ value })));
-              return {
-                  type: "HEADER",
-                  domain,
-              };
+          catch (_) {
+              return EMPTY_PIVOT_CELL;
           }
-          const [measure, ...domainArgs] = args.slice(1);
-          const domain = pivot.parseArgsToPivotDomain(domainArgs.map((value) => ({ value })));
-          return {
-              type: "VALUE",
-              domain,
-              measure: measure?.toString() || "",
-          };
       }
       getPivot(pivotId) {
           return this.pivots[pivotId];
@@ -57761,6 +57775,7 @@ stores.inject(MyMetaStore, storeInstance);
        * evaluated and updated with the number type.
        */
       setDecimal(sheetId, zones, step) {
+          const positionsByFormat = {};
           // Find the each cell with a number value and get the format
           for (const zone of recomputeZones(zones)) {
               for (const position of positions(zone)) {
@@ -57770,15 +57785,20 @@ stores.inject(MyMetaStore, storeInstance);
                       // of the format
                       this.getters.getLocale();
                       const newFormat = changeDecimalPlaces(numberFormat, step);
-                      // Apply the new format on the whole zone
-                      this.dispatch("SET_FORMATTING", {
-                          sheetId,
-                          target: [positionToZone(position)],
-                          format: newFormat,
-                      });
+                      positionsByFormat[newFormat] = positionsByFormat[newFormat] || [];
+                      positionsByFormat[newFormat].push(position);
                   }
               }
           }
+          // consolidate all positions with the same format in bigger zones
+          for (const newFormat in positionsByFormat) {
+              const zones = recomputeZones(positionsByFormat[newFormat].map((position) => positionToZone(position)));
+              this.dispatch("SET_FORMATTING", {
+                  sheetId,
+                  format: newFormat,
+                  target: zones,
+              });
+          }
       }
       /**
        * Take a range of cells and return the format of the first cell containing a
@@ -63089,6 +63109,8 @@ stores.inject(MyMetaStore, storeInstance);
 
     .o-sidePanel-handle-container {
       width: 8px;
+      position: fixed;
+      top: 50%;
     }
     .o-sidePanel-handle {
       cursor: col-resize;
@@ -65708,7 +65730,7 @@ stores.inject(MyMetaStore, storeInstance);
   }
 
   class StateObserver {
-      changes = [];
+      changes;
       commands = [];
       /**
        * Record the changes which could happen in the given callback, save them in a
@@ -65740,7 +65762,7 @@ stores.inject(MyMetaStore, storeInstance);
           if (value[key] === val) {
               return;
           }
-          this.changes.push({
+          this.changes?.push({
               key,
               target: value,
               before: value[key],
@@ -68406,9 +68428,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.4.2";
-  __info__.date = "2024-07-24T10:26:00.237Z";
-  __info__.hash = "f5ede5b";
+  __info__.version = "17.4.4";
+  __info__.date = "2024-08-09T12:24:07.798Z";
+  __info__.hash = "a43e855";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);