@odoo/o-spreadsheet 17.4.34 → 17.4.36

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.4.34
- * @date 2025-05-12T05:23:37.387Z
- * @hash e3ac48a
+ * @version 17.4.36
+ * @date 2025-05-20T05:54:11.995Z
+ * @hash fabd931
  */
 
 import { useEnv, useSubEnv, onWillUnmount, useComponent, status, Component, useRef, onMounted, useEffect, useState, onPatched, onWillPatch, onWillUpdateProps, useExternalListener, onWillStart, xml, useChildSubEnv, markRaw, toRaw } from '@odoo/owl';
@@ -1914,7 +1914,7 @@ function isDateAfter(date, dateAfter) {
  */
 const getFormulaNumberRegex = memoize(function getFormulaNumberRegex(decimalSeparator) {
     decimalSeparator = escapeRegExp(decimalSeparator);
-    return new RegExp(`(?:^-?\\d+(?:${decimalSeparator}?\\d*(?:e\\d+)?)?|^-?${decimalSeparator}\\d+)(?!\\w|!)`);
+    return new RegExp(`(?:^-?\\d+(?:${decimalSeparator}?\\d*(?:e(\\+|-)?\\d+)?)?|^-?${decimalSeparator}\\d+)(?!\\w|!)`);
 });
 const getNumberRegex = memoize(function getNumberRegex(locale) {
     const decimalSeparator = escapeRegExp(locale.decimalSeparator);
@@ -5209,6 +5209,13 @@ function getDuplicateSheetName(nameToDuplicate, existingNames) {
     }
     return name;
 }
+function isSheetNameEqual(name1, name2) {
+    if (name1 === undefined || name2 === undefined) {
+        return false;
+    }
+    return (getUnquotedSheetName(name1.trim().toUpperCase()) ===
+        getUnquotedSheetName(name2.trim().toUpperCase()));
+}
 
 function computeTextLinesHeight(textLineHeight, numberOfLines = 1) {
     return numberOfLines * (textLineHeight + MIN_CELL_TEXT_MARGIN) - MIN_CELL_TEXT_MARGIN;
@@ -6510,10 +6517,9 @@ const AGGREGATOR_NAMES = {
     avg: _t("Average"),
     sum: _t("Sum"),
 };
-const NUMBER_CHAR_AGGREGATORS = ["max", "min", "avg", "sum", "count_distinct", "count"];
 const AGGREGATORS_BY_FIELD_TYPE = {
-    integer: NUMBER_CHAR_AGGREGATORS,
-    char: NUMBER_CHAR_AGGREGATORS,
+    integer: ["max", "min", "avg", "sum", "count_distinct", "count"],
+    char: ["count_distinct", "count"],
     boolean: ["count_distinct", "count", "bool_and", "bool_or"],
 };
 const AGGREGATORS = {};
@@ -6727,8 +6733,11 @@ class CellClipboardHandler extends AbstractCellClipboardHandler {
                 const evaluatedCell = this.getters.getEvaluatedCell(position);
                 const pivotId = this.getters.getPivotIdFromPosition(position);
                 const spreader = this.getters.getArrayFormulaSpreadingOn(position);
-                if (pivotId) {
-                    if (spreader && (!deepEquals(spreader, position) || !isCopyingOneCell)) {
+                if (pivotId && spreader) {
+                    const pivotZone = this.getters.getSpreadZone(spreader);
+                    if ((!deepEquals(spreader, position) || !isCopyingOneCell) &&
+                        pivotZone &&
+                        !data.zones.some((z) => isZoneInside(pivotZone, z))) {
                         const pivotCell = this.getters.getPivotCellFromPosition(position);
                         const formulaPivotId = this.getters.getPivotFormulaId(pivotId);
                         const pivotFormula = createPivotFormula(formulaPivotId, pivotCell);
@@ -9450,7 +9459,10 @@ function proxifyStoreMutation(store, callback) {
                 const functionProxy = new Proxy(value, {
                     // trap the function call
                     apply(target, thisArg, argArray) {
-                        Reflect.apply(target, thisStore, argArray);
+                        const res = Reflect.apply(target, thisStore, argArray);
+                        if (res === "noStateChange") {
+                            return;
+                        }
                         callback();
                     },
                 });
@@ -9472,7 +9484,7 @@ function getDependencyContainer(env) {
 const ModelStore = createAbstractStore("Model");
 
 class RendererStore {
-    mutators = ["register", "unRegister"];
+    mutators = ["register", "unRegister", "drawLayer"];
     renderers = {};
     register(renderer) {
         if (!renderer.renderingLayers.length) {
@@ -9492,14 +9504,14 @@ class RendererStore {
     }
     drawLayer(context, layer) {
         const renderers = this.renderers[layer];
-        if (!renderers) {
-            return;
-        }
-        for (const renderer of renderers) {
-            context.ctx.save();
-            renderer.drawLayer(context, layer);
-            context.ctx.restore();
+        if (renderers) {
+            for (const renderer of renderers) {
+                context.ctx.save();
+                renderer.drawLayer(context, layer);
+                context.ctx.restore();
+            }
         }
+        return "noStateChange";
     }
 }
 
@@ -10112,7 +10124,7 @@ class ComposerStore extends SpreadsheetStore {
             .find((token) => {
             const { xc, sheetName: sheet } = splitReference(token.value);
             const sheetName = sheet || this.getters.getSheetName(this.sheetId);
-            if (this.getters.getSheetName(activeSheetId) !== sheetName) {
+            if (!isSheetNameEqual(this.getters.getSheetName(activeSheetId), sheetName)) {
                 return false;
             }
             const refRange = this.getters.getRangeFromSheetXC(activeSheetId, xc);
@@ -10417,27 +10429,30 @@ class ComposerFocusStore extends SpreadsheetStore {
     }
     focusTopBarComposer(selection) {
         if (this.getters.isReadonly()) {
-            return;
+            return "noStateChange";
         }
         this.topBarFocus = "contentFocus";
         this.gridFocusMode = "inactive";
-        this.setComposerContent({ selection } || {});
+        this.setComposerContent({ selection });
+        return;
     }
     focusGridComposerContent() {
         if (this.getters.isReadonly()) {
-            return;
+            return "noStateChange";
         }
         this.topBarFocus = "inactive";
         this.gridFocusMode = "contentFocus";
         this.setComposerContent({});
+        return;
     }
     focusGridComposerCell(content, selection) {
         if (this.getters.isReadonly()) {
-            return;
+            return "noStateChange";
         }
         this.topBarFocus = "inactive";
         this.gridFocusMode = "cellFocus";
-        this.setComposerContent({ content, selection } || {});
+        this.setComposerContent({ content, selection });
+        return;
     }
     /**
      * Start the edition or update the content if it's already started.
@@ -20322,6 +20337,9 @@ function isEmpty(data) {
     return data === undefined || data.value === null;
 }
 const getNeutral = { number: 0, string: "", boolean: false };
+function areAlmostEqual(value1, value2, epsilon = 2e-16) {
+    return Math.abs(value1 - value2) < epsilon;
+}
 const EQ = {
     description: _t("Equal."),
     args: [
@@ -20343,6 +20361,9 @@ const EQ = {
         if (isEvaluationError(_value2)) {
             throw value2;
         }
+        if (typeof _value1 === "number" && typeof _value2 === "number") {
+            return areAlmostEqual(_value1, _value2);
+        }
         return _value1 === _value2;
     },
 };
@@ -20382,6 +20403,9 @@ const GT = {
     ],
     compute: function (value1, value2) {
         return applyRelationalOperator(value1, value2, (v1, v2) => {
+            if (typeof v1 === "number" && typeof v2 === "number") {
+                return !areAlmostEqual(v1, v2) && v1 > v2;
+            }
             return v1 > v2;
         });
     },
@@ -20397,6 +20421,9 @@ const GTE = {
     ],
     compute: function (value1, value2) {
         return applyRelationalOperator(value1, value2, (v1, v2) => {
+            if (typeof v1 === "number" && typeof v2 === "number") {
+                return areAlmostEqual(v1, v2) || v1 > v2;
+            }
             return v1 >= v2;
         });
     },
@@ -21497,10 +21524,18 @@ autoCompleteProviders.add("functions", {
 });
 
 class DOMFocusableElementStore {
-    mutators = ["setFocusableElement"];
+    mutators = ["setFocusableElement", "focus"];
     focusableElement = undefined;
     setFocusableElement(element) {
         this.focusableElement = element;
+        return "noStateChange";
+    }
+    focus() {
+        if (this.focusableElement === document.activeElement) {
+            return "noStateChange";
+        }
+        this.focusableElement?.focus();
+        return;
     }
 }
 
@@ -21757,12 +21792,20 @@ class HoveredCellStore extends SpreadsheetStore {
         }
     }
     hover(position) {
+        if (position.col === this.col && position.row === this.row) {
+            return "noStateChange";
+        }
         this.col = position.col;
         this.row = position.row;
+        return;
     }
     clear() {
+        if (this.col === undefined && this.row === undefined) {
+            return "noStateChange";
+        }
         this.col = undefined;
         this.row = undefined;
+        return;
     }
 }
 
@@ -21784,7 +21827,11 @@ class CellPopoverStore extends SpreadsheetStore {
         this.persistentPopover = { col, row, sheetId, type };
     }
     close() {
+        if (!this.persistentPopover) {
+            return "noStateChange";
+        }
         this.persistentPopover = undefined;
+        return;
     }
     get persistentCellPopover() {
         return ((this.persistentPopover && { isOpen: true, ...this.persistentPopover }) || { isOpen: false });
@@ -37026,7 +37073,9 @@ function compareDimensionValues(dimension, a, b) {
         return dimension.order === "asc" ? -1 : 1;
     }
     if (dimension.type === "integer" || dimension.type === "date") {
-        return dimension.order === "asc" ? Number(a) - Number(b) : Number(b) - Number(a);
+        return dimension.order === "asc"
+            ? toNumber(a, DEFAULT_LOCALE) - toNumber(b, DEFAULT_LOCALE)
+            : toNumber(b, DEFAULT_LOCALE) - toNumber(a, DEFAULT_LOCALE);
     }
     return dimension.order === "asc" ? a.localeCompare(b) : b.localeCompare(a);
 }
@@ -37460,7 +37509,12 @@ class SpreadsheetPivot {
                     entry[field.name] = { value: null, type: CellValueType.empty };
                 }
                 else {
-                    entry[field.name] = cell;
+                    if (field.type === "char") {
+                        entry[field.name] = { ...cell, value: cell.formattedValue || null };
+                    }
+                    else {
+                        entry[field.name] = cell;
+                    }
                 }
             }
             entry["__count"] = { value: 1, type: CellValueType.number };
@@ -39655,7 +39709,7 @@ class GridComposer extends Component {
             this.isEditing = isEditing;
             if (!isEditing) {
                 this.rect = this.defaultRect;
-                this.DOMFocusableElementStore.focusableElement?.focus();
+                this.DOMFocusableElementStore.focus();
                 return;
             }
             const position = this.env.model.getters.getActivePosition();
@@ -42091,10 +42145,6 @@ function useGridDrawing(refName, model, canvasSize) {
         ctx.scale(dpr, dpr);
         for (const layer of OrderedLayers()) {
             model.drawLayer(renderingContext, layer);
-            // @ts-ignore 'drawLayer' is not declated as a mutator because:
-            // it does not mutate anything. Most importantly it's used
-            // during rendering. Invoking a mutator during rendering would
-            // trigger another rendering, ultimately resulting in an infinite loop.
             rendererStore.drawLayer(renderingContext, layer);
         }
     }
@@ -42717,7 +42767,7 @@ class Grid extends Component {
         this.cellPopovers = useStore(CellPopoverStore);
         useEffect(() => {
             if (!this.sidePanel.isOpen) {
-                this.DOMFocusableElementStore.focusableElement?.focus();
+                this.DOMFocusableElementStore.focus();
             }
         }, () => [this.sidePanel.isOpen]);
     }
@@ -42921,7 +42971,7 @@ class Grid extends Component {
     focusDefaultElement() {
         if (!this.env.model.getters.getSelectedFigureId() &&
             this.composerStore.editionMode === "inactive") {
-            this.DOMFocusableElementStore.focusableElement?.focus();
+            this.DOMFocusableElementStore.focus();
         }
     }
     get gridEl() {
@@ -44658,7 +44708,7 @@ function getRangeSize(reference, defaultSheetIndex, data) {
     ({ xc, sheetName } = splitReference(reference));
     let rangeSheetIndex;
     if (sheetName) {
-        const index = data.sheets.findIndex((sheet) => sheet.name === sheetName);
+        const index = data.sheets.findIndex((sheet) => isSheetNameEqual(sheet.name, sheetName));
         if (index < 0) {
             throw new Error("Unable to find a sheet with the name " + sheetName);
         }
@@ -44879,7 +44929,7 @@ function convertFormula(formula, data) {
     formula = formula.replace(externalReferenceRegex, (match, externalRefId, sheetName, cellRef) => {
         externalRefId = Number(externalRefId) - 1;
         cellRef = cellRef.replace(/\$/g, "");
-        const sheetIndex = data.externalBooks[externalRefId].sheetNames.findIndex((name) => name === sheetName);
+        const sheetIndex = data.externalBooks[externalRefId].sheetNames.findIndex((name) => isSheetNameEqual(name, sheetName));
         if (sheetIndex === -1) {
             return match;
         }
@@ -45208,7 +45258,7 @@ function convertPivotTableConfig(pivotTable) {
  */
 function convertTableFormulaReferences(convertedSheets, xlsxSheets) {
     for (let tableSheet of convertedSheets) {
-        const tables = xlsxSheets.find((s) => s.sheetName === tableSheet.name).tables;
+        const tables = xlsxSheets.find((s) => isSheetNameEqual(s.sheetName, tableSheet.name)).tables;
         for (let table of tables) {
             const tabRef = table.name + "[";
             for (let sheet of convertedSheets) {
@@ -50885,7 +50935,7 @@ class RangeAdapter {
                     if (range.sheetId === cmd.sheetId) {
                         return { changeType: "CHANGE", range };
                     }
-                    if (cmd.name && range.invalidSheetName === cmd.name) {
+                    if (isSheetNameEqual(range.invalidSheetName, cmd.name)) {
                         const invalidSheetName = undefined;
                         const sheetId = cmd.sheetId;
                         const newRange = range.clone({ sheetId, invalidSheetName });
@@ -51456,7 +51506,7 @@ class SheetPlugin extends CorePlugin {
         if (name) {
             const unquotedName = getUnquotedSheetName(name);
             for (const key in this.sheetIdsMapName) {
-                if (key.toUpperCase() === unquotedName.toUpperCase()) {
+                if (isSheetNameEqual(key, unquotedName)) {
                     return this.sheetIdsMapName[key];
                 }
             }
@@ -51704,7 +51754,7 @@ class SheetPlugin extends CorePlugin {
         }
         const { orderedSheetIds, sheets } = this;
         const name = cmd.name && cmd.name.trim().toLowerCase();
-        if (orderedSheetIds.find((id) => sheets[id]?.name.toLowerCase() === name && id !== cmd.sheetId)) {
+        if (orderedSheetIds.find((id) => isSheetNameEqual(sheets[id]?.name, name) && id !== cmd.sheetId)) {
             return "DuplicatedSheetName" /* CommandResult.DuplicatedSheetName */;
         }
         if (FORBIDDEN_SHEETNAME_CHARS_IN_EXCEL_REGEX.test(name)) {
@@ -63036,11 +63086,11 @@ class BottomBarSheet extends Component {
         if (ev.key === "Enter") {
             ev.preventDefault();
             this.stopEdition();
-            this.DOMFocusableElementStore.focusableElement?.focus();
+            this.DOMFocusableElementStore.focus();
         }
         if (ev.key === "Escape") {
             this.cancelEdition();
-            this.DOMFocusableElementStore.focusableElement?.focus();
+            this.DOMFocusableElementStore.focus();
         }
     }
     onMouseEventSheetName(ev) {
@@ -69470,6 +69520,6 @@ const constants = {
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, PivotRuntimeDefinition, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, SpreadsheetPivotTable, UIPlugin, __info__, addFunction, addRenderingLayer, astToFormula, 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 = "17.4.34";
-__info__.date = "2025-05-12T05:23:37.387Z";
-__info__.hash = "e3ac48a";
+__info__.version = "17.4.36";
+__info__.date = "2025-05-20T05:54:11.995Z";
+__info__.hash = "fabd931";