@odoo/o-spreadsheet 18.0.18 → 18.0.20

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.18
- * @date 2025-03-07T10:38:36.883Z
- * @hash 06c9bc5
+ * @version 18.0.20
+ * @date 2025-03-19T08:21:32.426Z
+ * @hash 3f48d8b
  */
 
 'use strict';
@@ -421,7 +421,6 @@ function escapeRegExp(str) {
  * Sparse arrays remain sparse.
  */
 function deepCopy(obj) {
-    const result = Array.isArray(obj) ? [] : {};
     switch (typeof obj) {
         case "object": {
             if (obj === null) {
@@ -433,8 +432,18 @@ function deepCopy(obj) {
             else if (!(isPlainObject(obj) || obj instanceof Array)) {
                 throw new Error("Unsupported type: only objects and arrays are supported");
             }
-            for (const key in obj) {
-                result[key] = deepCopy(obj[key]);
+            const result = Array.isArray(obj) ? new Array(obj.length) : {};
+            if (Array.isArray(obj)) {
+                for (let i = 0, len = obj.length; i < len; i++) {
+                    if (i in obj) {
+                        result[i] = deepCopy(obj[i]);
+                    }
+                }
+            }
+            else {
+                for (const key in obj) {
+                    result[key] = deepCopy(obj[key]);
+                }
             }
             return result;
         }
@@ -2536,21 +2545,30 @@ function mergeContiguousZones(zones) {
     return mergedZones;
 }
 
+const globalReverseLookup$1 = new WeakMap();
+const globalIdCounter = new WeakMap();
 /**
  * Get the id of the given item (its key in the given dictionary).
  * If the given item does not exist in the dictionary, it creates one with a new id.
  */
 function getItemId(item, itemsDic) {
-    for (const key in itemsDic) {
-        if (deepEquals(itemsDic[key], item)) {
-            return parseInt(key, 10);
-        }
+    if (!globalReverseLookup$1.has(itemsDic)) {
+        globalReverseLookup$1.set(itemsDic, new Map());
+        globalIdCounter.set(itemsDic, 0);
+    }
+    const reverseLookup = globalReverseLookup$1.get(itemsDic);
+    const canonical = getCanonicalRepresentation(item);
+    if (reverseLookup.has(canonical)) {
+        const id = reverseLookup.get(canonical);
+        itemsDic[id] = item;
+        return id;
     }
     // Generate new Id if the item didn't exist in the dictionary
-    const ids = Object.keys(itemsDic);
-    const maxId = ids.length === 0 ? 0 : largeMax(ids.map((id) => parseInt(id, 10)));
-    itemsDic[maxId + 1] = item;
-    return maxId + 1;
+    const newId = globalIdCounter.get(itemsDic) + 1;
+    reverseLookup.set(canonical, newId);
+    globalIdCounter.set(itemsDic, newId);
+    itemsDic[newId] = item;
+    return newId;
 }
 function groupItemIdsByZones(positionsByItemId) {
     const result = {};
@@ -2562,6 +2580,33 @@ function groupItemIdsByZones(positionsByItemId) {
     }
     return result;
 }
+function getCanonicalRepresentation(item) {
+    if (item === null)
+        return "null";
+    if (item === undefined)
+        return "undefined";
+    if (typeof item !== "object")
+        return String(item);
+    if (Array.isArray(item)) {
+        const len = item.length;
+        let result = "[";
+        for (let i = 0; i < len; i++) {
+            if (i > 0)
+                result += ",";
+            result += getCanonicalRepresentation(item[i]);
+        }
+        return result + "]";
+    }
+    const keys = Object.keys(item).sort();
+    let repr = "{";
+    for (const key of keys) {
+        if (item[key] !== undefined) {
+            repr += `"${key}":${getCanonicalRepresentation(item[key])},`;
+        }
+    }
+    repr += "}";
+    return repr;
+}
 
 // -----------------------------------------------------------------------------
 // Date Type
@@ -6064,11 +6109,13 @@ function getDefaultCellHeight(ctx, cell, colSize) {
     if (!cell || (!cell.isFormula && !cell.content)) {
         return DEFAULT_CELL_HEIGHT;
     }
-    const maxWidth = cell.style?.wrapping === "wrap" ? colSize - 2 * MIN_CELL_TEXT_MARGIN : undefined;
-    const numberOfLines = cell.isFormula
-        ? 1
-        : splitTextToWidth(ctx, cell.content, cell.style, maxWidth).length;
-    const fontSize = computeTextFontSizeInPixels(cell.style);
+    const content = cell.isFormula ? "" : cell.content;
+    return getCellContentHeight(ctx, content, cell.style, colSize);
+}
+function getCellContentHeight(ctx, content, style, colSize) {
+    const maxWidth = style?.wrapping === "wrap" ? colSize - 2 * MIN_CELL_TEXT_MARGIN : undefined;
+    const numberOfLines = splitTextToWidth(ctx, content, style, maxWidth).length;
+    const fontSize = computeTextFontSizeInPixels(style);
     return computeTextLinesHeight(fontSize, numberOfLines) + 2 * PADDING_AUTORESIZE_VERTICAL;
 }
 function getDefaultContextFont(fontSize, bold = false, italic = false) {
@@ -8231,13 +8278,6 @@ class CellClipboardHandler extends AbstractCellClipboardHandler {
         this.clearClippedZones(content);
         const selection = target[0];
         this.pasteZone(sheetId, selection.left, selection.top, content.cells, options);
-        this.dispatch("MOVE_RANGES", {
-            target: content.zones,
-            sheetId: content.sheetId,
-            targetSheetId: sheetId,
-            col: selection.left,
-            row: selection.top,
-        });
     }
     /**
      * Clear the clipped zones: remove the cells and clear the formatting
@@ -8746,14 +8786,15 @@ class MergeClipboardHandler extends AbstractCellClipboardHandler {
             }
             merges.push(mergesInRow);
         }
-        return { merges };
+        return { merges, sheetId };
     }
     /**
      * Paste the clipboard content in the given target
      */
     paste(target, content, options) {
         if (options.isCutOperation) {
-            return;
+            const copiedMerges = content.merges.flat().filter(isDefined);
+            this.dispatch("REMOVE_MERGE", { sheetId: content.sheetId, target: copiedMerges });
         }
         this.pasteFromCopy(target.sheetId, target.zones, content.merges, options);
     }
@@ -8788,6 +8829,27 @@ class MergeClipboardHandler extends AbstractCellClipboardHandler {
     }
 }
 
+class ReferenceClipboardHandler extends AbstractCellClipboardHandler {
+    copy(data) {
+        return {
+            zones: data.clippedZones,
+            sheetId: data.sheetId,
+        };
+    }
+    paste(target, content, options) {
+        if (options.isCutOperation) {
+            const selection = target.zones[0];
+            this.dispatch("MOVE_RANGES", {
+                target: content.zones,
+                sheetId: content.sheetId,
+                targetSheetId: target.sheetId,
+                col: selection.left,
+                row: selection.top,
+            });
+        }
+    }
+}
+
 class SheetClipboardHandler extends AbstractCellClipboardHandler {
     isPasteAllowed(sheetId, target, content, options) {
         if (!("cells" in content)) {
@@ -8955,7 +9017,8 @@ clipboardHandlersRegistries.cellHandlers
     .add("merge", MergeClipboardHandler)
     .add("border", BorderClipboardHandler)
     .add("table", TableClipboardHandler)
-    .add("conditionalFormat", ConditionalFormatClipboardHandler);
+    .add("conditionalFormat", ConditionalFormatClipboardHandler)
+    .add("references", ReferenceClipboardHandler);
 
 function transformZone(zone, executed) {
     if (executed.type === "REMOVE_COLUMNS_ROWS") {
@@ -11564,16 +11627,25 @@ function addRelsToFile(relsFiles, path, rel) {
     }
     return id;
 }
+const globalReverseLookup = new WeakMap();
 function pushElement(property, propertyList) {
-    let len = propertyList.length;
-    const operator = typeof property === "object" ? deepEquals : (a, b) => a === b;
-    for (let i = 0; i < len; i++) {
-        if (operator(property, propertyList[i])) {
-            return i;
+    let reverseLookup = globalReverseLookup.get(propertyList);
+    if (!reverseLookup) {
+        reverseLookup = new Map();
+        for (let i = 0; i < propertyList.length; i++) {
+            const canonical = getCanonicalRepresentation(propertyList[i]);
+            reverseLookup.set(canonical, i);
         }
+        globalReverseLookup.set(propertyList, reverseLookup);
+    }
+    const canonical = getCanonicalRepresentation(property);
+    if (reverseLookup.has(canonical)) {
+        return reverseLookup.get(canonical);
     }
-    propertyList[propertyList.length] = property;
-    return propertyList.length - 1;
+    const maxId = propertyList.length;
+    propertyList.push(property);
+    reverseLookup.set(canonical, maxId);
+    return maxId;
 }
 const chartIds = [];
 /**
@@ -13426,7 +13498,7 @@ class XlsxChartExtractor extends XlsxBaseExtractor {
                 title: { text: chartTitle },
                 type: CHART_TYPE_CONVERSION_MAP[chartType],
                 dataSets: this.extractChartDatasets(this.querySelectorAll(rootChartElement, `c:${chartType}`), chartType),
-                labelRange: this.extractChildTextContent(rootChartElement, `c:ser ${chartType === "scatterChart" ? "c:numRef" : "c:cat"} c:f`),
+                labelRange: this.extractLabelRange(chartType, rootChartElement),
                 backgroundColor: this.extractChildAttr(rootChartElement, "c:chartSpace > c:spPr a:srgbClr", "val", {
                     default: "ffffff",
                 }).asString(),
@@ -13438,6 +13510,13 @@ class XlsxChartExtractor extends XlsxBaseExtractor {
             };
         })[0];
     }
+    extractLabelRange(chartType, rootChartElement) {
+        if (chartType === "scatterChart") {
+            return (this.extractChildTextContent(rootChartElement, `c:ser c:strRef c:f`) ||
+                this.extractChildTextContent(rootChartElement, `c:ser c:numRef c:f`));
+        }
+        return this.extractChildTextContent(rootChartElement, `c:ser c:cat c:f`);
+    }
     extractComboChart(chartElement) {
         // Title can be separated into multiple xml elements (for styling and such), we only import the text
         const chartTitle = this.mapOnElements({ parent: chartElement, query: "c:title a:t" }, (textElement) => {
@@ -16899,6 +16978,7 @@ class ScorecardChart extends owl.Component {
 const autoCompleteProviders = new Registry();
 
 autoCompleteProviders.add("dataValidation", {
+    displayAllOnInitialContent: true,
     getProposals(tokenAtCursor, content) {
         if (content.startsWith("=")) {
             return [];
@@ -28375,7 +28455,7 @@ autoCompleteProviders.add("pivot_group_values", {
                 text,
                 description: usedLabel,
                 htmlContent: [{ value: text, color }],
-                fuzzySearchKey: value + usedLabel,
+                fuzzySearchKey: text + usedLabel,
             };
         });
     },
@@ -38778,8 +38858,8 @@ class AbstractComposerStore extends SpreadsheetStore {
         this.updateRangeColor();
     }
     cancelEdition() {
-        this.cancelEditionAndActivateSheet();
         this.resetContent();
+        this.cancelEditionAndActivateSheet();
     }
     setCurrentContent(content, selection) {
         if (selection && !this.isSelectionValid(content.length, selection.start, selection.end)) {
@@ -38795,8 +38875,8 @@ class AbstractComposerStore extends SpreadsheetStore {
         switch (cmd.type) {
             case "SELECT_FIGURE":
                 if (cmd.id) {
-                    this.cancelEditionAndActivateSheet();
                     this.resetContent();
+                    this.cancelEditionAndActivateSheet();
                 }
                 break;
             case "START_CHANGE_HIGHLIGHT":
@@ -39147,6 +39227,15 @@ class AbstractComposerStore extends SpreadsheetStore {
             const exactMatch = proposals?.find((p) => p.text === tokenAtCursor.value);
             // remove tokens that are likely to be other parts of the formula that slipped in the token if it's a string
             const searchTerm = tokenAtCursor.value.replace(/[ ,\(\)]/g, "");
+            if (this._currentContent === this.initialContent &&
+                provider.displayAllOnInitialContent &&
+                proposals?.length) {
+                return {
+                    proposals,
+                    selectProposal: provider.selectProposal,
+                    autoSelectFirstProposal: provider.autoSelectFirstProposal ?? false,
+                };
+            }
             if (exactMatch && this._currentContent !== this.initialContent) {
                 // this means the user has chosen a proposal
                 return;
@@ -44682,6 +44771,7 @@ class RemoveDuplicatesPanel extends owl.Component {
         columns: {},
     });
     setup() {
+        this.updateColumns();
         owl.onWillUpdateProps(() => this.updateColumns());
     }
     toggleHasHeader() {
@@ -46470,8 +46560,8 @@ class CellComposerStore extends AbstractComposerStore {
                 const sheetIdExists = !!this.getters.tryGetSheet(this.sheetId);
                 if (!sheetIdExists && this.editionMode !== "inactive") {
                     this.sheetId = this.getters.getActiveSheetId();
-                    this.cancelEditionAndActivateSheet();
                     this.resetContent();
+                    this.cancelEditionAndActivateSheet();
                     this.notificationStore.raiseError(CELL_DELETED_MESSAGE);
                 }
                 break;
@@ -62827,12 +62917,7 @@ class SheetUIPlugin extends UIPlugin {
                 }
                 break;
             case "AUTORESIZE_ROWS":
-                this.dispatch("RESIZE_COLUMNS_ROWS", {
-                    elements: cmd.rows,
-                    dimension: "ROW",
-                    size: null,
-                    sheetId: cmd.sheetId,
-                });
+                this.autoResizeRows(cmd.sheetId, cmd.rows);
                 break;
         }
     }
@@ -62997,6 +63082,48 @@ class SheetUIPlugin extends UIPlugin {
         }
         return "Success" /* CommandResult.Success */;
     }
+    autoResizeRows(sheetId, rows) {
+        const rowSizes = [];
+        for (const row of rows) {
+            let evaluatedRowSize = 0;
+            for (const cellId of this.getters.getRowCells(sheetId, row)) {
+                const cell = this.getters.getCellById(cellId);
+                if (!cell) {
+                    continue;
+                }
+                const position = this.getters.getCellPosition(cell.id);
+                const colSize = this.getters.getColSize(sheetId, position.col);
+                if (cell.isFormula) {
+                    const content = this.getters.getEvaluatedCell(position).formattedValue;
+                    const evaluatedSize = getCellContentHeight(this.ctx, content, cell?.style, colSize);
+                    if (evaluatedSize > evaluatedRowSize && evaluatedSize > DEFAULT_CELL_HEIGHT) {
+                        evaluatedRowSize = evaluatedSize;
+                    }
+                }
+                else {
+                    const content = cell.content;
+                    const dynamicRowSize = getCellContentHeight(this.ctx, content, cell?.style, colSize);
+                    // Only keep the size of evaluated cells if it's bigger than the dynamic row size
+                    if (dynamicRowSize >= evaluatedRowSize && dynamicRowSize > DEFAULT_CELL_HEIGHT) {
+                        evaluatedRowSize = 0;
+                    }
+                }
+            }
+            rowSizes.push(evaluatedRowSize || null);
+        }
+        const groupedSizes = new Map(rowSizes.map((size) => [size, []]));
+        for (let i = 0; i < rowSizes.length; i++) {
+            groupedSizes.get(rowSizes[i])?.push(rows[i]);
+        }
+        for (const [size, rows] of groupedSizes) {
+            this.dispatch("RESIZE_COLUMNS_ROWS", {
+                elements: rows,
+                dimension: "ROW",
+                size,
+                sheetId,
+            });
+        }
+    }
 }
 
 class TableComputedStylePlugin extends UIPlugin {
@@ -72016,7 +72143,7 @@ function addStyles(styles) {
         }
         if (alignAttrs.length > 0) {
             attributes.push(["applyAlignment", "1"]); // for Libre Office
-            styleNodes.push(escapeXml /*xml*/ `<xf ${formatAttributes(attributes)}>${escapeXml /*xml*/ `<alignment ${formatAttributes(alignAttrs)} />`}</xf> `);
+            styleNodes.push(escapeXml /*xml*/ `<xf ${formatAttributes(attributes)}><alignment ${formatAttributes(alignAttrs)} /></xf> `);
         }
         else {
             styleNodes.push(escapeXml /*xml*/ `<xf ${formatAttributes(attributes)} />`);
@@ -73480,6 +73607,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "18.0.18";
-__info__.date = "2025-03-07T10:38:36.883Z";
-__info__.hash = "06c9bc5";
+__info__.version = "18.0.20";
+__info__.date = "2025-03-19T08:21:32.426Z";
+__info__.hash = "3f48d8b";

package/dist/o-spreadsheet.d.ts

@@ -5276,6 +5276,7 @@ declare class SheetUIPlugin extends UIPlugin {
      * not outside the sheet.
      */
     private checkZonesAreInSheet;
+    private autoResizeRows;
 }
 
 declare class HeaderPositionsUIPlugin extends UIPlugin {
@@ -8211,11 +8212,12 @@ interface ComposerStoreInterface {
  * We declare the providers in the registry as an object (rather than a class)
  * to allow a type-safe way to declare the provider.
  * We still want to be able to use `this` for the getters and dispatch for simplicity.
- * Binding happens at runtime in the edition plugin.
+ * Binding happens at runtime in the composer store.
  */
 interface AutoCompleteProviderDefinition {
     sequence?: number;
     autoSelectFirstProposal?: boolean;
+    displayAllOnInitialContent?: boolean;
     maxDisplayedProposals?: number;
     getProposals(this: {
         composer: ComposerStoreInterface;