@odoo/o-spreadsheet 17.0.59 → 17.0.61

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 17.0.59
- * @date 2025-04-25T08:04:05.375Z
- * @hash eb92388
+ * @version 17.0.61
+ * @date 2025-05-12T05:21:02.159Z
+ * @hash 0368641
  */
 
 'use strict';
@@ -4486,6 +4486,25 @@ function moveHeaderIndexesOnHeaderDeletion(deletedHeaders, headers) {
     })
         .filter(isDefined$1);
 }
+function getNextSheetName(existingNames, baseName = "Sheet") {
+    let i = 1;
+    let name = `${baseName}${i}`;
+    while (existingNames.includes(name)) {
+        name = `${baseName}${i}`;
+        i++;
+    }
+    return name;
+}
+function getDuplicateSheetName(nameToDuplicate, existingNames) {
+    let i = 1;
+    const baseName = _t("Copy of %s", nameToDuplicate);
+    let name = baseName.toString();
+    while (existingNames.includes(name)) {
+        name = `${baseName} (${i})`;
+        i++;
+    }
+    return name;
+}
 
 function computeTextLinesHeight(textLineHeight, numberOfLines = 1) {
     return numberOfLines * (textLineHeight + MIN_CELL_TEXT_MARGIN) - MIN_CELL_TEXT_MARGIN;
@@ -8028,10 +8047,13 @@ const duplicateSheet = {
     name: _t("Duplicate"),
     execute: (env) => {
         const sheetIdFrom = env.model.getters.getActiveSheetId();
+        const sheetNameFrom = env.model.getters.getSheetName(sheetIdFrom);
         const sheetIdTo = env.model.uuidGenerator.smallUuid();
+        const sheetNameTo = env.model.getters.getDuplicateSheetName(sheetNameFrom);
         env.model.dispatch("DUPLICATE_SHEET", {
             sheetId: sheetIdFrom,
             sheetIdTo,
+            sheetNameTo,
         });
         env.model.dispatch("ACTIVATE_SHEET", { sheetIdFrom, sheetIdTo });
     },
@@ -8791,9 +8813,6 @@ class BarChart extends AbstractChart {
         };
     }
     getDefinitionForExcel() {
-        // Excel does not support aggregating labels
-        if (this.aggregated)
-            return undefined;
         const dataSets = this.dataSets
             .map((ds) => toExcelDataset(this.getters, ds))
             .filter((ds) => ds.range !== "" && ds.range !== INCORRECT_RANGE_STRING);
@@ -9370,9 +9389,6 @@ class LineChart extends AbstractChart {
         return new LineChart(definition, this.sheetId, this.getters);
     }
     getDefinitionForExcel() {
-        // Excel does not support aggregating labels
-        if (this.aggregated)
-            return undefined;
         const dataSets = this.dataSets
             .map((ds) => toExcelDataset(this.getters, ds))
             .filter((ds) => ds.range !== "" && ds.range !== INCORRECT_RANGE_STRING);
@@ -9678,9 +9694,6 @@ class PieChart extends AbstractChart {
         return new PieChart(definition, sheetId, this.getters);
     }
     getDefinitionForExcel() {
-        // Excel does not support aggregating labels
-        if (this.aggregated)
-            return undefined;
         const dataSets = this.dataSets
             .map((ds) => toExcelDataset(this.getters, ds))
             .filter((ds) => ds.range !== "" && ds.range !== INCORRECT_RANGE_STRING);
@@ -27158,6 +27171,13 @@ class Composer extends owl.Component {
             this.props.onInputContextMenu?.(ev);
         }
     }
+    onWheel(event) {
+        // detect if scrollbar is available
+        if (this.composerRef.el &&
+            this.composerRef.el.scrollHeight > this.composerRef.el.clientHeight) {
+            event.stopPropagation();
+        }
+    }
     // ---------------------------------------------------------------------------
     // Private
     // ---------------------------------------------------------------------------
@@ -27466,12 +27486,15 @@ class GridComposer extends owl.Component {
         });
     }
     get composerProps() {
+        // Remove the wrapper border width
+        const maxHeight = this.props.gridDims.height - this.rect.y - 2 * COMPOSER_BORDER_WIDTH;
         return {
             focus: this.props.focus,
             isDefaultFocus: true,
             onComposerContentFocused: this.props.onComposerContentFocused,
             onComposerCellFocused: this.props.onComposerCellFocused,
             onInputContextMenu: this.props.onInputContextMenu,
+            inputStyle: `max-height: ${maxHeight}px;`,
         };
     }
     get containerStyle() {
@@ -28484,18 +28507,28 @@ function useRefListener(ref, ...listener) {
 function useInterval(callback, delay) {
     let intervalId;
     const { setInterval, clearInterval } = window;
+    const pause = () => {
+        clearInterval(intervalId);
+        intervalId = undefined;
+    };
+    const safeCallback = () => {
+        try {
+            callback();
+        }
+        catch (e) {
+            pause();
+            throw e;
+        }
+    };
     owl.useEffect(() => {
-        intervalId = setInterval(callback, delay);
+        intervalId = setInterval(safeCallback, delay);
         return () => clearInterval(intervalId);
     }, () => [delay]);
     return {
-        pause: () => {
-            clearInterval(intervalId);
-            intervalId = undefined;
-        },
+        pause,
         resume: () => {
             if (intervalId === undefined) {
-                intervalId = setInterval(callback, delay);
+                intervalId = setInterval(safeCallback, delay);
             }
         },
     };
@@ -28940,7 +28973,7 @@ css /* scss */ `
     position: absolute;
     top: 0;
     left: ${HEADER_WIDTH}px;
-    right: 0;
+    right: ${SCROLLBAR_WIDTH}px;
     height: ${HEADER_HEIGHT}px;
     &.o-dragging {
       cursor: grabbing;
@@ -29115,9 +29148,8 @@ css /* scss */ `
     position: absolute;
     top: ${HEADER_HEIGHT}px;
     left: 0;
-    right: 0;
+    bottom: ${SCROLLBAR_WIDTH}px;
     width: ${HEADER_WIDTH}px;
-    height: calc(100% - ${HEADER_HEIGHT + SCROLLBAR_WIDTH}px);
     &.o-dragging {
       cursor: grabbing;
     }
@@ -34355,6 +34387,7 @@ function repairInitialMessages(data, initialMessages) {
     initialMessages = dropCommands(initialMessages, "SORT_CELLS");
     initialMessages = dropCommands(initialMessages, "SET_DECIMAL");
     initialMessages = fixChartDefinitions(data, initialMessages);
+    initialMessages = fixTranslatedDuplicateSheetName(data, initialMessages);
     return initialMessages;
 }
 /**
@@ -34474,6 +34507,40 @@ function fixOverlappingFilters(data) {
     }
     return data;
 }
+function fixTranslatedDuplicateSheetName(data, initialMessages) {
+    const sheetNames = {};
+    for (const sheet of data.sheets || []) {
+        sheetNames[sheet.id] = sheet.name;
+    }
+    const messages = [];
+    for (const message of initialMessages) {
+        if (message.type === "REMOTE_REVISION") {
+            const commands = [];
+            for (const cmd of message.commands) {
+                switch (cmd.type) {
+                    case "DUPLICATE_SHEET":
+                        cmd.sheetNameTo =
+                            cmd.sheetNameTo ??
+                                getDuplicateSheetName(sheetNames[cmd.sheetId], Object.values(sheetNames));
+                        break;
+                    case "CREATE_SHEET":
+                    case "RENAME_SHEET":
+                        sheetNames[cmd.sheetId] = cmd.name || getNextSheetName(Object.values(sheetNames));
+                        break;
+                }
+                commands.push(cmd);
+            }
+            messages.push({
+                ...message,
+                commands,
+            });
+        }
+        else {
+            messages.push(message);
+        }
+    }
+    return initialMessages;
+}
 // -----------------------------------------------------------------------------
 // Helpers
 // -----------------------------------------------------------------------------
@@ -36704,9 +36771,7 @@ class ChartPlugin extends CorePlugin {
             : "Success" /* CommandResult.Success */;
     }
     checkChartExists(cmd) {
-        return this.getters.getFigureSheetId(cmd.id)
-            ? "Success" /* CommandResult.Success */
-            : "ChartDoesNotExist" /* CommandResult.ChartDoesNotExist */;
+        return this.isChartDefined(cmd.id) ? "Success" /* CommandResult.Success */ : "ChartDoesNotExist" /* CommandResult.ChartDoesNotExist */;
     }
 }
 
@@ -39505,6 +39570,7 @@ class SheetPlugin extends CorePlugin {
         "getCommandZones",
         "getUnboundedZone",
         "checkElementsIncludeAllNonFrozenHeaders",
+        "getDuplicateSheetName",
     ];
     sheetIdsMapName = {};
     orderedSheetIds = [];
@@ -39529,7 +39595,11 @@ class SheetPlugin extends CorePlugin {
                 return this.checkValidations(cmd, this.checkSheetName, this.checkSheetPosition);
             }
             case "DUPLICATE_SHEET": {
-                return this.sheets[cmd.sheetIdTo] ? "DuplicatedSheetId" /* CommandResult.DuplicatedSheetId */ : "Success" /* CommandResult.Success */;
+                if (this.sheets[cmd.sheetIdTo])
+                    return "DuplicatedSheetId" /* CommandResult.DuplicatedSheetId */;
+                if (this.orderedSheetIds.map(this.getSheetName.bind(this)).includes(cmd.sheetNameTo))
+                    return "DuplicatedSheetName" /* CommandResult.DuplicatedSheetName */;
+                return "Success" /* CommandResult.Success */;
             }
             case "MOVE_SHEET":
                 try {
@@ -39602,7 +39672,7 @@ class SheetPlugin extends CorePlugin {
                 this.showSheet(cmd.sheetId);
                 break;
             case "DUPLICATE_SHEET":
-                this.duplicateSheet(cmd.sheetId, cmd.sheetIdTo);
+                this.duplicateSheet(cmd.sheetId, cmd.sheetIdTo, cmd.sheetNameTo);
                 break;
             case "DELETE_SHEET":
                 this.deleteSheet(this.sheets[cmd.sheetId]);
@@ -39812,14 +39882,8 @@ class SheetPlugin extends CorePlugin {
         return dimension === "COL" ? this.getNumberCols(sheetId) : this.getNumberRows(sheetId);
     }
     getNextSheetName(baseName = "Sheet") {
-        let i = 1;
         const names = this.orderedSheetIds.map(this.getSheetName.bind(this));
-        let name = `${baseName}${i}`;
-        while (names.includes(name)) {
-            name = `${baseName}${i}`;
-            i++;
-        }
-        return name;
+        return getNextSheetName(names, baseName);
     }
     getSheetSize(sheetId) {
         return {
@@ -40079,9 +40143,8 @@ class SheetPlugin extends CorePlugin {
     showSheet(sheetId) {
         this.history.update("sheets", sheetId, "isVisible", true);
     }
-    duplicateSheet(fromId, toId) {
+    duplicateSheet(fromId, toId, toName) {
         const sheet = this.getSheet(fromId);
-        const toName = this.getDuplicateSheetName(sheet.name);
         const newSheet = deepCopy(sheet);
         newSheet.id = toId;
         newSheet.name = toName;
@@ -40113,15 +40176,8 @@ class SheetPlugin extends CorePlugin {
         this.history.update("sheetIdsMapName", sheetIdsMapName);
     }
     getDuplicateSheetName(sheetName) {
-        let i = 1;
         const names = this.orderedSheetIds.map(this.getSheetName.bind(this));
-        const baseName = _t("Copy of %s", sheetName);
-        let name = baseName.toString();
-        while (names.includes(name)) {
-            name = `${baseName} (${i})`;
-            i++;
-        }
-        return name;
+        return getDuplicateSheetName(sheetName, names);
     }
     deleteSheet(sheet) {
         const name = sheet.name;
@@ -50605,6 +50661,8 @@ class GridSelectionPlugin extends UIPlugin {
             });
             this.selectCell(col, row);
         }
+        const { col, row } = this.gridSelection.anchor.cell;
+        this.moveClient({ sheetId: this.activeSheet.id, col, row });
     }
     /**
      * Ensure selections are not outside sheet boundaries.
@@ -51344,8 +51402,11 @@ class SheetViewPlugin extends UIPlugin {
             }
             case "REMOVE_FILTER_TABLE":
             case "UPDATE_FILTER":
-                this.sheetsWithDirtyViewports.add(cmd.sheetId);
-                break;
+            case "UNFREEZE_ROWS":
+            case "UNFREEZE_COLUMNS":
+            case "FREEZE_COLUMNS":
+            case "FREEZE_ROWS":
+            case "UNFREEZE_COLUMNS_ROWS":
             case "REMOVE_COLUMNS_ROWS":
             case "RESIZE_COLUMNS_ROWS":
             case "HIDE_COLUMNS_ROWS":
@@ -51358,11 +51419,9 @@ class SheetViewPlugin extends UIPlugin {
             case "FOLD_HEADER_GROUPS_IN_ZONE":
             case "UNFOLD_HEADER_GROUPS_IN_ZONE":
             case "UNFOLD_ALL_HEADER_GROUPS":
-            case "FOLD_ALL_HEADER_GROUPS": {
-                const sheetId = "sheetId" in cmd ? cmd.sheetId : this.getters.getActiveSheetId();
-                this.sheetsWithDirtyViewports.add(sheetId);
+            case "FOLD_ALL_HEADER_GROUPS":
+                this.sheetsWithDirtyViewports.add(cmd.sheetId);
                 break;
-            }
             case "UPDATE_CELL":
                 // update cell content or format can change hidden rows because of data filters
                 if ("content" in cmd || "format" in cmd || cmd.style?.fontSize !== undefined) {
@@ -51378,13 +51437,6 @@ class SheetViewPlugin extends UIPlugin {
             case "ACTIVATE_SHEET":
                 this.sheetsWithDirtyViewports.add(cmd.sheetIdTo);
                 break;
-            case "UNFREEZE_ROWS":
-            case "UNFREEZE_COLUMNS":
-            case "FREEZE_COLUMNS":
-            case "FREEZE_ROWS":
-            case "UNFREEZE_COLUMNS_ROWS":
-                this.resetViewports(this.getters.getActiveSheetId());
-                break;
             case "DELETE_SHEET":
                 this.sheetsWithDirtyViewports.delete(cmd.sheetId);
                 break;
@@ -58256,6 +58308,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.0.59";
-__info__.date = "2025-04-25T08:04:05.375Z";
-__info__.hash = "eb92388";
+__info__.version = "17.0.61";
+__info__.date = "2025-05-12T05:21:02.159Z";
+__info__.hash = "0368641";

package/dist/o-spreadsheet.d.ts

@@ -1892,6 +1892,7 @@ interface DeleteSheetCommand extends SheetDependentCommand {
 interface DuplicateSheetCommand extends SheetDependentCommand {
     type: "DUPLICATE_SHEET";
     sheetIdTo: UID;
+    sheetNameTo: string;
 }
 interface MoveSheetCommand extends SheetDependentCommand {
     type: "MOVE_SHEET";
@@ -4030,7 +4031,7 @@ interface SheetState {
     readonly cellPosition: Record<UID, CellPosition | undefined>;
 }
 declare class SheetPlugin extends CorePlugin<SheetState> implements SheetState {
-    static getters: readonly ["getSheetName", "tryGetSheetName", "getSheet", "tryGetSheet", "getSheetIdByName", "getSheetIds", "getVisibleSheetIds", "isSheetVisible", "getEvaluationSheets", "doesHeaderExist", "doesHeadersExist", "getCell", "getCellPosition", "getColsZone", "getRowCells", "getRowsZone", "getNumberCols", "getNumberRows", "getNumberHeaders", "getGridLinesVisibility", "getNextSheetName", "getSheetSize", "getSheetZone", "getPaneDivisions", "checkZonesExistInSheet", "getCommandZones", "getUnboundedZone", "checkElementsIncludeAllNonFrozenHeaders"];
+    static getters: readonly ["getSheetName", "tryGetSheetName", "getSheet", "tryGetSheet", "getSheetIdByName", "getSheetIds", "getVisibleSheetIds", "isSheetVisible", "getEvaluationSheets", "doesHeaderExist", "doesHeadersExist", "getCell", "getCellPosition", "getColsZone", "getRowCells", "getRowsZone", "getNumberCols", "getNumberRows", "getNumberHeaders", "getGridLinesVisibility", "getNextSheetName", "getSheetSize", "getSheetZone", "getPaneDivisions", "checkZonesExistInSheet", "getCommandZones", "getUnboundedZone", "checkElementsIncludeAllNonFrozenHeaders", "getDuplicateSheetName"];
     readonly sheetIdsMapName: Record<string, UID | undefined>;
     readonly orderedSheetIds: UID[];
     readonly sheets: Record<UID, Sheet | undefined>;
@@ -4110,7 +4111,7 @@ declare class SheetPlugin extends CorePlugin<SheetState> implements SheetState {
     private hideSheet;
     private showSheet;
     private duplicateSheet;
-    private getDuplicateSheetName;
+    getDuplicateSheetName(sheetName: string): string;
     private deleteSheet;
     /**
      * Delete column. This requires a lot of handling:
@@ -7342,6 +7343,7 @@ declare class Composer extends Component<ComposerProps, SpreadsheetChildEnv> {
     onClick(): void;
     onDblClick(): void;
     onContextMenu(ev: MouseEvent): void;
+    onWheel(event: WheelEvent): void;
     private processContent;
     /**
      * Get the HTML content corresponding to the current composer token, divided by lines.