@odoo/o-spreadsheet 18.1.20 → 18.1.21

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.1.20
- * @date 2025-05-13T17:52:28.174Z
- * @hash 3e43a46
+ * @version 18.1.21
+ * @date 2025-05-20T05:54:45.398Z
+ * @hash 89ed6a9
  */
 
 'use strict';
@@ -1608,18 +1608,53 @@ function lettersToNumber(letters) {
     let result = 0;
     const l = letters.length;
     for (let i = 0; i < l; i++) {
-        const charCode = letters.charCodeAt(i);
-        const colIndex = charCode >= 65 && charCode <= 90 ? charCode - 64 : charCode - 96;
+        const colIndex = charToNumber(letters[i]);
         result = result * 26 + colIndex;
     }
     return result - 1;
 }
+function charToNumber(char) {
+    const charCode = char.charCodeAt(0);
+    return charCode >= 65 && charCode <= 90 ? charCode - 64 : charCode - 96;
+}
 function isCharALetter(char) {
     return (char >= "A" && char <= "Z") || (char >= "a" && char <= "z");
 }
 function isCharADigit(char) {
     return char >= "0" && char <= "9";
 }
+// we limit the max column to 3 letters and max row to 7 digits for performance reasons
+const MAX_COL = lettersToNumber("ZZZ");
+const MAX_ROW = 9999998;
+function consumeSpaces(chars) {
+    while (chars.current === " ") {
+        chars.advanceBy(1);
+    }
+}
+function consumeLetters(chars) {
+    if (chars.current === "$")
+        chars.advanceBy(1);
+    if (!chars.current || !isCharALetter(chars.current)) {
+        return -1;
+    }
+    let colCoordinate = 0;
+    while (chars.current && isCharALetter(chars.current)) {
+        colCoordinate = colCoordinate * 26 + charToNumber(chars.shift());
+    }
+    return colCoordinate;
+}
+function consumeDigits(chars) {
+    if (chars.current === "$")
+        chars.advanceBy(1);
+    if (!chars.current || !isCharADigit(chars.current)) {
+        return -1;
+    }
+    let num = 0;
+    while (chars.current && isCharADigit(chars.current)) {
+        num = num * 10 + Number(chars.shift());
+    }
+    return num;
+}
 /**
  * Convert a "XC" coordinate to cartesian coordinates.
  *
@@ -1630,33 +1665,17 @@ function isCharADigit(char) {
  * Note: it also accepts lowercase coordinates, but not fixed references
  */
 function toCartesian(xc) {
-    xc = xc.trim();
-    let letterPart = "";
-    let numberPart = "";
-    let i = 0;
-    // Process letter part
-    if (xc[i] === "$")
-        i++;
-    while (i < xc.length && isCharALetter(xc[i])) {
-        letterPart += xc[i++];
-    }
-    if (letterPart.length === 0 || letterPart.length > 3) {
-        // limit to max 3 letters for performance reasons
+    const chars = new TokenizingChars(xc);
+    consumeSpaces(chars);
+    const letterPart = consumeLetters(chars);
+    if (letterPart === -1 || !chars.current) {
         throw new Error(`Invalid cell description: ${xc}`);
     }
-    // Process number part
-    if (xc[i] === "$")
-        i++;
-    while (i < xc.length && isCharADigit(xc[i])) {
-        numberPart += xc[i++];
-    }
-    if (i !== xc.length || numberPart.length === 0 || numberPart.length > 7) {
-        // limit to max 7 numbers for performance reasons
-        throw new Error(`Invalid cell description: ${xc}`);
-    }
-    const col = lettersToNumber(letterPart);
-    const row = Number(numberPart) - 1;
-    if (isNaN(row)) {
+    const num = consumeDigits(chars);
+    consumeSpaces(chars);
+    const col = letterPart - 1;
+    const row = num - 1;
+    if (!chars.isOver() || col > MAX_COL || row > MAX_ROW) {
         throw new Error(`Invalid cell description: ${xc}`);
     }
     return { col, row };
@@ -2068,67 +2087,6 @@ class LazyTranslatedString extends String {
     }
 }
 
-/** Reference of a cell (eg. A1, $B$5) */
-const cellReference = new RegExp(/\$?([A-Z]{1,3})\$?([0-9]{1,7})/, "i");
-// Same as above, but matches the exact string (nothing before or after)
-const singleCellReference = new RegExp(/^\$?([A-Z]{1,3})\$?([0-9]{1,7})$/, "i");
-/** Reference of a column header (eg. A, AB, $A) */
-const colHeader = new RegExp(/^\$?([A-Z]{1,3})+$/, "i");
-/** Reference of a row header (eg. 1, $1) */
-const rowHeader = new RegExp(/^\$?([0-9]{1,7})+$/, "i");
-/** Reference of a column (eg. A, $CA, Sheet1!B) */
-const colReference = new RegExp(/^\s*('.+'!|[^']+!)?\$?([A-Z]{1,3})$/, "i");
-/** Reference of a row (eg. 1, 59, Sheet1!9) */
-const rowReference = new RegExp(/^\s*('.+'!|[^']+!)?\$?([0-9]{1,7})$/, "i");
-/** Reference of a normal range or a full row range (eg. A1:B1, 1:$5, $A2:5) */
-const fullRowXc = /(\$?[A-Z]{1,3})?\$?[0-9]{1,7}\s*:\s*(\$?[A-Z]{1,3})?\$?[0-9]{1,7}\s*/i;
-/** Reference of a normal range or a column row range (eg. A1:B1, A:$B, $A1:C) */
-const fullColXc = /\$?[A-Z]{1,3}(\$?[0-9]{1,7})?\s*:\s*\$?[A-Z]{1,3}(\$?[0-9]{1,7})?\s*/i;
-/** Reference of a cell or a range, it can be a bounded range, a full row or a full column */
-const rangeReference = new RegExp(/^\s*('.+'!|[^']+!)?/.source +
-    "(" +
-    [cellReference.source, fullRowXc.source, fullColXc.source].join("|") +
-    ")" +
-    /$/.source, "i");
-/**
- * Return true if the given xc is the reference of a column (e.g. A or AC or Sheet1!A)
- */
-function isColReference(xc) {
-    return colReference.test(xc);
-}
-/**
- * Return true if the given xc is the reference of a column (e.g. 1 or Sheet1!1)
- */
-function isRowReference(xc) {
-    return rowReference.test(xc);
-}
-function isColHeader(str) {
-    return colHeader.test(str);
-}
-function isRowHeader(str) {
-    return rowHeader.test(str);
-}
-/**
- * Return true if the given xc is the reference of a single cell,
- * without any specified sheet (e.g. A1)
- */
-function isSingleCellReference(xc) {
-    return singleCellReference.test(xc);
-}
-function splitReference(ref) {
-    if (!ref.includes("!")) {
-        return { xc: ref };
-    }
-    const parts = ref.split("!");
-    const xc = parts.pop();
-    const sheetName = getUnquotedSheetName(parts.join("!")) || undefined;
-    return { sheetName, xc };
-}
-/** Return a reference SheetName!xc from the given arguments */
-function getFullReference(sheetName, xc) {
-    return sheetName !== undefined ? `${getCanonicalSymbolName(sheetName)}!${xc}` : xc;
-}
-
 /**
  * Convert from a cartesian reference to a Zone
  * The range boundaries will be kept in the same order as the
@@ -2146,63 +2104,55 @@ function getFullReference(sheetName, xc) {
  *
  */
 function toZoneWithoutBoundaryChanges(xc) {
-    if (xc.includes("!")) {
-        xc = xc.split("!").at(-1);
-    }
-    if (xc.includes("$")) {
-        xc = xc.replaceAll("$", "");
-    }
-    let firstRangePart = "";
-    let secondRangePart;
-    if (xc.includes(":")) {
-        [firstRangePart, secondRangePart] = xc.split(":");
-        firstRangePart = firstRangePart.trim();
-        secondRangePart = secondRangePart.trim();
-    }
-    else {
-        firstRangePart = xc.trim();
-    }
+    const chars = new TokenizingChars(xc);
+    consumeSpaces(chars);
+    const sheetSeparatorIndex = xc.indexOf("!");
+    if (sheetSeparatorIndex !== -1) {
+        chars.advanceBy(sheetSeparatorIndex + 1);
+    }
+    const leftLetters = consumeLetters(chars);
+    const leftNumbers = consumeDigits(chars);
     let top, bottom, left, right;
     let fullCol = false;
     let fullRow = false;
     let hasHeader = false;
-    if (isColReference(firstRangePart)) {
-        left = right = lettersToNumber(firstRangePart);
+    if (leftNumbers === -1) {
+        left = right = leftLetters - 1;
         top = bottom = 0;
         fullCol = true;
     }
-    else if (isRowReference(firstRangePart)) {
-        top = bottom = parseInt(firstRangePart, 10) - 1;
+    else if (leftLetters === -1) {
+        top = bottom = leftNumbers - 1;
         left = right = 0;
         fullRow = true;
     }
     else {
-        const c = toCartesian(firstRangePart);
-        left = right = c.col;
-        top = bottom = c.row;
+        left = right = leftLetters - 1;
+        top = bottom = leftNumbers - 1;
         hasHeader = true;
     }
-    if (secondRangePart) {
-        if (isColReference(secondRangePart)) {
-            right = lettersToNumber(secondRangePart);
+    consumeSpaces(chars);
+    if (chars.current === ":") {
+        chars.advanceBy(1);
+        consumeSpaces(chars);
+        const rightLetters = consumeLetters(chars);
+        const rightNumbers = consumeDigits(chars);
+        if (rightNumbers === -1) {
+            right = rightLetters - 1;
             fullCol = true;
         }
-        else if (isRowReference(secondRangePart)) {
-            bottom = parseInt(secondRangePart, 10) - 1;
+        else if (rightLetters === -1) {
+            bottom = rightNumbers - 1;
             fullRow = true;
         }
         else {
-            const c = toCartesian(secondRangePart);
-            right = c.col;
-            bottom = c.row;
+            right = rightLetters - 1;
+            bottom = rightNumbers - 1;
             top = fullCol ? bottom : top;
             left = fullRow ? right : left;
             hasHeader = true;
         }
     }
-    if (fullCol && fullRow) {
-        throw new Error("Wrong zone xc. The zone cannot be at the same time a full column and a full row");
-    }
     const zone = {
         top,
         left,
@@ -2231,7 +2181,16 @@ function toZoneWithoutBoundaryChanges(xc) {
  */
 function toUnboundedZone(xc) {
     const zone = toZoneWithoutBoundaryChanges(xc);
-    return reorderZone(zone);
+    const orderedZone = reorderZone(zone);
+    const bottom = orderedZone.bottom;
+    const right = orderedZone.right;
+    if ((bottom !== undefined && bottom > MAX_ROW) || (right !== undefined && right > MAX_COL)) {
+        throw new Error(`Range string out of bounds: ${xc}`); // limit the size of the zone for performance
+    }
+    if (bottom === undefined && right === undefined) {
+        throw new Error("Wrong zone xc. The zone cannot be at the same time a full column and a full row");
+    }
+    return orderedZone;
 }
 /**
  * Convert from a cartesian reference to a Zone.
@@ -5967,6 +5926,67 @@ function scrollDelay(value) {
     return MIN_DELAY + (MAX_DELAY - MIN_DELAY) * Math.exp(-ACCELERATION * (value - 1));
 }
 
+/** Reference of a cell (eg. A1, $B$5) */
+const cellReference = new RegExp(/\$?([A-Z]{1,3})\$?([0-9]{1,7})/, "i");
+// Same as above, but matches the exact string (nothing before or after)
+const singleCellReference = new RegExp(/^\$?([A-Z]{1,3})\$?([0-9]{1,7})$/, "i");
+/** Reference of a column header (eg. A, AB, $A) */
+const colHeader = new RegExp(/^\$?([A-Z]{1,3})+$/, "i");
+/** Reference of a row header (eg. 1, $1) */
+const rowHeader = new RegExp(/^\$?([0-9]{1,7})+$/, "i");
+/** Reference of a column (eg. A, $CA, Sheet1!B) */
+const colReference = new RegExp(/^\s*('.+'!|[^']+!)?\$?([A-Z]{1,3})$/, "i");
+/** Reference of a row (eg. 1, 59, Sheet1!9) */
+const rowReference = new RegExp(/^\s*('.+'!|[^']+!)?\$?([0-9]{1,7})$/, "i");
+/** Reference of a normal range or a full row range (eg. A1:B1, 1:$5, $A2:5) */
+const fullRowXc = /(\$?[A-Z]{1,3})?\$?[0-9]{1,7}\s*:\s*(\$?[A-Z]{1,3})?\$?[0-9]{1,7}\s*/i;
+/** Reference of a normal range or a column row range (eg. A1:B1, A:$B, $A1:C) */
+const fullColXc = /\$?[A-Z]{1,3}(\$?[0-9]{1,7})?\s*:\s*\$?[A-Z]{1,3}(\$?[0-9]{1,7})?\s*/i;
+/** Reference of a cell or a range, it can be a bounded range, a full row or a full column */
+const rangeReference = new RegExp(/^\s*('.+'!|[^']+!)?/.source +
+    "(" +
+    [cellReference.source, fullRowXc.source, fullColXc.source].join("|") +
+    ")" +
+    /$/.source, "i");
+/**
+ * Return true if the given xc is the reference of a column (e.g. A or AC or Sheet1!A)
+ */
+function isColReference(xc) {
+    return colReference.test(xc);
+}
+/**
+ * Return true if the given xc is the reference of a column (e.g. 1 or Sheet1!1)
+ */
+function isRowReference(xc) {
+    return rowReference.test(xc);
+}
+function isColHeader(str) {
+    return colHeader.test(str);
+}
+function isRowHeader(str) {
+    return rowHeader.test(str);
+}
+/**
+ * Return true if the given xc is the reference of a single cell,
+ * without any specified sheet (e.g. A1)
+ */
+function isSingleCellReference(xc) {
+    return singleCellReference.test(xc);
+}
+function splitReference(ref) {
+    if (!ref.includes("!")) {
+        return { xc: ref };
+    }
+    const parts = ref.split("!");
+    const xc = parts.pop();
+    const sheetName = getUnquotedSheetName(parts.join("!")) || undefined;
+    return { sheetName, xc };
+}
+/** Return a reference SheetName!xc from the given arguments */
+function getFullReference(sheetName, xc) {
+    return sheetName !== undefined ? `${getCanonicalSymbolName(sheetName)}!${xc}` : xc;
+}
+
 class RangeImpl {
     getSheetSize;
     _zone;
@@ -41346,12 +41366,13 @@ class StandaloneComposerStore extends AbstractComposerStore {
         return providersDefinitions;
     }
     getComposerContent() {
+        let content = this._currentContent;
         if (this.editionMode === "inactive") {
             // References in the content might not be linked to the current active sheet
             // We here force the sheet name prefix for all references that are not in
             // the current active sheet
             const defaultRangeSheetId = this.args().defaultRangeSheetId;
-            return rangeTokenize(this.args().content)
+            content = rangeTokenize(this.args().content)
                 .map((token) => {
                 if (token.type === "REFERENCE") {
                     const range = this.getters.getRangeFromSheetXC(defaultRangeSheetId, token.value);
@@ -41361,7 +41382,7 @@ class StandaloneComposerStore extends AbstractComposerStore {
             })
                 .join("");
         }
-        return this._currentContent;
+        return localizeContent(content, this.getters.getLocale());
     }
     stopEdition() {
         this._stopEdition();
@@ -45107,6 +45128,9 @@ class PivotMeasureEditor extends owl.Component {
         }
         return undefined;
     }
+    get isCalculatedMeasureInvalid() {
+        return this.env.model.getters.getMeasureCompiledFormula(this.props.measure).isBadExpression;
+    }
 }
 
 css /* scss */ `
@@ -60479,10 +60503,9 @@ class Evaluator {
         return this.evaluatedCells.keysForSheet(sheetId);
     }
     getArrayFormulaSpreadingOn(position) {
-        const hasArrayFormulaResult = this.getEvaluatedCell(position).type !== CellValueType.empty &&
-            !this.getters.getCell(position)?.isFormula;
-        if (!hasArrayFormulaResult) {
-            return this.spreadingRelations.isArrayFormula(position) ? position : undefined;
+        const isEmpty = this.getEvaluatedCell(position).type === CellValueType.empty;
+        if (isEmpty) {
+            return undefined;
         }
         const arrayFormulas = this.spreadingRelations.searchFormulaPositionsSpreadingOn(position.sheetId, positionToZone(position));
         return Array.from(arrayFormulas).find((position) => !this.blockedArrayFormulas.has(position));
@@ -76326,6 +76349,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "18.1.20";
-__info__.date = "2025-05-13T17:52:28.174Z";
-__info__.hash = "3e43a46";
+__info__.version = "18.1.21";
+__info__.date = "2025-05-20T05:54:45.398Z";
+__info__.hash = "89ed6a9";

package/dist/o-spreadsheet.d.ts

@@ -1275,6 +1275,46 @@ declare class ColorGenerator {
     next(): string;
 }
 
+interface SearchOptions {
+    matchCase: boolean;
+    exactMatch: boolean;
+    searchFormulas: boolean;
+    searchScope: "allSheets" | "activeSheet" | "specificRange";
+    specificRange?: Range;
+}
+
+/**
+ * Deep copy arrays, plain objects and primitive values.
+ * Throws an error for other types such as class instances.
+ * Sparse arrays remain sparse.
+ */
+declare function deepCopy<T>(obj: T): T;
+declare function unquote(string: string, quoteChar?: "'" | '"'): string;
+/** Replace the excel-excluded characters of a sheetName */
+declare function sanitizeSheetName(sheetName: string, replacementChar?: string): string;
+declare function isMarkdownLink(str: string): boolean;
+/**
+ * Build a markdown link from a label and an url
+ */
+declare function markdownLink(label: string, url: string): string;
+declare function parseMarkdownLink(str: string): {
+    url: string;
+    label: string;
+};
+/**
+ * This helper function can be used as a type guard when filtering arrays.
+ * const foo: number[] = [1, 2, undefined, 4].filter(isDefined)
+ */
+declare function isDefined<T>(argument: T | undefined): argument is T;
+/**
+ * Lazy value computed by the provided function.
+ */
+declare function lazy<T>(fn: (() => T) | T): Lazy<T>;
+/**
+ * Compares two objects.
+ */
+declare function deepEquals(o1: any, o2: any): boolean;
+
 /**
  * Convert a (col) number to the corresponding letter.
  *
@@ -1351,46 +1391,6 @@ declare function formatValue(value: CellValue, { format, locale, formatWidth }:
 }): FormattedValue;
 declare function createCurrencyFormat(currency: Partial<Currency>): Format;
 
-interface SearchOptions {
-    matchCase: boolean;
-    exactMatch: boolean;
-    searchFormulas: boolean;
-    searchScope: "allSheets" | "activeSheet" | "specificRange";
-    specificRange?: Range;
-}
-
-/**
- * Deep copy arrays, plain objects and primitive values.
- * Throws an error for other types such as class instances.
- * Sparse arrays remain sparse.
- */
-declare function deepCopy<T>(obj: T): T;
-declare function unquote(string: string, quoteChar?: "'" | '"'): string;
-/** Replace the excel-excluded characters of a sheetName */
-declare function sanitizeSheetName(sheetName: string, replacementChar?: string): string;
-declare function isMarkdownLink(str: string): boolean;
-/**
- * Build a markdown link from a label and an url
- */
-declare function markdownLink(label: string, url: string): string;
-declare function parseMarkdownLink(str: string): {
-    url: string;
-    label: string;
-};
-/**
- * This helper function can be used as a type guard when filtering arrays.
- * const foo: number[] = [1, 2, undefined, 4].filter(isDefined)
- */
-declare function isDefined<T>(argument: T | undefined): argument is T;
-/**
- * Lazy value computed by the provided function.
- */
-declare function lazy<T>(fn: (() => T) | T): Lazy<T>;
-/**
- * Compares two objects.
- */
-declare function deepEquals(o1: any, o2: any): boolean;
-
 /**
  * Return true if the argument is a "number string".
  *
@@ -10096,6 +10096,7 @@ declare class PivotMeasureEditor extends Component<Props$i> {
     toggleMeasureVisibility(): void;
     openShowValuesAs(): void;
     getColoredSymbolToken(token: Token): Color | undefined;
+    get isCalculatedMeasureInvalid(): boolean;
 }
 
 interface Props$h {