@odoo/o-spreadsheet 18.2.31 → 18.2.33

package/dist/o-spreadsheet.esm.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 18.2.31
- * @date 2025-09-23T12:31:15.573Z
- * @hash 839667e
+ * @version 18.2.33
+ * @date 2025-10-16T06:39:40.421Z
+ * @hash 280596c
  */
 
 import { useEnv, useSubEnv, onWillUnmount, useComponent, status, Component, useRef, onMounted, useEffect, App, blockDom, useState, onPatched, onWillPatch, onWillUpdateProps, useExternalListener, onWillStart, xml, useChildSubEnv, markRaw, toRaw } from '@odoo/owl';
@@ -873,9 +873,7 @@ function removeIndexesFromArray(array, indexes) {
     return newArray;
 }
 function insertItemsAtIndex(array, items, index) {
-    const newArray = [...array];
-    newArray.splice(index, 0, ...items);
-    return newArray;
+    return array.slice(0, index).concat(items).concat(array.slice(index));
 }
 function replaceItemAtIndex(array, newItem, index) {
     const newArray = [...array];
@@ -3916,7 +3914,17 @@ function toNumberMatrix(data, argName) {
     return toMatrix(data).map((row) => {
         return row.map((cell) => {
             if (typeof cell.value !== "number") {
-                throw new EvaluationError(_t("Function [[FUNCTION_NAME]] expects number values for %s, but got a %s.", argName, typeof cell.value));
+                let message = "";
+                if (typeof cell === "object") {
+                    message = _t("Function [[FUNCTION_NAME]] expects number values for %s, but got an empty value.", argName);
+                }
+                else if (typeof cell === "string") {
+                    message = _t("Function [[FUNCTION_NAME]] expects number values for %s, but got a string.", argName);
+                }
+                else if (typeof cell === "boolean") {
+                    message = _t("Function [[FUNCTION_NAME]] expects number values for %s, but got a boolean.", argName);
+                }
+                throw new EvaluationError(message);
             }
             return cell.value;
         });
@@ -5116,7 +5124,7 @@ function tokensToTextInternalFormat(tokens) {
  * Replace in place tokens "mm" and "m" that denote minutes in date format with "MM" to avoid confusion with months.
  *
  * As per OpenXML specification, in date formats if a date token "m" or "mm" is followed by a date token "s" or
- * preceded by a data token "h", then it's not a month but an minute.
+ * preceded by a data token "h", then it's not a month but a minute.
  */
 function convertTokensToMinutesInDateFormat(tokens) {
     const dateParts = tokens.filter((token) => token.type === "DATE_PART");
@@ -5159,6 +5167,9 @@ function internalFormatPartToFormat(internalFormat) {
             case "REPEATED_CHAR":
                 format += "*" + token.value;
                 break;
+            case "DATE_PART":
+                format += token.value === "MM" ? "mm" : token.value; // Convert "MM" back to "mm" for minutes
+                break;
             default:
                 format += token.value;
         }
@@ -8815,7 +8826,7 @@ class CellClipboardHandler extends AbstractCellClipboardHandler {
     pasteCell(origin, target, clipboardOption) {
         const { sheetId, col, row } = target;
         const targetCell = this.getters.getEvaluatedCell(target);
-        const originFormat = origin?.format ?? origin.evaluatedCell.format;
+        const originFormat = origin?.format || origin.evaluatedCell.format;
         if (clipboardOption?.pasteOption === "asValue") {
             this.dispatch("UPDATE_CELL", {
                 ...target,
@@ -10436,6 +10447,10 @@ const chartShowValuesPlugin = {
         }
         const ctx = chart.ctx;
         ctx.save();
+        const { left, top, height, width } = chart.chartArea;
+        ctx.beginPath();
+        ctx.rect(left, top, width, height);
+        ctx.clip();
         ctx.textAlign = "center";
         ctx.textBaseline = "middle";
         ctx.miterLimit = 1; // Avoid sharp artifacts on strokeText
@@ -13641,7 +13656,7 @@ const GROWTH = {
     ],
     compute: function (knownDataY, knownDataX = [[]], newDataX = [[]], b = { value: true }) {
         assertNonEmptyMatrix(knownDataY, "known_data_y");
-        return expM(predictLinearValues(logM(toNumberMatrix(knownDataY, "the first argument (known_data_y)")), toNumberMatrix(knownDataX, "the second argument (known_data_x)"), toNumberMatrix(newDataX, "the third argument (new_data_y)"), toBoolean(b)));
+        return expM(predictLinearValues(logM(toNumberMatrix(knownDataY, "known_data_y")), toNumberMatrix(knownDataX, "known_data_x"), toNumberMatrix(newDataX, "new_data_y"), toBoolean(b)));
     },
 };
 // -----------------------------------------------------------------------------
@@ -13706,7 +13721,7 @@ const LINEST = {
     ],
     compute: function (dataY, dataX = [[]], calculateB = { value: true }, verbose = { value: false }) {
         assertNonEmptyMatrix(dataY, "data_y");
-        return fullLinearRegression(toNumberMatrix(dataX, "the first argument (data_y)"), toNumberMatrix(dataY, "the second argument (data_x)"), toBoolean(calculateB), toBoolean(verbose));
+        return fullLinearRegression(toNumberMatrix(dataX, "data_x"), toNumberMatrix(dataY, "data_y"), toBoolean(calculateB), toBoolean(verbose));
     },
     isExported: true,
 };
@@ -13723,7 +13738,7 @@ const LOGEST = {
     ],
     compute: function (dataY, dataX = [[]], calculateB = { value: true }, verbose = { value: false }) {
         assertNonEmptyMatrix(dataY, "data_y");
-        const coeffs = fullLinearRegression(toNumberMatrix(dataX, "the second argument (data_x)"), logM(toNumberMatrix(dataY, "the first argument (data_y)")), toBoolean(calculateB), toBoolean(verbose));
+        const coeffs = fullLinearRegression(toNumberMatrix(dataX, "data_x"), logM(toNumberMatrix(dataY, "data_y")), toBoolean(calculateB), toBoolean(verbose));
         for (let i = 0; i < coeffs.length; i++) {
             coeffs[i][0] = Math.exp(coeffs[i][0]);
         }
@@ -14313,7 +14328,7 @@ const TREND = {
     ],
     compute: function (knownDataY, knownDataX = [[]], newDataX = [[]], b = { value: true }) {
         assertNonEmptyMatrix(knownDataY, "known_data_y");
-        return predictLinearValues(toNumberMatrix(knownDataY, "the first argument (known_data_y)"), toNumberMatrix(knownDataX, "the second argument (known_data_x)"), toNumberMatrix(newDataX, "the third argument (new_data_y)"), toBoolean(b));
+        return predictLinearValues(toNumberMatrix(knownDataY, "known_data_y"), toNumberMatrix(knownDataX, "known_data_x"), toNumberMatrix(newDataX, "new_data_y"), toBoolean(b));
     },
 };
 // -----------------------------------------------------------------------------
@@ -23744,6 +23759,74 @@ iconsOnCellRegistry.add("conditional_formatting", (getters, position) => {
     }
 });
 
+/**
+ * Get the relative path between two files
+ *
+ * Eg.:
+ * from "folder1/file1.txt" to "folder2/file2.txt" => "../folder2/file2.txt"
+ */
+function getRelativePath(from, to) {
+    const fromPathParts = from.split("/");
+    const toPathParts = to.split("/");
+    let relPath = "";
+    let startIndex = 0;
+    for (let i = 0; i < fromPathParts.length - 1; i++) {
+        if (fromPathParts[i] === toPathParts[i]) {
+            startIndex++;
+        }
+        else {
+            relPath += "../";
+        }
+    }
+    relPath += toPathParts.slice(startIndex).join("/");
+    return relPath;
+}
+/**
+ * Convert an array of element into an object where the objects keys were the elements position in the array.
+ * Can give an offset as argument, and all the array indexes will we shifted by this offset in the returned object.
+ *
+ * eg. : ["a", "b"] => {0:"a", 1:"b"}
+ */
+function arrayToObject(array, indexOffset = 0) {
+    const obj = {};
+    for (let i = 0; i < array.length; i++) {
+        if (array[i]) {
+            obj[i + indexOffset] = array[i];
+        }
+    }
+    return obj;
+}
+/**
+ * In xlsx we can have string with unicode characters with the format _x00fa_.
+ * Replace with characters understandable by JS
+ */
+function fixXlsxUnicode(str) {
+    return str.replace(/_x([0-9a-zA-Z]{4})_/g, (match, code) => {
+        return String.fromCharCode(parseInt(code, 16));
+    });
+}
+/** Get a header in the SheetData. Create the header if it doesn't exist in the SheetData */
+function getSheetDataHeader(sheetData, dimension, index) {
+    if (dimension === "COL") {
+        if (!sheetData.cols[index]) {
+            sheetData.cols[index] = {};
+        }
+        return sheetData.cols[index];
+    }
+    if (!sheetData.rows[index]) {
+        sheetData.rows[index] = {};
+    }
+    return sheetData.rows[index];
+}
+/** Prefix the string by "=" if the string looks like a formula */
+function prefixFormulaWithEqual(formula) {
+    if (formula[0] === "=") {
+        return formula;
+    }
+    const tokens = tokenize(formula);
+    return tokens.length === 1 && tokens[0].type !== "REFERENCE" ? formula : "=" + formula;
+}
+
 /**
  * Map of the different types of conversions warnings and their name in error messages
  */
@@ -24251,66 +24334,6 @@ function hexaToInt(hex) {
  */
 const DEFAULT_SYSTEM_COLOR = "FF000000";
 
-/**
- * Get the relative path between two files
- *
- * Eg.:
- * from "folder1/file1.txt" to "folder2/file2.txt" => "../folder2/file2.txt"
- */
-function getRelativePath(from, to) {
-    const fromPathParts = from.split("/");
-    const toPathParts = to.split("/");
-    let relPath = "";
-    let startIndex = 0;
-    for (let i = 0; i < fromPathParts.length - 1; i++) {
-        if (fromPathParts[i] === toPathParts[i]) {
-            startIndex++;
-        }
-        else {
-            relPath += "../";
-        }
-    }
-    relPath += toPathParts.slice(startIndex).join("/");
-    return relPath;
-}
-/**
- * Convert an array of element into an object where the objects keys were the elements position in the array.
- * Can give an offset as argument, and all the array indexes will we shifted by this offset in the returned object.
- *
- * eg. : ["a", "b"] => {0:"a", 1:"b"}
- */
-function arrayToObject(array, indexOffset = 0) {
-    const obj = {};
-    for (let i = 0; i < array.length; i++) {
-        if (array[i]) {
-            obj[i + indexOffset] = array[i];
-        }
-    }
-    return obj;
-}
-/**
- * In xlsx we can have string with unicode characters with the format _x00fa_.
- * Replace with characters understandable by JS
- */
-function fixXlsxUnicode(str) {
-    return str.replace(/_x([0-9a-zA-Z]{4})_/g, (match, code) => {
-        return String.fromCharCode(parseInt(code, 16));
-    });
-}
-/** Get a header in the SheetData. Create the header if it doesn't exist in the SheetData */
-function getSheetDataHeader(sheetData, dimension, index) {
-    if (dimension === "COL") {
-        if (!sheetData.cols[index]) {
-            sheetData.cols[index] = {};
-        }
-        return sheetData.cols[index];
-    }
-    if (!sheetData.rows[index]) {
-        sheetData.rows[index] = {};
-    }
-    return sheetData.rows[index];
-}
-
 const XLSX_DATE_FORMAT_REGEX = /^(yy|yyyy|m{1,5}|d{1,4}|h{1,2}|s{1,2}|am\/pm|a\/m|\s|-|\/|\.|:)+$/i;
 /**
  * Convert excel format to o_spreadsheet format
@@ -24520,9 +24543,9 @@ function convertConditionalFormats(xlsxCfs, dxfs, warningManager) {
                 if (!rule.operator || !rule.formula || rule.formula.length === 0)
                     continue;
                 operator = convertCFCellIsOperator(rule.operator);
-                values.push(prefixFormula(rule.formula[0]));
+                values.push(prefixFormulaWithEqual(rule.formula[0]));
                 if (rule.formula.length === 2) {
-                    values.push(prefixFormula(rule.formula[1]));
+                    values.push(prefixFormulaWithEqual(rule.formula[1]));
                 }
                 break;
         }
@@ -24680,11 +24703,6 @@ function convertIcons(xlsxIconSet, index) {
             ? ICON_SETS[iconSet].neutral
             : ICON_SETS[iconSet].good;
 }
-/** Prefix the string by "=" if the string looks like a formula */
-function prefixFormula(formula) {
-    const tokens = tokenize(formula);
-    return tokens.length === 1 && tokens[0].type !== "REFERENCE" ? formula : "=" + formula;
-}
 // ---------------------------------------------------------------------------
 // Warnings
 // ---------------------------------------------------------------------------
@@ -25105,7 +25123,7 @@ function convertDataValidationRules(xlsxDataValidations, warningManager) {
                 dvRules.push(decimalRule);
                 break;
             case "list":
-                const listRule = convertListrule(dvId++, dv);
+                const listRule = convertListRule(dvId++, dv);
                 dvRules.push(listRule);
                 break;
             case "date":
@@ -25125,9 +25143,9 @@ function convertDataValidationRules(xlsxDataValidations, warningManager) {
     return dvRules;
 }
 function convertDecimalRule(id, dv) {
-    const values = [dv.formula1.toString()];
+    const values = [prefixFormulaWithEqual(dv.formula1.toString())];
     if (dv.formula2) {
-        values.push(dv.formula2.toString());
+        values.push(prefixFormulaWithEqual(dv.formula2.toString()));
     }
     return {
         id: id.toString(),
@@ -25139,7 +25157,7 @@ function convertDecimalRule(id, dv) {
         },
     };
 }
-function convertListrule(id, dv) {
+function convertListRule(id, dv) {
     const formula1 = dv.formula1.toString();
     const isRangeRule = rangeReference.test(formula1);
     return {
@@ -25155,9 +25173,9 @@ function convertListrule(id, dv) {
 }
 function convertDateRule(id, dv) {
     let criterion;
-    const values = [dv.formula1.toString()];
+    const values = [prefixFormulaWithEqual(dv.formula1.toString())];
     if (dv.formula2) {
-        values.push(dv.formula2.toString());
+        values.push(prefixFormulaWithEqual(dv.formula2.toString()));
         criterion = {
             type: XLSX_DV_DATE_OPERATOR_TO_DV_TYPE_MAPPING[dv.operator],
             values: getDateCriterionFormattedValues(values, DEFAULT_LOCALE),
@@ -25184,7 +25202,7 @@ function convertCustomRule(id, dv) {
         isBlocking: dv.errorStyle !== "warning",
         criterion: {
             type: "customFormula",
-            values: [`=${dv.formula1.toString()}`],
+            values: [prefixFormulaWithEqual(dv.formula1.toString())],
         },
     };
 }
@@ -28916,6 +28934,7 @@ function getChartTimeOptions(labels, labelFormat, locale) {
         parser: luxonFormat,
         displayFormats,
         unit: timeUnit ?? false,
+        tooltipFormat: luxonFormat,
     };
 }
 /**
@@ -30094,6 +30113,7 @@ function getLineChartScales(definition, args) {
         };
         Object.assign(scales.x, axis);
         scales.x.ticks.maxTicksLimit = 15;
+        delete scales?.x?.ticks?.callback;
     }
     else if (axisType === "linear") {
         scales.x.type = "linear";
@@ -44549,12 +44569,13 @@ class DataValidationEditor extends Component {
         onCloseSidePanel: { type: Function, optional: true },
     };
     state = useState({ rule: this.defaultDataValidationRule, errors: [] });
+    editingSheetId;
     setup() {
+        this.editingSheetId = this.env.model.getters.getActiveSheetId();
         if (this.props.rule) {
-            const sheetId = this.env.model.getters.getActiveSheetId();
             this.state.rule = {
                 ...this.props.rule,
-                ranges: this.props.rule.ranges.map((range) => this.env.model.getters.getRangeString(range, sheetId)),
+                ranges: this.props.rule.ranges.map((range) => this.env.model.getters.getRangeString(range, this.editingSheetId)),
             };
             this.state.rule.criterion.type = this.props.rule.criterion.type;
         }
@@ -44588,7 +44609,6 @@ class DataValidationEditor extends Component {
         const locale = this.env.model.getters.getLocale();
         const criterion = rule.criterion;
         const criterionEvaluator = dataValidationEvaluatorRegistry.get(criterion.type);
-        const sheetId = this.env.model.getters.getActiveSheetId();
         const values = criterion.values
             .slice(0, criterionEvaluator.numberOfValues(criterion))
             .map((value) => value?.trim())
@@ -44596,8 +44616,8 @@ class DataValidationEditor extends Component {
             .map((value) => canonicalizeContent(value, locale));
         rule.criterion = { ...criterion, values };
         return {
-            sheetId,
-            ranges: this.state.rule.ranges.map((xc) => this.env.model.getters.getRangeDataFromXc(sheetId, xc)),
+            sheetId: this.editingSheetId,
+            ranges: this.state.rule.ranges.map((xc) => this.env.model.getters.getRangeDataFromXc(this.editingSheetId, xc)),
             rule,
         };
     }
@@ -45127,6 +45147,7 @@ css /* scss */ `
       .o-button {
         height: 19px;
         width: 19px;
+        box-sizing: content-box;
         .o-icon {
           height: 14px;
           width: 14px;
@@ -45884,7 +45905,7 @@ class PivotMeasureEditor extends Component {
         return undefined;
     }
     get isCalculatedMeasureInvalid() {
-        return this.env.model.getters.getMeasureCompiledFormula(this.props.measure).isBadExpression;
+        return compile(this.props.measure.computedBy?.formula ?? "").isBadExpression;
     }
 }
 
@@ -56622,7 +56643,7 @@ class HeaderSizePlugin extends CorePlugin {
                 let sizes = [...this.sizes[cmd.sheetId][cmd.dimension]];
                 const addIndex = getAddHeaderStartIndex(cmd.position, cmd.base);
                 const baseSize = sizes[cmd.base];
-                sizes.splice(addIndex, 0, ...Array(cmd.quantity).fill(baseSize));
+                sizes = insertItemsAtIndex(sizes, Array(cmd.quantity).fill(baseSize), addIndex);
                 this.history.update("sizes", cmd.sheetId, cmd.dimension, sizes);
                 break;
             }
@@ -56774,9 +56795,8 @@ class HeaderVisibilityPlugin extends CorePlugin {
                 break;
             }
             case "ADD_COLUMNS_ROWS": {
-                const hiddenHeaders = [...this.hiddenHeaders[cmd.sheetId][cmd.dimension]];
                 const addIndex = getAddHeaderStartIndex(cmd.position, cmd.base);
-                hiddenHeaders.splice(addIndex, 0, ...Array(cmd.quantity).fill(false));
+                const hiddenHeaders = insertItemsAtIndex([...this.hiddenHeaders[cmd.sheetId][cmd.dimension]], Array(cmd.quantity).fill(false), addIndex);
                 this.history.update("hiddenHeaders", cmd.sheetId, cmd.dimension, hiddenHeaders);
                 break;
             }
@@ -61059,12 +61079,12 @@ class SpreadsheetRTree {
         this.rTrees[sheetId].remove(item, this.rtreeItemComparer);
     }
     rtreeItemComparer(left, right) {
-        return (left.data == right.data &&
-            left.boundingBox.sheetId === right.boundingBox.sheetId &&
+        return (left.boundingBox.sheetId === right.boundingBox.sheetId &&
             left.boundingBox?.zone.left === right.boundingBox.zone.left &&
             left.boundingBox?.zone.top === right.boundingBox.zone.top &&
             left.boundingBox?.zone.right === right.boundingBox.zone.right &&
-            left.boundingBox?.zone.bottom === right.boundingBox.zone.bottom);
+            left.boundingBox?.zone.bottom === right.boundingBox.zone.bottom &&
+            deepEquals(left.data, right.data));
     }
 }
 /**
@@ -61137,7 +61157,7 @@ class FormulaDependencyGraph {
      * in the correct order they should be evaluated.
      * This is called a topological ordering (excluding cycles)
      */
-    getCellsDependingOn(ranges) {
+    getCellsDependingOn(ranges, ignore) {
         const visited = this.createEmptyPositionSet();
         const queue = Array.from(ranges).reverse();
         while (queue.length > 0) {
@@ -61152,7 +61172,7 @@ class FormulaDependencyGraph {
             const impactedPositions = this.rTree.search(range).map((dep) => dep.data);
             const nextInQueue = {};
             for (const position of impactedPositions) {
-                if (!visited.has(position)) {
+                if (!visited.has(position) && !ignore.has(position)) {
                     if (!nextInQueue[position.sheetId]) {
                         nextInQueue[position.sheetId] = [];
                     }
@@ -61709,7 +61729,7 @@ class Evaluator {
     }
     invalidatePositionsDependingOnSpread(sheetId, resultZone) {
         // the result matrix is split in 2 zones to exclude the array formula position
-        const invalidatedPositions = this.formulaDependencies().getCellsDependingOn(excludeTopLeft(resultZone).map((zone) => ({ sheetId, zone })));
+        const invalidatedPositions = this.formulaDependencies().getCellsDependingOn(excludeTopLeft(resultZone).map((zone) => ({ sheetId, zone })), this.nextPositionsToUpdate);
         invalidatedPositions.delete({ sheetId, col: resultZone.left, row: resultZone.top });
         this.nextPositionsToUpdate.addMany(invalidatedPositions);
     }
@@ -61827,7 +61847,7 @@ class Evaluator {
         for (const sheetId in zonesBySheetIds) {
             ranges.push(...zonesBySheetIds[sheetId].map((zone) => ({ sheetId, zone })));
         }
-        return this.formulaDependencies().getCellsDependingOn(ranges);
+        return this.formulaDependencies().getCellsDependingOn(ranges, this.nextPositionsToUpdate);
     }
 }
 function forEachSpreadPositionInMatrix(nbColumns, nbRows, callback) {
@@ -77306,6 +77326,6 @@ const chartHelpers = { ...CHART_HELPERS, ...CHART_RUNTIME_HELPERS };
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, CommandResult, CorePlugin, CoreViewPlugin, DispatchResult, EvaluationError, Model, PivotRuntimeDefinition, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, SpreadsheetPivotTable, UIPlugin, __info__, addFunction, addRenderingLayer, astToFormula, chartHelpers, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, stores, tokenColors, tokenize };
 
 
-__info__.version = "18.2.31";
-__info__.date = "2025-09-23T12:31:15.573Z";
-__info__.hash = "839667e";
+__info__.version = "18.2.33";
+__info__.date = "2025-10-16T06:39:40.421Z";
+__info__.hash = "280596c";