@odoo/o-spreadsheet 18.2.32 → 18.2.34

package/dist/o-spreadsheet.cjs.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.32
- * @date 2025-10-07T09:59:52.165Z
- * @hash a42c448
+ * @version 18.2.34
+ * @date 2025-11-12T14:15:32.431Z
+ * @hash 9ca0c4c
  */
 
 'use strict';
@@ -3916,7 +3916,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;
         });
@@ -8818,7 +8828,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,
@@ -10439,6 +10449,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
@@ -10887,7 +10901,7 @@ let ScorecardChart$1 = class ScorecardChart extends AbstractChart {
         return {
             background: context.background,
             type: "scorecard",
-            keyValue: context.range ? context.range[0].dataRange : undefined,
+            keyValue: context.range?.[0]?.dataRange,
             title: context.title || { text: "" },
             baselineMode: DEFAULT_SCORECARD_BASELINE_MODE,
             baselineColorUp: DEFAULT_SCORECARD_BASELINE_COLOR_UP,
@@ -13644,7 +13658,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)));
     },
 };
 // -----------------------------------------------------------------------------
@@ -13709,7 +13723,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,
 };
@@ -13726,7 +13740,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]);
         }
@@ -14316,7 +14330,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));
     },
 };
 // -----------------------------------------------------------------------------
@@ -18572,28 +18586,38 @@ function getTokenNextReferenceType(xc) {
     return xc;
 }
 /**
- * Returns the given XC with the given reference type. The XC string should not contain a sheet name.
+ * Returns the given XC with the given reference type.
  */
 function setXcToFixedReferenceType(xc, referenceType) {
-    if (xc.includes("!")) {
-        throw new Error("The given XC should not contain a sheet name");
-    }
+    let sheetName;
+    ({ sheetName, xc } = splitReference(xc));
+    sheetName = sheetName ? sheetName + "!" : "";
     xc = xc.replace(/\$/g, "");
-    let indexOfNumber;
+    const splitIndex = xc.indexOf(":");
+    if (splitIndex >= 0) {
+        return `${sheetName}${_setXcToFixedReferenceType(xc.slice(0, splitIndex), referenceType)}:${_setXcToFixedReferenceType(xc.slice(splitIndex + 1), referenceType)}`;
+    }
+    else {
+        return sheetName + _setXcToFixedReferenceType(xc, referenceType);
+    }
+}
+function _setXcToFixedReferenceType(xc, referenceType) {
+    const indexOfNumber = xc.search(/[0-9]/);
+    const hasCol = indexOfNumber !== 0;
+    const hasRow = indexOfNumber >= 0;
     switch (referenceType) {
         case "col":
+            if (!hasCol)
+                return xc;
             return "$" + xc;
         case "row":
-            indexOfNumber = xc.search(/[0-9]/);
+            if (!hasRow)
+                return xc;
             return xc.slice(0, indexOfNumber) + "$" + xc.slice(indexOfNumber);
         case "colrow":
-            indexOfNumber = xc.search(/[0-9]/);
-            if (indexOfNumber === -1 || indexOfNumber === 0) {
-                // no row number (eg. A) or no column (eg. 1)
+            if (!hasRow || !hasCol)
                 return "$" + xc;
-            }
-            xc = xc.slice(0, indexOfNumber) + "$" + xc.slice(indexOfNumber);
-            return "$" + xc;
+            return "$" + xc.slice(0, indexOfNumber) + "$" + xc.slice(indexOfNumber);
         case "none":
             return xc;
     }
@@ -21108,6 +21132,7 @@ class AbstractComposerStore extends SpreadsheetStore {
         this.highlightStore.register(this);
         this.onDispose(() => {
             this.highlightStore.unRegister(this);
+            this._cancelEdition();
         });
     }
     handleEvent(event) {
@@ -23747,6 +23772,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
  */
@@ -24254,66 +24347,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
@@ -24523,9 +24556,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;
         }
@@ -24683,11 +24716,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
 // ---------------------------------------------------------------------------
@@ -25108,7 +25136,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":
@@ -25128,9 +25156,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(),
@@ -25142,7 +25170,7 @@ function convertDecimalRule(id, dv) {
         },
     };
 }
-function convertListrule(id, dv) {
+function convertListRule(id, dv) {
     const formula1 = dv.formula1.toString();
     const isRangeRule = rangeReference.test(formula1);
     return {
@@ -25158,9 +25186,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),
@@ -25187,7 +25215,7 @@ function convertCustomRule(id, dv) {
         isBlocking: dv.errorStyle !== "warning",
         criterion: {
             type: "customFormula",
-            values: [`=${dv.formula1.toString()}`],
+            values: [prefixFormulaWithEqual(dv.formula1.toString())],
         },
     };
 }
@@ -28919,6 +28947,7 @@ function getChartTimeOptions(labels, labelFormat, locale) {
         parser: luxonFormat,
         displayFormats,
         unit: timeUnit ?? false,
+        tooltipFormat: luxonFormat,
     };
 }
 /**
@@ -30097,6 +30126,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";
@@ -31125,7 +31155,7 @@ class GaugeChart extends AbstractChart {
             background: context.background,
             title: context.title || { text: "" },
             type: "gauge",
-            dataRange: context.range ? context.range[0].dataRange : undefined,
+            dataRange: context.range?.[0]?.dataRange,
             sectionRule: {
                 colors: {
                     lowerColor: DEFAULT_GAUGE_LOWER_COLOR,
@@ -44208,6 +44238,7 @@ class DataValidationInput extends owl.Component {
             placeholder: this.placeholder,
             class: "o-sidePanel-composer",
             defaultRangeSheetId: this.env.model.getters.getActiveSheetId(),
+            defaultStatic: true,
         };
     }
     get errorMessage() {
@@ -44552,12 +44583,13 @@ class DataValidationEditor extends owl.Component {
         onCloseSidePanel: { type: Function, optional: true },
     };
     state = owl.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;
         }
@@ -44591,7 +44623,6 @@ class DataValidationEditor extends owl.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())
@@ -44599,8 +44630,8 @@ class DataValidationEditor extends owl.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,
         };
     }
@@ -45130,6 +45161,7 @@ css /* scss */ `
       .o-button {
         height: 19px;
         width: 19px;
+        box-sizing: content-box;
         .o-icon {
           height: 14px;
           width: 14px;
@@ -57834,7 +57866,7 @@ class RangeAdapter {
         let sheetName = "";
         if (prefixSheet) {
             if (rangeImpl.invalidSheetName) {
-                sheetName = rangeImpl.invalidSheetName;
+                sheetName = getCanonicalSymbolName(rangeImpl.invalidSheetName);
             }
             else {
                 sheetName = getCanonicalSymbolName(this.getters.getSheetName(rangeImpl.sheetId));
@@ -61139,7 +61171,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) {
@@ -61154,7 +61186,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] = [];
                     }
@@ -61711,7 +61743,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);
     }
@@ -61829,7 +61861,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) {
@@ -77353,6 +77385,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "18.2.32";
-__info__.date = "2025-10-07T09:59:52.165Z";
-__info__.hash = "a42c448";
+__info__.version = "18.2.34";
+__info__.date = "2025-11-12T14:15:32.431Z";
+__info__.hash = "9ca0c4c";