@odoo/o-spreadsheet 18.0.5 → 18.0.7

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.0.5
- * @date 2024-11-22T14:19:19.721Z
- * @hash 9eb34d9
+ * @version 18.0.7
+ * @date 2024-12-05T10:41:39.380Z
+ * @hash a2652c5
  */
 
 'use strict';
@@ -334,8 +334,8 @@ const LINE_FILL_TRANSPARENCY = 0.4;
 const DEBOUNCE_TIME = 200;
 const MESSAGE_VERSION = 1;
 // Sheets
-const FORBIDDEN_SHEET_CHARS = ["'", "*", "?", "/", "\\", "[", "]"];
-const FORBIDDEN_IN_EXCEL_REGEX = /'|\*|\?|\/|\\|\[|\]/;
+const FORBIDDEN_SHEETNAME_CHARS = ["'", "*", "?", "/", "\\", "[", "]"];
+const FORBIDDEN_SHEETNAME_CHARS_IN_EXCEL_REGEX = /'|\*|\?|\/|\\|\[|\]/;
 // Cells
 const FORMULA_REF_IDENTIFIER = "|";
 // Components
@@ -388,6 +388,7 @@ const DEFAULT_CURRENCY = {
 //------------------------------------------------------------------------------
 // Miscellaneous
 //------------------------------------------------------------------------------
+const sanitizeSheetNameRegex = new RegExp(FORBIDDEN_SHEETNAME_CHARS_IN_EXCEL_REGEX, "g");
 /**
  * Remove quotes from a quoted string
  * ```js
@@ -483,6 +484,10 @@ function getCanonicalSymbolName(symbolName) {
     }
     return symbolName;
 }
+/** Replace the excel-excluded characters of a sheetName */
+function sanitizeSheetName(sheetName, replacementChar = " ") {
+    return sheetName.replace(sanitizeSheetNameRegex, replacementChar);
+}
 function clip(val, min, max) {
     return val < min ? min : val > max ? max : val;
 }
@@ -3542,7 +3547,6 @@ exports.CommandResult = void 0;
     CommandResult["ValueCellIsInvalidFormula"] = "ValueCellIsInvalidFormula";
     CommandResult["InvalidDefinition"] = "InvalidDefinition";
     CommandResult["InvalidColor"] = "InvalidColor";
-    CommandResult["DataBarRangeValuesMismatch"] = "DataBarRangeValuesMismatch";
 })(exports.CommandResult || (exports.CommandResult = {}));
 
 const DEFAULT_LOCALES = [
@@ -9417,34 +9421,42 @@ function interpolateData(config, values, labels, newLabels) {
     const labelRange = labelMax - labelMin;
     const normalizedLabels = labels.map((v) => (v - labelMin) / labelRange);
     const normalizedNewLabels = newLabels.map((v) => (v - labelMin) / labelRange);
-    switch (config.type) {
-        case "polynomial": {
-            const order = config.order ?? 2;
-            if (order === 1) {
-                return predictLinearValues([values], [normalizedLabels], [normalizedNewLabels], true)[0];
-            }
-            const coeffs = polynomialRegression(values, normalizedLabels, order, true).flat();
-            return normalizedNewLabels.map((v) => evaluatePolynomial(coeffs, v, order));
-        }
-        case "exponential": {
-            const positiveLogValues = [];
-            const filteredLabels = [];
-            for (let i = 0; i < values.length; i++) {
-                if (values[i] > 0) {
-                    positiveLogValues.push(Math.log(values[i]));
-                    filteredLabels.push(normalizedLabels[i]);
+    try {
+        switch (config.type) {
+            case "polynomial": {
+                const order = config.order;
+                if (!order) {
+                    return Array.from({ length: newLabels.length }, () => NaN);
+                }
+                if (order === 1) {
+                    return predictLinearValues([values], [normalizedLabels], [normalizedNewLabels], true)[0];
+                }
+                const coeffs = polynomialRegression(values, normalizedLabels, order, true).flat();
+                return normalizedNewLabels.map((v) => evaluatePolynomial(coeffs, v, order));
+            }
+            case "exponential": {
+                const positiveLogValues = [];
+                const filteredLabels = [];
+                for (let i = 0; i < values.length; i++) {
+                    if (values[i] > 0) {
+                        positiveLogValues.push(Math.log(values[i]));
+                        filteredLabels.push(normalizedLabels[i]);
+                    }
                 }
+                if (!filteredLabels.length) {
+                    return Array.from({ length: newLabels.length }, () => NaN);
+                }
+                return expM(predictLinearValues([positiveLogValues], [filteredLabels], [normalizedNewLabels], true))[0];
             }
-            if (!filteredLabels.length) {
-                return [];
+            case "logarithmic": {
+                return predictLinearValues([values], logM([normalizedLabels]), logM([normalizedNewLabels]), true)[0];
             }
-            return expM(predictLinearValues([positiveLogValues], [filteredLabels], [normalizedNewLabels], true))[0];
-        }
-        case "logarithmic": {
-            return predictLinearValues([values], logM([normalizedLabels]), logM([normalizedNewLabels]), true)[0];
+            default:
+                return [];
         }
-        default:
-            return [];
+    }
+    catch (e) {
+        return Array.from({ length: newLabels.length }, () => NaN);
     }
 }
 function formatTickValue(localeFormat) {
@@ -27243,13 +27255,12 @@ migrationStepRegistry
     versionFrom: "7",
     migrate(data) {
         const namesTaken = [];
-        const globalForbiddenInExcel = new RegExp(FORBIDDEN_IN_EXCEL_REGEX, "g");
         for (let sheet of data.sheets || []) {
             if (!sheet.name) {
                 continue;
             }
             const oldName = sheet.name;
-            const escapedName = oldName.replace(globalForbiddenInExcel, "_");
+            const escapedName = sanitizeSheetName(oldName, "_");
             let i = 1;
             let newName = escapedName;
             while (namesTaken.includes(newName)) {
@@ -27922,7 +27933,6 @@ const CfTerms = {
         ["ValueLowerInvalidFormula" /* CommandResult.ValueLowerInvalidFormula */]: _t("Invalid lower inflection point formula"),
         ["EmptyRange" /* CommandResult.EmptyRange */]: _t("A range needs to be defined"),
         ["ValueCellIsInvalidFormula" /* CommandResult.ValueCellIsInvalidFormula */]: _t("At least one of the provided values is an invalid formula"),
-        ["DataBarRangeValuesMismatch" /* CommandResult.DataBarRangeValuesMismatch */]: _t("All the ranges and the range values must have the same size"),
         Unexpected: _t("The rule is invalid for an unknown reason"),
     },
     ColorScale: _t("Color scale"),
@@ -30107,9 +30117,12 @@ function createPyramidChartRuntime(chart, getters) {
     if (datasets && datasets[1]) {
         datasets[1].data = datasets[1].data.map((value) => (value > 0 ? -value : 0));
     }
+    const maxValue = Math.max(...config.data?.datasets.map((dataSet) => Math.max(...dataSet.data.map(Math.abs))));
     const scales = config.options.scales;
     const scalesXCallback = scales.x.ticks.callback;
     scales.x.ticks.callback = (value) => scalesXCallback(Math.abs(value));
+    scales.x.suggestedMin = -maxValue;
+    scales.x.suggestedMax = maxValue;
     const tooltipLabelCallback = config.options.plugins.tooltip.callbacks.label;
     config.options.plugins.tooltip.callbacks.label = (item) => {
         const tooltipItem = { ...item, parsed: { y: item.parsed.y, x: Math.abs(item.parsed.x) } };
@@ -37461,7 +37474,7 @@ class ChartWithAxisDesignPanel extends owl.Component {
             case "polynomial":
                 config = {
                     type: "polynomial",
-                    order: type === "linear" ? 1 : 2,
+                    order: type === "linear" ? 1 : this.getMaxPolynomialDegree(),
                 };
                 break;
             case "exponential":
@@ -38537,20 +38550,8 @@ class AbstractComposerStore extends SpreadsheetStore {
     replaceSelectedRange(zone) {
         const ref = this.getZoneReference(zone);
         const currentToken = this.tokenAtCursor;
-        let replaceStart = this.selectionStart;
-        if (currentToken?.type === "REFERENCE") {
-            replaceStart = currentToken.start;
-        }
-        else if (currentToken?.type === "RIGHT_PAREN") {
-            // match left parenthesis
-            const leftParenthesisIndex = this.currentTokens.findIndex((token) => token.type === "LEFT_PAREN" && token.parenIndex === currentToken.parenIndex);
-            const functionToken = this.currentTokens[leftParenthesisIndex - 1];
-            if (functionToken === undefined) {
-                return;
-            }
-            replaceStart = functionToken.start;
-        }
-        this.replaceText(ref, replaceStart, this.selectionEnd);
+        const start = currentToken?.type === "REFERENCE" ? currentToken.start : this.selectionStart;
+        this.replaceText(ref, start, this.selectionEnd);
     }
     /**
      * Replace the reference of the old zone by the new one.
@@ -38583,17 +38584,6 @@ class AbstractComposerStore extends SpreadsheetStore {
     getZoneReference(zone) {
         const inputSheetId = this.sheetId;
         const sheetId = this.getters.getActiveSheetId();
-        if (zone.top === zone.bottom && zone.left === zone.right) {
-            const position = { sheetId, col: zone.left, row: zone.top };
-            const pivotId = this.getters.getPivotIdFromPosition(position);
-            const pivotCell = this.getters.getPivotCellFromPosition(position);
-            const cell = this.getters.getCell(position);
-            if (pivotId && pivotCell.type !== "EMPTY" && !cell?.isFormula) {
-                const formulaPivotId = this.getters.getPivotFormulaId(pivotId);
-                const formula = createPivotFormula(formulaPivotId, pivotCell);
-                return localizeFormula(formula, this.getters.getLocale()).slice(1); // strip leading =
-            }
-        }
         const range = this.getters.getRangeFromZone(sheetId, zone);
         return this.getters.getSelectionRangeString(range, inputSheetId);
     }
@@ -38664,35 +38654,19 @@ class AbstractComposerStore extends SpreadsheetStore {
             const colorIndex = this.colorIndexByRange[rangeString];
             return colors$1[colorIndex % colors$1.length];
         };
-        const highlights = [];
-        for (const range of this.getReferencedRanges()) {
+        return this.getReferencedRanges().map((range) => {
             const rangeString = this.getters.getRangeString(range, editionSheetId);
             const { numberOfRows, numberOfCols } = zoneToDimension(range.zone);
             const zone = numberOfRows * numberOfCols === 1
                 ? this.getters.expandZone(range.sheetId, range.zone)
                 : range.zone;
-            highlights.push({
+            return {
                 zone,
                 color: rangeColor(rangeString),
                 sheetId: range.sheetId,
                 interactive: true,
-            });
-        }
-        const activeSheetId = this.getters.getActiveSheetId();
-        const selectionZone = this.model.selection.getAnchor().zone;
-        const isSelectionHightlighted = highlights.find((highlight) => highlight.sheetId === activeSheetId && isEqual(highlight.zone, selectionZone));
-        if (this.editionMode === "selecting" && !isSelectionHightlighted) {
-            highlights.push({
-                zone: selectionZone,
-                color: "#445566",
-                sheetId: activeSheetId,
-                dashed: true,
-                interactive: false,
-                noFill: true,
-                thinLine: true,
-            });
-        }
-        return highlights;
+            };
+        });
     }
     /**
      * Return ranges currently referenced in the composer
@@ -51863,8 +51837,6 @@ class ConditionalFormatPlugin extends CorePlugin {
             case "IconSetRule": {
                 return this.checkValidations(rule, this.chainValidations(this.checkInflectionPoints(this.checkNaN), this.checkLowerBiggerThanUpper), this.chainValidations(this.checkInflectionPoints(this.checkFormulaCompilation)));
             }
-            case "DataBarRule":
-                return this.checkDataBarRangeValues(rule, cmd.ranges, cmd.sheetId);
         }
         return "Success" /* CommandResult.Success */;
     }
@@ -51984,18 +51956,6 @@ class ConditionalFormatPlugin extends CorePlugin {
         }
         return "Success" /* CommandResult.Success */;
     }
-    checkDataBarRangeValues(rule, ranges, sheetId) {
-        if (rule.rangeValues) {
-            const { numberOfCols, numberOfRows } = zoneToDimension(this.getters.getRangeFromSheetXC(sheetId, rule.rangeValues).zone);
-            for (const range of ranges) {
-                const dimensions = zoneToDimension(this.getters.getRangeFromRangeData(range).zone);
-                if (numberOfCols !== dimensions.numberOfCols || numberOfRows !== dimensions.numberOfRows) {
-                    return "DataBarRangeValuesMismatch" /* CommandResult.DataBarRangeValuesMismatch */;
-                }
-            }
-        }
-        return "Success" /* CommandResult.Success */;
-    }
     removeConditionalFormatting(id, sheet) {
         const cfIndex = this.cfRules[sheet].findIndex((s) => s.id === id);
         if (cfIndex !== -1) {
@@ -54292,7 +54252,7 @@ class SheetPlugin extends CorePlugin {
         if (orderedSheetIds.find((id) => sheets[id]?.name.toLowerCase() === name && id !== cmd.sheetId)) {
             return "DuplicatedSheetName" /* CommandResult.DuplicatedSheetName */;
         }
-        if (FORBIDDEN_IN_EXCEL_REGEX.test(name)) {
+        if (FORBIDDEN_SHEETNAME_CHARS_IN_EXCEL_REGEX.test(name)) {
             return "ForbiddenCharactersInSheetName" /* CommandResult.ForbiddenCharactersInSheetName */;
         }
         return "Success" /* CommandResult.Success */;
@@ -58376,8 +58336,12 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
         const zoneOfValues = rangeValues.zone;
         for (let row = zone.top; row <= zone.bottom; row++) {
             for (let col = zone.left; col <= zone.right; col++) {
-                const cell = this.getEvaluatedCellInZone(sheetId, zone, col, row, zoneOfValues);
-                if (cell.type !== CellValueType.number || cell.value <= 0) {
+                const targetCol = col - zone.left + zoneOfValues.left;
+                const targetRow = row - zone.top + zoneOfValues.top;
+                const cell = this.getters.getEvaluatedCell({ sheetId, col: targetCol, row: targetRow });
+                if (!isInside(targetCol, targetRow, zoneOfValues) ||
+                    cell.type !== CellValueType.number ||
+                    cell.value <= 0) {
                     // values negatives or 0 are ignored
                     continue;
                 }
@@ -58390,11 +58354,6 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
             }
         }
     }
-    getEvaluatedCellInZone(sheetId, zone, col, row, targetZone) {
-        const targetCol = col - zone.left + targetZone.left;
-        const targetRow = row - zone.top + targetZone.top;
-        return this.getters.getEvaluatedCell({ sheetId, col: targetCol, row: targetRow });
-    }
     /** Compute the color scale for the given range and CF rule, and apply in in the given computedStyle object */
     applyColorScale(sheetId, range, rule, computedStyle) {
         const minValue = this.parsePoint(sheetId, range, rule.minimum, "min");
@@ -59696,11 +59655,13 @@ class PivotUIPlugin extends UIPlugin {
             return EMPTY_PIVOT_CELL;
         }
         if (functionName === "PIVOT") {
-            const includeTotal = args[2] === false ? false : undefined;
-            const includeColumnHeaders = args[3] === false ? false : undefined;
+            const includeTotal = toScalar(args[2]);
+            const shouldIncludeTotal = includeTotal === undefined ? true : toBoolean(includeTotal);
+            const includeColumnHeaders = toScalar(args[3]);
+            const shouldIncludeColumnHeaders = includeColumnHeaders === undefined ? true : toBoolean(includeColumnHeaders);
             const pivotCells = pivot
                 .getTableStructure()
-                .getPivotCells(includeTotal, includeColumnHeaders);
+                .getPivotCells(shouldIncludeTotal, shouldIncludeColumnHeaders);
             const pivotCol = position.col - mainPosition.col;
             const pivotRow = position.row - mainPosition.row;
             return pivotCells[pivotCol][pivotRow];
@@ -61856,7 +61817,7 @@ class InsertPivotPlugin extends UIPlugin {
     getPivotDuplicateSheetName(pivotName) {
         let i = 1;
         const names = this.getters.getSheetIds().map((id) => this.getters.getSheetName(id));
-        const sanitizedName = pivotName.replace(new RegExp(FORBIDDEN_IN_EXCEL_REGEX, "g"), " ");
+        const sanitizedName = sanitizeSheetName(pivotName);
         let name = sanitizedName;
         while (names.includes(name)) {
             name = `${sanitizedName} (${i})`;
@@ -65936,7 +65897,7 @@ function interactiveRenameSheet(env, sheetId, name, errorCallback) {
         env.raiseError(_t("A sheet with the name %s already exists. Please select another name.", name), errorCallback);
     }
     else if (result.reasons.includes("ForbiddenCharactersInSheetName" /* CommandResult.ForbiddenCharactersInSheetName */)) {
-        env.raiseError(_t("Some used characters are not allowed in a sheet name (Forbidden characters are %s).", FORBIDDEN_SHEET_CHARS.join(" ")), errorCallback);
+        env.raiseError(_t("Some used characters are not allowed in a sheet name (Forbidden characters are %s).", FORBIDDEN_SHEETNAME_CHARS.join(" ")), errorCallback);
     }
 }
 
@@ -67166,7 +67127,7 @@ class ActionButton extends owl.Component {
     setup() {
         owl.onWillUpdateProps((nextProps) => {
             if (nextProps.action !== this.props.action) {
-                this.actionButton = createAction(this.props.action);
+                this.actionButton = createAction(nextProps.action);
             }
         });
     }
@@ -69345,9 +69306,6 @@ class SelectionStreamProcessorImpl {
     getBackToDefault() {
         this.stream.getBackToDefault();
     }
-    getAnchor() {
-        return this.anchor;
-    }
     modifyAnchor(anchor, mode, options) {
         const sheetId = this.getters.getActiveSheetId();
         anchor = {
@@ -72435,6 +72393,7 @@ const helpers = {
     areDomainArgsFieldsValid,
     splitReference,
     formatTickValue,
+    sanitizeSheetName,
 };
 const links = {
     isMarkdownLink,
@@ -72570,6 +72529,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "18.0.5";
-__info__.date = "2024-11-22T14:19:19.721Z";
-__info__.hash = "9eb34d9";
+__info__.version = "18.0.7";
+__info__.date = "2024-12-05T10:41:39.380Z";
+__info__.hash = "a2652c5";

package/dist/o-spreadsheet.d.ts

@@ -1306,7 +1306,6 @@ type StatefulStream<Event, State> = {
  * Allows to select cells in the grid and update the selection
  */
 interface SelectionProcessor {
-    getAnchor(): Immutable<AnchorZone>;
     selectZone(anchor: AnchorZone, options?: SelectionEventOptions): DispatchResult;
     selectCell(col: number, row: number): DispatchResult;
     moveAnchorCell(direction: Direction$1, step: SelectionStep): DispatchResult;
@@ -2670,8 +2669,7 @@ declare const enum CommandResult {
     EmptyName = "EmptyName",
     ValueCellIsInvalidFormula = "ValueCellIsInvalidFormula",
     InvalidDefinition = "InvalidDefinition",
-    InvalidColor = "InvalidColor",
-    DataBarRangeValuesMismatch = "DataBarRangeValuesMismatch"
+    InvalidColor = "InvalidColor"
 }
 interface CommandHandler<T> {
     allowDispatch(command: T): CommandResult | CommandResult[];
@@ -3921,7 +3919,6 @@ declare class ConditionalFormatPlugin extends CorePlugin<ConditionalFormatState>
     private checkMidBiggerThanMax;
     private checkMinBiggerThanMid;
     private checkCFValues;
-    private checkDataBarRangeValues;
     private removeConditionalFormatting;
     private changeCFPriority;
 }
@@ -4628,7 +4625,6 @@ declare class EvaluationConditionalFormatPlugin extends UIPlugin {
     private applyIcon;
     private computeIcon;
     private applyDataBar;
-    private getEvaluatedCellInZone;
     /** Compute the color scale for the given range and CF rule, and apply in in the given computedStyle object */
     private applyColorScale;
     private computeColorDiffUnits;
@@ -5709,6 +5705,8 @@ declare function createCurrencyFormat(currency: Partial<Currency>): Format;
  */
 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
@@ -12088,6 +12086,7 @@ declare const helpers: {
     areDomainArgsFieldsValid: typeof areDomainArgsFieldsValid;
     splitReference: typeof splitReference;
     formatTickValue: typeof formatTickValue;
+    sanitizeSheetName: typeof sanitizeSheetName;
 };
 declare const links: {
     isMarkdownLink: typeof isMarkdownLink;