@odoo/o-spreadsheet 17.0.27 → 17.0.29

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 17.0.27
- * @date 2024-07-02T09:01:48.383Z
- * @hash c1c2d01
+ * @version 17.0.29
+ * @date 2024-07-08T05:40:27.608Z
+ * @hash 2d467e3
  */
 
 import { Component, useRef, onMounted, useEffect, onWillPatch, useState, onWillUpdateProps, onPatched, useComponent, useExternalListener, onWillUnmount, onWillStart, xml, useChildSubEnv, useSubEnv, markRaw } from '@odoo/owl';
@@ -657,7 +657,7 @@ function getAddHeaderStartIndex(position, base) {
 /**
  * Compares two objects.
  */
-function deepEquals(o1, o2, ignoreFunctions) {
+function deepEquals(o1, o2) {
     if (o1 === o2)
         return true;
     if ((o1 && !o2) || (o2 && !o1))
@@ -673,17 +673,13 @@ function deepEquals(o1, o2, ignoreFunctions) {
         }
     }
     for (const key in o1) {
-        const typeOfO1Key = typeof o1[key];
-        if (typeOfO1Key !== typeof o2[key])
+        if (typeof o1[key] !== typeof o2[key])
             return false;
-        if (typeOfO1Key === "object") {
-            if (!deepEquals(o1[key], o2[key], ignoreFunctions))
+        if (typeof o1[key] === "object") {
+            if (!deepEquals(o1[key], o2[key]))
                 return false;
         }
         else {
-            if (ignoreFunctions && typeOfO1Key === "function") {
-                continue;
-            }
             if (o1[key] !== o2[key])
                 return false;
         }
@@ -754,11 +750,9 @@ class JetSet extends Set {
         return this;
     }
     deleteMany(iterable) {
-        let wasDeleted = false;
         for (const element of iterable) {
-            wasDeleted ||= super.delete(element);
+            super.delete(element);
         }
-        return wasDeleted;
     }
 }
 /**
@@ -1815,7 +1809,7 @@ const getFormulaNumberRegex = memoize(function getFormulaNumberRegex(decimalSepa
 });
 const getNumberRegex = memoize(function getNumberRegex(locale) {
     const decimalSeparator = escapeRegExp(locale.decimalSeparator);
-    const thousandsSeparator = escapeRegExp(locale.thousandsSeparator);
+    const thousandsSeparator = escapeRegExp(locale.thousandsSeparator || "");
     const pIntegerAndDecimals = `(\\d+(${thousandsSeparator}\\d{3,})*(${decimalSeparator}\\d*)?)`; // pattern that match integer number with or without decimal digits
     const pOnlyDecimals = `(${decimalSeparator}\\d+)`; // pattern that match only expression with decimal digits
     const pScientificFormat = "(e(\\+|-)?\\d+)?"; // pattern that match scientific format between zero and one time (should be placed before pPercentFormat)
@@ -1842,7 +1836,7 @@ function isNumber(value, locale) {
     return getNumberRegex(locale).test(value.trim());
 }
 const getInvaluableSymbolsRegexp = memoize(function getInvaluableSymbolsRegexp(locale) {
-    return new RegExp(`[\$€${escapeRegExp(locale.thousandsSeparator)}]`, "g");
+    return new RegExp(`[\$€${escapeRegExp(locale.thousandsSeparator || "")}]`, "g");
 });
 /**
  * Convert a string into a number. It assumes that the string actually represents
@@ -2544,6 +2538,9 @@ function getPredicate(descr, isQuery, locale) {
     return result;
 }
 function operandToRegExp(operand) {
+    if (operand === "*") {
+        return /.+/;
+    }
     let exp = "";
     let predecessor = "";
     for (let char of operand) {
@@ -2567,9 +2564,9 @@ function operandToRegExp(operand) {
     }
     return new RegExp("^" + exp + "$", "i");
 }
-function evaluatePredicate(value, criterion) {
+function evaluatePredicate(value = "", criterion) {
     const { operator, operand } = criterion;
-    if (value === undefined || operand === undefined || value === null || operand === null) {
+    if (operand === undefined || value === null || operand === null) {
         return false;
     }
     if (typeof operand === "number" && operator === "=") {
@@ -4782,7 +4779,6 @@ class ChartJsComponent extends Component {
     static template = "o-spreadsheet-ChartJsComponent";
     canvas = useRef("graphContainer");
     chart;
-    currentRuntime;
     get background() {
         return this.chartRuntime.background;
     }
@@ -4799,17 +4795,10 @@ class ChartJsComponent extends Component {
     setup() {
         onMounted(() => {
             const runtime = this.chartRuntime;
-            this.currentRuntime = runtime;
             // Note: chartJS modify the runtime in place, so it's important to give it a copy
             this.createChart(deepCopy(runtime.chartJsConfig));
         });
-        useEffect(() => {
-            const runtime = this.chartRuntime;
-            if (!deepEquals(runtime, this.currentRuntime, "ignoreFunctions")) {
-                this.currentRuntime = runtime;
-                this.updateChartJs(deepCopy(runtime));
-            }
-        });
+        useEffect(() => this.updateChartJs(deepCopy(this.chartRuntime)), () => [this.chartRuntime]);
     }
     createChart(chartData) {
         const canvas = this.canvas.el;
@@ -7677,19 +7666,22 @@ class TokenizingChars {
 }
 
 function isValidLocale(locale) {
-    if (!(locale &&
-        typeof locale === "object" &&
-        typeof locale.name === "string" &&
-        typeof locale.code === "string" &&
-        typeof locale.thousandsSeparator === "string" &&
-        typeof locale.decimalSeparator === "string" &&
-        typeof locale.dateFormat === "string" &&
-        typeof locale.timeFormat === "string" &&
-        typeof locale.formulaArgSeparator === "string")) {
+    if (!locale ||
+        typeof locale !== "object" ||
+        !(!locale.thousandsSeparator || typeof locale.thousandsSeparator === "string")) {
         return false;
     }
-    if (!Object.values(locale).every((v) => v)) {
-        return false;
+    for (const property of [
+        "code",
+        "name",
+        "decimalSeparator",
+        "dateFormat",
+        "timeFormat",
+        "formulaArgSeparator",
+    ]) {
+        if (!locale[property] || typeof locale[property] !== "string") {
+            return false;
+        }
     }
     if (locale.formulaArgSeparator === locale.decimalSeparator) {
         return false;
@@ -7787,7 +7779,10 @@ function canonicalizeNumberLiteral(content, locale) {
     if (locale.decimalSeparator === "." || !isNumber(content, locale)) {
         return content;
     }
-    return content.replace(locale.thousandsSeparator, "").replace(locale.decimalSeparator, ".");
+    if (locale.thousandsSeparator) {
+        content = content.replaceAll(locale.thousandsSeparator, "");
+    }
+    return content.replace(locale.decimalSeparator, ".");
 }
 /**
  * Change a content string from the given locale to its canonical form (en_US locale). Also convert date string.
@@ -9465,15 +9460,6 @@ function createLineChartRuntime(chart, getters) {
     }
     const colors = new ChartColors();
     for (let [index, { label, data }] of dataSetsValues.entries()) {
-        if (["linear", "time"].includes(axisType)) {
-            // Replace empty string labels by undefined to make sure chartJS doesn't decide that "" is the same as 0
-            data = data.map((y, index) => ({ x: labels[index] || undefined, y }));
-        }
-        const color = colors.next();
-        let backgroundRGBA = colorToRGBA(color);
-        if (chart.stacked) {
-            backgroundRGBA.a = LINE_FILL_TRANSPARENCY;
-        }
         if (chart.cumulative) {
             let accumulator = 0;
             data = data.map((value) => {
@@ -9484,6 +9470,15 @@ function createLineChartRuntime(chart, getters) {
                 return value;
             });
         }
+        if (["linear", "time"].includes(axisType)) {
+            // Replace empty string labels by undefined to make sure chartJS doesn't decide that "" is the same as 0
+            data = data.map((y, index) => ({ x: labels[index] || undefined, y }));
+        }
+        const color = colors.next();
+        let backgroundRGBA = colorToRGBA(color);
+        if (chart.stacked) {
+            backgroundRGBA.a = LINE_FILL_TRANSPARENCY;
+        }
         const backgroundColor = rgbaToHex(backgroundRGBA);
         const dataset = {
             label,
@@ -24281,7 +24276,13 @@ class SettingsPanel extends Component {
     }
     async loadLocales() {
         this.loadedLocales = (await this.env.loadLocales())
-            .filter(isValidLocale)
+            .filter((locale) => {
+            const isValid = isValidLocale(locale);
+            if (!isValid) {
+                console.warn(`Invalid locale: ${locale["code"]} ${locale}`);
+            }
+            return isValid;
+        })
             .sort((a, b) => a.name.localeCompare(b.name));
     }
     get numberFormatPreview() {
@@ -30064,6 +30065,7 @@ class Grid extends Component {
             return;
         }
         if (clipboardData.types.indexOf(ClipboardMIMEType.PlainText) > -1) {
+            ev.preventDefault();
             const content = clipboardData.getData(ClipboardMIMEType.PlainText);
             const target = this.env.model.getters.getSelectedZones();
             const clipboardString = this.env.model.getters.getClipboardTextContent();
@@ -30326,6 +30328,8 @@ const NON_RETROCOMPATIBLE_FUNCTIONS = [
     "BITOR",
     "BITRSHIFT",
     "BITXOR",
+    "BYCOL",
+    "BYROW",
     "CEILING.MATH",
     "CEILING.PRECISE",
     "CHISQ.DIST",
@@ -30333,6 +30337,8 @@ const NON_RETROCOMPATIBLE_FUNCTIONS = [
     "CHISQ.INV",
     "CHISQ.INV.RT",
     "CHISQ.TEST",
+    "CHOOSECOLS",
+    "CHOOSEROWS",
     "COMBINA",
     "CONCAT",
     "CONFIDENCE.NORM",
@@ -30345,14 +30351,17 @@ const NON_RETROCOMPATIBLE_FUNCTIONS = [
     "CSCH",
     "DAYS",
     "DECIMAL",
+    "DROP",
     "ERF.PRECISE",
     "ERFC.PRECISE",
+    "EXPAND",
     "EXPON.DIST",
     "F.DIST",
     "F.DIST.RT",
     "F.INV",
     "F.INV.RT",
     "F.TEST",
+    "FIELDVALUE",
     "FILTERXML",
     "FLOOR.MATH",
     "FLOOR.PRECISE",
@@ -30367,6 +30376,7 @@ const NON_RETROCOMPATIBLE_FUNCTIONS = [
     "GAMMA.INV",
     "GAMMALN.PRECISE",
     "GAUSS",
+    "HSTACK",
     "HYPGEOM.DIST",
     "IFNA",
     "IFS",
@@ -30379,9 +30389,14 @@ const NON_RETROCOMPATIBLE_FUNCTIONS = [
     "IMSINH",
     "IMTAN",
     "ISFORMULA",
+    "ISOMITTED",
     "ISOWEEKNUM",
+    "LAMBDA",
+    "LET",
     "LOGNORM.DIST",
     "LOGNORM.INV",
+    "MAKEARRAY",
+    "MAP",
     "MAXIFS",
     "MINIFS",
     "MODE.MULT",
@@ -30401,17 +30416,26 @@ const NON_RETROCOMPATIBLE_FUNCTIONS = [
     "PERMUTATIONA",
     "PHI",
     "POISSON.DIST",
+    "PQSOURCE",
+    "PYTHON_STR",
+    "PYTHON_TYPE",
+    "PYTHON_TYPENAME",
     "QUARTILE.EXC",
     "QUARTILE.INC",
     "QUERYSTRING",
+    "RANDARRAY",
     "RANK.AVG",
     "RANK.EQ",
+    "REDUCE",
     "RRI",
+    "SCAN",
     "SEC",
     "SECH",
+    "SEQUENCE",
     "SHEET",
     "SHEETS",
     "SKEW.P",
+    "SORTBY",
     "STDEV.P",
     "STDEV.S",
     "SWITCH",
@@ -30421,13 +30445,24 @@ const NON_RETROCOMPATIBLE_FUNCTIONS = [
     "T.INV",
     "T.INV.2T",
     "T.TEST",
+    "TAKE",
+    "TEXTAFTER",
+    "TEXTBEFORE",
     "TEXTJOIN",
+    "TEXTSPLIT",
+    "TOCOL",
+    "TOROW",
     "UNICHAR",
     "UNICODE",
+    "UNIQUE",
     "VAR.P",
     "VAR.S",
+    "VSTACK",
     "WEBSERVICE",
     "WEIBULL.DIST",
+    "WRAPCOLS",
+    "WRAPROWS",
+    "XLOOKUP",
     "XOR",
     "Z.TEST",
 ];
@@ -34606,6 +34641,21 @@ class BordersPlugin extends CorePlugin {
             return [];
         return Object.keys(sheetBorders).map((index) => parseInt(index, 10));
     }
+    /**
+     * Get all the rows which contains at least a border
+     */
+    getRowsWithBorders(sheetId) {
+        const sheetBorders = this.borders[sheetId]?.filter(isDefined$1);
+        if (!sheetBorders)
+            return [];
+        const rowsWithBorders = new Set();
+        for (const rowBorders of sheetBorders) {
+            for (const rowBorder in rowBorders) {
+                rowsWithBorders.add(parseInt(rowBorder, 10));
+            }
+        }
+        return Array.from(rowsWithBorders);
+    }
     /**
      * Get the range of all the rows in the sheet
      */
@@ -34655,7 +34705,7 @@ class BordersPlugin extends CorePlugin {
                 destructive: false,
             });
         }
-        this.getRowsRange(sheetId)
+        this.getRowsWithBorders(sheetId)
             .filter((row) => row >= start)
             .sort((a, b) => (delta < 0 ? a - b : b - a)) // start by the end when moving up
             .forEach((row) => {
@@ -35119,7 +35169,7 @@ function parseExpression(tokens, parent_priority = 0) {
  * Parse an expression (as a string) into an AST.
  */
 function parse(str) {
-    return parseTokens(tokenize(str));
+    return parseTokens(rangeTokenize(str));
 }
 function parseTokens(tokens) {
     tokens = tokens.filter((x) => x.type !== "SPACE");
@@ -41637,6 +41687,7 @@ class Evaluator {
         }
         finally {
             this.cellsBeingComputed.delete(cellId);
+            this.nextPositionsToUpdate.delete(positionId);
         }
     }
     computeAndSave(position) {
@@ -41678,11 +41729,36 @@ class Evaluator {
         forEachSpreadPositionInMatrix(nbColumns, nbRows, 
         // thanks to the isMatrix check above, we know that formulaReturn is MatrixFunctionReturn
         this.spreadValues(formulaPosition, formulaReturn));
+        this.invalidatePositionsDependingOnSpread(formulaPosition, nbColumns, nbRows);
         return createEvaluatedCell(formulaReturn[0][0].value, {
             format: cellData.format || formulaReturn[0][0]?.format,
             locale: this.getters.getLocale(),
         });
     }
+    invalidatePositionsDependingOnSpread(arrayFormulaPosition, nbColumns, nbRows) {
+        // the result matrix is split in 2 zones to exclude the array formula position
+        const top = arrayFormulaPosition.row;
+        const left = arrayFormulaPosition.col;
+        const bottom = top + nbRows - 1;
+        const leftColumnZone = {
+            top: top + 1,
+            bottom,
+            left,
+            right: left,
+        };
+        const rightPartZone = {
+            top,
+            bottom,
+            left: left + 1,
+            right: left + nbColumns - 1,
+        };
+        const sheetId = arrayFormulaPosition.sheetId;
+        const invalidatedPositions = this.formulaDependencies().getCellsDependingOn([
+            { sheetId, zone: rightPartZone },
+            { sheetId, zone: leftColumnZone },
+        ]);
+        this.nextPositionsToUpdate.addMany(invalidatedPositions);
+    }
     assertSheetHasEnoughSpaceToSpreadFormulaResult({ sheetId, col, row }, matrixResult) {
         const numberOfCols = this.getters.getNumberCols(sheetId);
         const numberOfRows = this.getters.getNumberRows(sheetId);
@@ -41731,9 +41807,6 @@ class Evaluator {
             });
             const positionId = this.encoder.encode(position);
             this.setEvaluatedCell(positionId, evaluatedCell);
-            // check if formula dependencies present in the spread zone
-            // if so, they need to be recomputed
-            this.nextPositionsToUpdate.addMany(this.getCellsDependingOn([positionId]));
         };
     }
     invalidateSpreading(positionId) {
@@ -57468,6 +57541,6 @@ const constants = {
 export { AbstractChart, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, astToFormula, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, tokenize };
 
 
-__info__.version = "17.0.27";
-__info__.date = "2024-07-02T09:01:48.383Z";
-__info__.hash = "c1c2d01";
+__info__.version = "17.0.29";
+__info__.date = "2024-07-08T05:40:27.608Z";
+__info__.hash = "2d467e3";