@odoo/o-spreadsheet 19.0.1 → 19.0.3

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 19.0.1
- * @date 2025-09-11T08:45:47.109Z
- * @hash bd79eea
+ * @version 19.0.3
+ * @date 2025-09-19T07:26:41.356Z
+ * @hash 84f3b74
  */
 
 'use strict';
@@ -890,6 +890,7 @@ const tokenColors = {
     ARG_SEPARATOR: functionColor,
     ORPHAN_RIGHT_PAREN: "#ff0000",
 };
+const DRAG_THRESHOLD = 5; // in pixels, to avoid unwanted drag when clicking
 
 //------------------------------------------------------------------------------
 // Miscellaneous
@@ -1368,8 +1369,19 @@ function memoize(func) {
         },
     }[funcName];
 }
+/**
+ * Removes the specified indexes from the array.
+ * Sparse (empty) elements are transformed to undefined (unless their index is explicitly removed).
+ */
 function removeIndexesFromArray(array, indexes) {
-    return array.filter((_, index) => !indexes.includes(index));
+    const toRemove = new Set(indexes);
+    const newArray = [];
+    for (let i = 0; i < array.length; i++) {
+        if (!toRemove.has(i)) {
+            newArray.push(array[i]);
+        }
+    }
+    return newArray;
 }
 function insertItemsAtIndex(array, items, index) {
     const newArray = [...array];
@@ -16084,8 +16096,9 @@ function interactiveSort(env, sheetId, anchor, zone, sortDirection, sortOptions)
 }
 
 function sortMatrix(matrix, locale, ...criteria) {
-    for (const [i, value] of criteria.entries()) {
-        assert(value !== undefined, _t("Value for parameter %d is missing, while the function [[FUNCTION_NAME]] expect a number or a range.", i + 1));
+    for (let i = 0; i < criteria.length; i++) {
+        const param = i % 2 === 0 ? "sort_column" : "is_ascending";
+        assert(criteria[i] !== undefined, _t("Value for parameter %s is missing in [[FUNCTION_NAME]].", param));
     }
     const sortingOrders = [];
     const sortColumns = [];
@@ -32380,7 +32393,11 @@ class CarouselFigure extends owl.Component {
         onFigureDeleted: Function,
         editFigureStyle: { type: Function, optional: true },
     };
-    static components = { ChartDashboardMenu };
+    static components = { ChartDashboardMenu, MenuPopover };
+    carouselTabsRef = owl.useRef("carouselTabs");
+    carouselTabsDropdownRef = owl.useRef("carouselTabsDropdown");
+    menuState = owl.useState({ isOpen: false, anchorRect: null, menuItems: [] });
+    hiddenItems = [];
     animationStore;
     setup() {
         this.animationStore = useStore(ChartAnimationStore);
@@ -32391,6 +32408,7 @@ class CarouselFigure extends owl.Component {
             else {
                 this.props.editFigureStyle?.({ "pointer-events": "auto" });
             }
+            this.updateTabsVisibility();
         });
     }
     get carousel() {
@@ -32450,6 +32468,49 @@ class CarouselFigure extends owl.Component {
         const style = { ...DEFAULT_CAROUSEL_TITLE_STYLE, ...this.carousel.title };
         return cssPropertiesToCss(cellTextStyleToCss(chartStyleToCellStyle(style)));
     }
+    updateTabsVisibility() {
+        const tabsContainerEl = this.carouselTabsRef.el;
+        const dropDownEl = this.carouselTabsDropdownRef.el;
+        if (!tabsContainerEl || !dropDownEl) {
+            return;
+        }
+        this.hiddenItems = [];
+        const containerRect = getBoundingRectAsPOJO(tabsContainerEl);
+        const tabs = Array.from(tabsContainerEl.children);
+        for (const tab of tabs) {
+            tab.style.display = "block";
+        }
+        const tabWidths = tabs.map((tab) => getBoundingRectAsPOJO(tab).width);
+        let currentWidth = 0;
+        for (let i = 0; i < tabs.length; i++) {
+            const shouldBeHidden = currentWidth + tabWidths[i] > containerRect.width;
+            currentWidth += tabWidths[i];
+            if (shouldBeHidden) {
+                tabs[i].style.display = "none";
+                this.hiddenItems.push(this.carousel.items[i]);
+            }
+        }
+        dropDownEl.style.display = this.hiddenItems.length ? "block" : "none";
+    }
+    get menuId() {
+        return "carousel-tabs-menu-";
+    }
+    toggleMenu(ev) {
+        if (ev.closedMenuId === this.menuId) {
+            this.menuState.isOpen = false;
+            return;
+        }
+        const rect = getRefBoundingRect(this.carouselTabsDropdownRef);
+        const menuItems = this.hiddenItems.map((item) => ({
+            name: this.getItemTitle(item),
+            execute: () => this.onCarouselTabClick(item),
+            isActive: () => this.isItemSelected(item),
+            isReadonlyAllowed: true,
+        }));
+        this.menuState.isOpen = true;
+        this.menuState.anchorRect = rect;
+        this.menuState.menuItems = createActions(menuItems);
+    }
 }
 
 class ChartFigure extends owl.Component {
@@ -50437,9 +50498,16 @@ class FiguresContainer extends owl.Component {
             maxX: this.env.model.getters.getColDimensions(sheetId, this.env.model.getters.getNumberCols(sheetId) - 1).end,
             maxY: this.env.model.getters.getRowDimensions(sheetId, this.env.model.getters.getNumberRows(sheetId) - 1).end,
         };
+        let hasStartedDnd = false;
         const onMouseMove = (ev) => {
             const getters = this.env.model.getters;
             const currentMousePosition = { x: ev.clientX, y: ev.clientY };
+            const offsetX = Math.abs(currentMousePosition.x - initialMousePosition.x);
+            const offsetY = Math.abs(currentMousePosition.y - initialMousePosition.y);
+            if (!hasStartedDnd && offsetX < DRAG_THRESHOLD && offsetY < DRAG_THRESHOLD) {
+                return; // add a small threshold to avoid dnd when just clicking
+            }
+            hasStartedDnd = true;
             const draggedFigure = dragFigureForMove(currentMousePosition, initialMousePosition, initialFigure, maxDimensions, initialScrollPosition, getters.getActiveSheetScrollInfo());
             const otherFigures = this.getOtherFigures(initialFigure.id);
             const overlappingCarousel = this.getCarouselOverlappingChart(draggedFigure, otherFigures);
@@ -58851,12 +58919,13 @@ class PivotLayoutConfigurator extends owl.Component {
     addCalculatedMeasure() {
         const { measures } = this.props.definition;
         const measureName = this.env.model.getters.generateNewCalculatedMeasureName(measures);
+        const aggregator = "sum";
         this.props.onDimensionsUpdated({
             measures: measures.concat([
                 {
-                    id: this.getMeasureId(measureName),
+                    id: this.getMeasureId(measureName, aggregator),
                     fieldName: measureName,
-                    aggregator: "sum",
+                    aggregator,
                     computedBy: {
                         sheetId: this.env.model.getters.getActiveSheetId(),
                         formula: "=0",
@@ -71161,7 +71230,7 @@ function withPivotPresentationLayer (PivotClass) {
                 return { value: 0 };
             }
             const { columns, rows } = super.definition;
-            if (columns.length + rows.length !== domain.length) {
+            if (measure.aggregator && columns.length + rows.length !== domain.length) {
                 const values = this.getValuesToAggregate(measure, domain);
                 const aggregator = AGGREGATORS_FN[measure.aggregator];
                 if (!aggregator) {
@@ -71180,11 +71249,17 @@ function withPivotPresentationLayer (PivotClass) {
                 if (columns.find((col) => col.nameWithGranularity === symbolName)) {
                     const { colDomain } = domainToColRowDomain(this, domain);
                     const symbolIndex = colDomain.findIndex((node) => node.field === symbolName);
+                    if (symbolIndex === -1) {
+                        return new NotAvailableError();
+                    }
                     return this.getPivotHeaderValueAndFormat(colDomain.slice(0, symbolIndex + 1));
                 }
                 if (rows.find((row) => row.nameWithGranularity === symbolName)) {
                     const { rowDomain } = domainToColRowDomain(this, domain);
                     const symbolIndex = rowDomain.findIndex((row) => row.field === symbolName);
+                    if (symbolIndex === -1) {
+                        return new NotAvailableError();
+                    }
                     return this.getPivotHeaderValueAndFormat(rowDomain.slice(0, symbolIndex + 1));
                 }
                 return this.getPivotCellValueAndFormat(symbolName, domain);
@@ -77563,13 +77638,10 @@ class GridSelectionPlugin extends UIPlugin {
         const deltaCol = isBasedBefore && isCol ? thickness : 0;
         const deltaRow = isBasedBefore && !isCol ? thickness : 0;
         const toRemove = isBasedBefore ? cmd.elements.map((el) => el + thickness) : cmd.elements;
-        const originalSize = Object.fromEntries(toRemove.map((element) => {
-            const size = isCol
-                ? this.getters.getColSize(cmd.sheetId, element)
-                : this.getters.getUserRowSize(cmd.sheetId, element);
-            const isDefaultCol = isCol && size === DEFAULT_CELL_WIDTH;
-            return [element, isDefaultCol ? undefined : size];
-        }));
+        const originalSize = {};
+        for (const element of toRemove) {
+            originalSize[element] = this.getters.getHeaderSize(cmd.sheetId, cmd.dimension, element);
+        }
         const target = [
             {
                 left: isCol ? start + deltaCol : 0,
@@ -77613,11 +77685,11 @@ class GridSelectionPlugin extends UIPlugin {
         for (const element of toRemove) {
             const size = originalSize[element];
             const currentSize = this.getters.getHeaderSize(cmd.sheetId, cmd.dimension, currentIndex);
-            if (size && size !== currentSize) {
+            if (size !== currentSize) {
                 resizingGroups[size] ??= [];
                 resizingGroups[size].push(currentIndex);
-                currentIndex += 1;
             }
+            currentIndex += 1;
         }
         for (const size in resizingGroups) {
             this.dispatch("RESIZE_COLUMNS_ROWS", {
@@ -88457,6 +88529,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "19.0.1";
-__info__.date = "2025-09-11T08:45:47.109Z";
-__info__.hash = "bd79eea";
+__info__.version = "19.0.3";
+__info__.date = "2025-09-19T07:26:41.356Z";
+__info__.hash = "84f3b74";

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 19.0.1
- * @date 2025-09-11T08:45:47.109Z
- * @hash bd79eea
+ * @version 19.0.3
+ * @date 2025-09-19T07:26:41.356Z
+ * @hash 84f3b74
  */
 
 import { useEnv, useSubEnv, onWillUnmount, useComponent, status, Component, useRef, onMounted, useEffect, App, blockDom, useState, onPatched, useExternalListener, onWillUpdateProps, onWillStart, onWillPatch, xml, useChildSubEnv, markRaw, toRaw } from '@odoo/owl';
@@ -888,6 +888,7 @@ const tokenColors = {
     ARG_SEPARATOR: functionColor,
     ORPHAN_RIGHT_PAREN: "#ff0000",
 };
+const DRAG_THRESHOLD = 5; // in pixels, to avoid unwanted drag when clicking
 
 //------------------------------------------------------------------------------
 // Miscellaneous
@@ -1366,8 +1367,19 @@ function memoize(func) {
         },
     }[funcName];
 }
+/**
+ * Removes the specified indexes from the array.
+ * Sparse (empty) elements are transformed to undefined (unless their index is explicitly removed).
+ */
 function removeIndexesFromArray(array, indexes) {
-    return array.filter((_, index) => !indexes.includes(index));
+    const toRemove = new Set(indexes);
+    const newArray = [];
+    for (let i = 0; i < array.length; i++) {
+        if (!toRemove.has(i)) {
+            newArray.push(array[i]);
+        }
+    }
+    return newArray;
 }
 function insertItemsAtIndex(array, items, index) {
     const newArray = [...array];
@@ -16082,8 +16094,9 @@ function interactiveSort(env, sheetId, anchor, zone, sortDirection, sortOptions)
 }
 
 function sortMatrix(matrix, locale, ...criteria) {
-    for (const [i, value] of criteria.entries()) {
-        assert(value !== undefined, _t("Value for parameter %d is missing, while the function [[FUNCTION_NAME]] expect a number or a range.", i + 1));
+    for (let i = 0; i < criteria.length; i++) {
+        const param = i % 2 === 0 ? "sort_column" : "is_ascending";
+        assert(criteria[i] !== undefined, _t("Value for parameter %s is missing in [[FUNCTION_NAME]].", param));
     }
     const sortingOrders = [];
     const sortColumns = [];
@@ -32378,7 +32391,11 @@ class CarouselFigure extends Component {
         onFigureDeleted: Function,
         editFigureStyle: { type: Function, optional: true },
     };
-    static components = { ChartDashboardMenu };
+    static components = { ChartDashboardMenu, MenuPopover };
+    carouselTabsRef = useRef("carouselTabs");
+    carouselTabsDropdownRef = useRef("carouselTabsDropdown");
+    menuState = useState({ isOpen: false, anchorRect: null, menuItems: [] });
+    hiddenItems = [];
     animationStore;
     setup() {
         this.animationStore = useStore(ChartAnimationStore);
@@ -32389,6 +32406,7 @@ class CarouselFigure extends Component {
             else {
                 this.props.editFigureStyle?.({ "pointer-events": "auto" });
             }
+            this.updateTabsVisibility();
         });
     }
     get carousel() {
@@ -32448,6 +32466,49 @@ class CarouselFigure extends Component {
         const style = { ...DEFAULT_CAROUSEL_TITLE_STYLE, ...this.carousel.title };
         return cssPropertiesToCss(cellTextStyleToCss(chartStyleToCellStyle(style)));
     }
+    updateTabsVisibility() {
+        const tabsContainerEl = this.carouselTabsRef.el;
+        const dropDownEl = this.carouselTabsDropdownRef.el;
+        if (!tabsContainerEl || !dropDownEl) {
+            return;
+        }
+        this.hiddenItems = [];
+        const containerRect = getBoundingRectAsPOJO(tabsContainerEl);
+        const tabs = Array.from(tabsContainerEl.children);
+        for (const tab of tabs) {
+            tab.style.display = "block";
+        }
+        const tabWidths = tabs.map((tab) => getBoundingRectAsPOJO(tab).width);
+        let currentWidth = 0;
+        for (let i = 0; i < tabs.length; i++) {
+            const shouldBeHidden = currentWidth + tabWidths[i] > containerRect.width;
+            currentWidth += tabWidths[i];
+            if (shouldBeHidden) {
+                tabs[i].style.display = "none";
+                this.hiddenItems.push(this.carousel.items[i]);
+            }
+        }
+        dropDownEl.style.display = this.hiddenItems.length ? "block" : "none";
+    }
+    get menuId() {
+        return "carousel-tabs-menu-";
+    }
+    toggleMenu(ev) {
+        if (ev.closedMenuId === this.menuId) {
+            this.menuState.isOpen = false;
+            return;
+        }
+        const rect = getRefBoundingRect(this.carouselTabsDropdownRef);
+        const menuItems = this.hiddenItems.map((item) => ({
+            name: this.getItemTitle(item),
+            execute: () => this.onCarouselTabClick(item),
+            isActive: () => this.isItemSelected(item),
+            isReadonlyAllowed: true,
+        }));
+        this.menuState.isOpen = true;
+        this.menuState.anchorRect = rect;
+        this.menuState.menuItems = createActions(menuItems);
+    }
 }
 
 class ChartFigure extends Component {
@@ -50435,9 +50496,16 @@ class FiguresContainer extends Component {
             maxX: this.env.model.getters.getColDimensions(sheetId, this.env.model.getters.getNumberCols(sheetId) - 1).end,
             maxY: this.env.model.getters.getRowDimensions(sheetId, this.env.model.getters.getNumberRows(sheetId) - 1).end,
         };
+        let hasStartedDnd = false;
         const onMouseMove = (ev) => {
             const getters = this.env.model.getters;
             const currentMousePosition = { x: ev.clientX, y: ev.clientY };
+            const offsetX = Math.abs(currentMousePosition.x - initialMousePosition.x);
+            const offsetY = Math.abs(currentMousePosition.y - initialMousePosition.y);
+            if (!hasStartedDnd && offsetX < DRAG_THRESHOLD && offsetY < DRAG_THRESHOLD) {
+                return; // add a small threshold to avoid dnd when just clicking
+            }
+            hasStartedDnd = true;
             const draggedFigure = dragFigureForMove(currentMousePosition, initialMousePosition, initialFigure, maxDimensions, initialScrollPosition, getters.getActiveSheetScrollInfo());
             const otherFigures = this.getOtherFigures(initialFigure.id);
             const overlappingCarousel = this.getCarouselOverlappingChart(draggedFigure, otherFigures);
@@ -58849,12 +58917,13 @@ class PivotLayoutConfigurator extends Component {
     addCalculatedMeasure() {
         const { measures } = this.props.definition;
         const measureName = this.env.model.getters.generateNewCalculatedMeasureName(measures);
+        const aggregator = "sum";
         this.props.onDimensionsUpdated({
             measures: measures.concat([
                 {
-                    id: this.getMeasureId(measureName),
+                    id: this.getMeasureId(measureName, aggregator),
                     fieldName: measureName,
-                    aggregator: "sum",
+                    aggregator,
                     computedBy: {
                         sheetId: this.env.model.getters.getActiveSheetId(),
                         formula: "=0",
@@ -71159,7 +71228,7 @@ function withPivotPresentationLayer (PivotClass) {
                 return { value: 0 };
             }
             const { columns, rows } = super.definition;
-            if (columns.length + rows.length !== domain.length) {
+            if (measure.aggregator && columns.length + rows.length !== domain.length) {
                 const values = this.getValuesToAggregate(measure, domain);
                 const aggregator = AGGREGATORS_FN[measure.aggregator];
                 if (!aggregator) {
@@ -71178,11 +71247,17 @@ function withPivotPresentationLayer (PivotClass) {
                 if (columns.find((col) => col.nameWithGranularity === symbolName)) {
                     const { colDomain } = domainToColRowDomain(this, domain);
                     const symbolIndex = colDomain.findIndex((node) => node.field === symbolName);
+                    if (symbolIndex === -1) {
+                        return new NotAvailableError();
+                    }
                     return this.getPivotHeaderValueAndFormat(colDomain.slice(0, symbolIndex + 1));
                 }
                 if (rows.find((row) => row.nameWithGranularity === symbolName)) {
                     const { rowDomain } = domainToColRowDomain(this, domain);
                     const symbolIndex = rowDomain.findIndex((row) => row.field === symbolName);
+                    if (symbolIndex === -1) {
+                        return new NotAvailableError();
+                    }
                     return this.getPivotHeaderValueAndFormat(rowDomain.slice(0, symbolIndex + 1));
                 }
                 return this.getPivotCellValueAndFormat(symbolName, domain);
@@ -77561,13 +77636,10 @@ class GridSelectionPlugin extends UIPlugin {
         const deltaCol = isBasedBefore && isCol ? thickness : 0;
         const deltaRow = isBasedBefore && !isCol ? thickness : 0;
         const toRemove = isBasedBefore ? cmd.elements.map((el) => el + thickness) : cmd.elements;
-        const originalSize = Object.fromEntries(toRemove.map((element) => {
-            const size = isCol
-                ? this.getters.getColSize(cmd.sheetId, element)
-                : this.getters.getUserRowSize(cmd.sheetId, element);
-            const isDefaultCol = isCol && size === DEFAULT_CELL_WIDTH;
-            return [element, isDefaultCol ? undefined : size];
-        }));
+        const originalSize = {};
+        for (const element of toRemove) {
+            originalSize[element] = this.getters.getHeaderSize(cmd.sheetId, cmd.dimension, element);
+        }
         const target = [
             {
                 left: isCol ? start + deltaCol : 0,
@@ -77611,11 +77683,11 @@ class GridSelectionPlugin extends UIPlugin {
         for (const element of toRemove) {
             const size = originalSize[element];
             const currentSize = this.getters.getHeaderSize(cmd.sheetId, cmd.dimension, currentIndex);
-            if (size && size !== currentSize) {
+            if (size !== currentSize) {
                 resizingGroups[size] ??= [];
                 resizingGroups[size].push(currentIndex);
-                currentIndex += 1;
             }
+            currentIndex += 1;
         }
         for (const size in resizingGroups) {
             this.dispatch("RESIZE_COLUMNS_ROWS", {
@@ -88405,6 +88477,6 @@ const chartHelpers = { ...CHART_HELPERS, ...CHART_RUNTIME_HELPERS };
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, ClientDisconnectedError, CommandResult, CorePlugin, CoreViewPlugin, DispatchResult, EvaluationError, LocalTransportService, Model, PivotRuntimeDefinition, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, SpreadsheetPivotTable, UIPlugin, __info__, addFunction, addRenderingLayer, astToFormula, chartHelpers, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, getCaretDownSvg, getCaretUpSvg, helpers, hooks, invalidateCFEvaluationCommands, invalidateChartEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, stores, tokenColors, tokenize };
 
 
-__info__.version = "19.0.1";
-__info__.date = "2025-09-11T08:45:47.109Z";
-__info__.hash = "bd79eea";
+__info__.version = "19.0.3";
+__info__.date = "2025-09-19T07:26:41.356Z";
+__info__.hash = "84f3b74";