@odoo/o-spreadsheet 19.0.15 → 19.0.17

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.15
- * @date 2025-12-26T10:19:23.408Z
- * @hash fe625c9
+ * @version 19.0.17
+ * @date 2026-01-14T10:01:24.044Z
+ * @hash 2165bad
  */
 
 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';
@@ -20098,9 +20098,10 @@ function assertDomainLength(domain) {
         throw new EvaluationError(_t("Function PIVOT takes an even number of arguments."));
     }
 }
-function addPivotDependencies(evalContext, coreDefinition, forMeasures) {
+function addPivotDependencies(evalContext, pivotId, forMeasures) {
     //TODO This function can be very costly when used with PIVOT.VALUE and PIVOT.HEADER
     const dependencies = [];
+    const coreDefinition = evalContext.getters.getPivotCoreDefinition(pivotId);
     if (coreDefinition.type === "SPREADSHEET" && coreDefinition.dataSet) {
         const { sheetId, zone } = coreDefinition.dataSet;
         const xc = zoneToXc(zone);
@@ -20117,8 +20118,7 @@ function addPivotDependencies(evalContext, coreDefinition, forMeasures) {
     }
     for (const measure of forMeasures) {
         if (measure.computedBy) {
-            const formula = evalContext.getters.getMeasureCompiledFormula(measure);
-            dependencies.push(...formula.dependencies.filter((range) => !range.invalidXc));
+            dependencies.push(...evalContext.getters.getMeasureFullDependencies(pivotId, measure));
         }
     }
     const originPosition = evalContext.__originCellPosition;
@@ -20615,7 +20615,7 @@ const PIVOT_VALUE = {
         assertDomainLength(domainArgs);
         const pivot = this.getters.getPivot(pivotId);
         const coreDefinition = this.getters.getPivotCoreDefinition(pivotId);
-        addPivotDependencies(this, coreDefinition, coreDefinition.measures.filter((m) => m.id === _measure));
+        addPivotDependencies(this, pivotId, coreDefinition.measures.filter((m) => m.id === _measure));
         pivot.init({ reload: pivot.needsReevaluation });
         const error = pivot.assertIsValid({ throwOnError: false });
         if (error) {
@@ -20648,8 +20648,7 @@ const PIVOT_HEADER = {
         const _pivotId = getPivotId(_pivotFormulaId, this.getters);
         assertDomainLength(domainArgs);
         const pivot = this.getters.getPivot(_pivotId);
-        const coreDefinition = this.getters.getPivotCoreDefinition(_pivotId);
-        addPivotDependencies(this, coreDefinition, []);
+        addPivotDependencies(this, _pivotId, []);
         pivot.init({ reload: pivot.needsReevaluation });
         const error = pivot.assertIsValid({ throwOnError: false });
         if (error) {
@@ -20707,7 +20706,7 @@ const PIVOT = {
         const pivotId = getPivotId(_pivotFormulaId, this.getters);
         const pivot = this.getters.getPivot(pivotId);
         const coreDefinition = this.getters.getPivotCoreDefinition(pivotId);
-        addPivotDependencies(this, coreDefinition, coreDefinition.measures);
+        addPivotDependencies(this, pivotId, coreDefinition.measures);
         pivot.init({ reload: pivot.needsReevaluation });
         const error = pivot.assertIsValid({ throwOnError: false });
         if (error) {
@@ -35482,7 +35481,7 @@ class AbstractComposerStore extends SpreadsheetStore {
     }
     captureSelection(zone, col, row) {
         this.model.selection.capture(this, {
-            cell: { col: col ?? zone.left, row: row ?? zone.right },
+            cell: { col: col ?? zone.left, row: row ?? zone.top },
             zone,
         }, {
             handleEvent: this.handleEvent.bind(this),
@@ -51369,7 +51368,6 @@ class GridOverlay extends Component {
         onGridMoved: Function,
         gridOverlayDimensions: String,
         slots: { type: Object, optional: true },
-        getGridSize: Function,
     };
     static components = {
         FiguresContainer,
@@ -51388,14 +51386,7 @@ class GridOverlay extends Component {
     setup() {
         useCellHovered(this.env, this.gridOverlay);
         const resizeObserver = new ResizeObserver(() => {
-            const boundingRect = this.gridOverlayEl.getBoundingClientRect();
-            const { width, height } = this.props.getGridSize();
-            this.props.onGridResized({
-                x: boundingRect.left,
-                y: boundingRect.top,
-                height: height,
-                width: width,
-            });
+            this.props.onGridResized();
         });
         onMounted(() => {
             resizeObserver.observe(this.gridOverlayEl);
@@ -57883,6 +57874,12 @@ class FindAndReplaceStore extends SpreadsheetStore {
             case "ACTIVATE_SHEET":
                 this.isSearchDirty = true;
                 this.shouldFinalizeUpdateSelection = true;
+                if (this.searchOptions.specificRange) {
+                    this.searchOptions.specificRange = {
+                        ...this.searchOptions.specificRange,
+                        sheetId: this.getters.getActiveSheetId(),
+                    };
+                }
                 break;
             case "REPLACE_SEARCH":
                 for (const match of cmd.matches) {
@@ -58302,9 +58299,20 @@ class FindAndReplacePanel extends Component {
         const specificRange = this.env.model.getters.getRangeFromSheetXC(this.env.model.getters.getActiveSheetId(), this.state.dataRange);
         this.store.updateSearchOptions({ specificRange });
     }
+    get specificRange() {
+        const range = this.store.searchOptions.specificRange;
+        return range ? this.env.model.getters.getRangeString(range, "forceSheetReference") : "";
+    }
     get pendingSearch() {
         return this.updateSearchContent.isDebouncePending();
     }
+    get selectionInputKey() {
+        // Selections input are made to work with objects linked to a sheet id. They store the active sheet id at their creation,
+        // and have specific behaviour linked to it (eg. go back to the initial sheet after confirmation).
+        // We don't want all those behaviors here, so we force the recreation of the component when the active sheet changes.
+        // The only drawback is that the input loses focus when changing sheet.
+        return this.env.model.getters.getActiveSheetId();
+    }
 }
 
 css /* scss */ `
@@ -61419,7 +61427,8 @@ class Grid extends Component {
         });
         return !(rect.width === 0 || rect.height === 0);
     }
-    onGridResized({ height, width }) {
+    onGridResized() {
+        const { height, width } = this.props.getGridSize();
         this.env.model.dispatch("RESIZE_SHEETVIEW", {
             width: width - HEADER_WIDTH,
             height: height - HEADER_HEIGHT,
@@ -67613,6 +67622,7 @@ class PivotCorePlugin extends CorePlugin {
         "getMeasureCompiledFormula",
         "getPivotName",
         "isExistingPivot",
+        "getMeasureFullDependencies",
     ];
     nextFormulaId = 1;
     pivots = {};
@@ -67695,7 +67705,7 @@ class PivotCorePlugin extends CorePlugin {
             }
             case "UPDATE_PIVOT": {
                 this.history.update("pivots", cmd.pivotId, "definition", deepCopy(cmd.pivot));
-                this.compileCalculatedMeasures(cmd.pivot.measures);
+                this.compileCalculatedMeasures(cmd.pivotId, cmd.pivot.measures);
                 break;
             }
         }
@@ -67713,9 +67723,14 @@ class PivotCorePlugin extends CorePlugin {
                 this.history.update("pivots", pivotId, "definition", newDefinition);
             }
         }
-        for (const sheetId in this.compiledMeasureFormulas) {
-            for (const formulaString in this.compiledMeasureFormulas[sheetId]) {
-                const compiledFormula = this.compiledMeasureFormulas[sheetId][formulaString];
+        for (const pivotId in this.compiledMeasureFormulas) {
+            for (const measureId in this.compiledMeasureFormulas[pivotId]) {
+                const measure = this.pivots[pivotId]?.definition.measures.find((m) => m.id === measureId);
+                if (!measure || !measure.computedBy) {
+                    continue;
+                }
+                const sheetId = measure.computedBy.sheetId;
+                const compiledFormula = this.compiledMeasureFormulas[pivotId][measureId].formula;
                 const newDependencies = [];
                 for (const range of compiledFormula.dependencies) {
                     const change = applyChange(range);
@@ -67727,8 +67742,9 @@ class PivotCorePlugin extends CorePlugin {
                     }
                 }
                 const newFormulaString = this.getters.getFormulaString(sheetId, compiledFormula.tokens, newDependencies);
-                if (newFormulaString !== formulaString) {
-                    this.replaceMeasureFormula(sheetId, formulaString, newFormulaString);
+                const oldFormulaString = measure.computedBy.formula;
+                if (newFormulaString !== oldFormulaString) {
+                    this.replaceMeasureFormula(pivotId, measure, newFormulaString);
                 }
             }
         }
@@ -67766,31 +67782,60 @@ class PivotCorePlugin extends CorePlugin {
     isExistingPivot(pivotId) {
         return pivotId in this.pivots;
     }
-    getMeasureCompiledFormula(measure) {
+    getMeasureCompiledFormula(pivotId, measure) {
         if (!measure.computedBy) {
             throw new Error(`Measure ${measure.fieldName} is not computed by formula`);
         }
-        const sheetId = measure.computedBy.sheetId;
-        return this.compiledMeasureFormulas[sheetId][measure.computedBy.formula];
+        return this.compiledMeasureFormulas[pivotId][measure.id].formula;
+    }
+    getMeasureFullDependencies(pivotId, measure) {
+        if (!measure.computedBy) {
+            throw new Error(`Measure ${measure.fieldName} is not computed by formula`);
+        }
+        return this.compiledMeasureFormulas[pivotId][measure.id].dependencies;
     }
     // -------------------------------------------------------------------------
     // Private
     // -------------------------------------------------------------------------
     addPivot(pivotId, pivot, formulaId = this.nextFormulaId.toString()) {
         this.history.update("pivots", pivotId, { definition: deepCopy(pivot), formulaId });
-        this.compileCalculatedMeasures(pivot.measures);
+        this.compileCalculatedMeasures(pivotId, pivot.measures);
         this.history.update("formulaIds", formulaId, pivotId);
         this.history.update("nextFormulaId", this.nextFormulaId + 1);
     }
-    compileCalculatedMeasures(measures) {
+    compileCalculatedMeasures(pivotId, measures) {
         for (const measure of measures) {
             if (measure.computedBy) {
-                const sheetId = measure.computedBy.sheetId;
                 const compiledFormula = this.compileMeasureFormula(measure.computedBy.sheetId, measure.computedBy.formula);
-                this.history.update("compiledMeasureFormulas", sheetId, measure.computedBy.formula, compiledFormula);
+                this.history.update("compiledMeasureFormulas", pivotId, measure.id, "formula", compiledFormula);
+            }
+        }
+        for (const measure of measures) {
+            if (measure.computedBy) {
+                const dependencies = this.computeMeasureFullDependencies(pivotId, measure);
+                this.history.update("compiledMeasureFormulas", pivotId, measure.id, "dependencies", dependencies);
             }
         }
     }
+    computeMeasureFullDependencies(pivotId, measure, exploredMeasures = new Set()) {
+        const rangeDependencies = [];
+        const definition = this.getPivotCoreDefinition(pivotId);
+        const formula = this.getMeasureCompiledFormula(pivotId, measure);
+        exploredMeasures.add(measure.id);
+        for (const token of formula.tokens) {
+            if (token.type !== "SYMBOL") {
+                continue;
+            }
+            const otherMeasure = definition.measures.find((measureCandidate) => getCanonicalSymbolName(measureCandidate.id) === token.value &&
+                measure.id !== measureCandidate.id);
+            if (!otherMeasure || exploredMeasures.has(otherMeasure.id) || !otherMeasure.computedBy) {
+                continue;
+            }
+            rangeDependencies.push(...this.computeMeasureFullDependencies(pivotId, otherMeasure, exploredMeasures));
+        }
+        rangeDependencies.push(...formula.dependencies.filter((range) => !range.invalidXc));
+        return rangeDependencies;
+    }
     insertPivot(position, formulaId, table) {
         this.resizeSheet(position.sheetId, position, table);
         const pivotCells = table.getPivotCells();
@@ -67849,21 +67894,17 @@ class PivotCorePlugin extends CorePlugin {
             dependencies: rangeDependencies,
         };
     }
-    replaceMeasureFormula(sheetId, formulaString, newFormulaString) {
-        this.history.update("compiledMeasureFormulas", sheetId, formulaString, undefined);
-        this.history.update("compiledMeasureFormulas", sheetId, newFormulaString, this.compileMeasureFormula(sheetId, newFormulaString));
-        for (const pivotId in this.pivots) {
-            const pivot = this.pivots[pivotId];
-            if (!pivot) {
-                continue;
-            }
-            for (const measure of pivot.definition.measures) {
-                if (measure.computedBy?.formula === formulaString) {
-                    const measureIndex = pivot.definition.measures.indexOf(measure);
-                    this.history.update("pivots", pivotId, "definition", "measures", measureIndex, "computedBy", { formula: newFormulaString, sheetId });
-                }
-            }
+    replaceMeasureFormula(pivotId, measure, newFormulaString) {
+        const pivot = this.pivots[pivotId];
+        if (!pivot) {
+            return;
         }
+        const measureIndex = pivot.definition.measures.indexOf(measure);
+        this.history.update("pivots", pivotId, "definition", "measures", measureIndex, "computedBy", {
+            formula: newFormulaString,
+            sheetId: measure.computedBy.sheetId,
+        });
+        this.compileCalculatedMeasures(pivotId, pivot.definition.measures);
     }
     checkSortedColumnInMeasures(definition) {
         const measures = definition.measures.map((measure) => measure.id);
@@ -71713,14 +71754,16 @@ const PERCENT_FORMAT = "0.00%";
 function withPivotPresentationLayer (PivotClass) {
     class PivotPresentationLayer extends PivotClass {
         getters;
+        pivotId;
         cache = {};
         rankAsc = {};
         rankDesc = {};
         runningTotal = {};
         runningTotalInPercent = {};
-        constructor(custom, params) {
+        constructor(pivotId, custom, params) {
             super(custom, params);
             this.getters = params.getters;
+            this.pivotId = pivotId;
         }
         markAsDirtyForEvaluation() {
             this.cache = {};
@@ -71770,7 +71813,7 @@ function withPivotPresentationLayer (PivotClass) {
                     return handleError(error, measure.aggregator.toUpperCase());
                 }
             }
-            const formula = this.getters.getMeasureCompiledFormula(measure);
+            const formula = this.getters.getMeasureCompiledFormula(this.pivotId, measure);
             const getSymbolValue = (symbolName) => {
                 const { columns, rows } = this.definition;
                 if (columns.find((col) => col.nameWithGranularity === symbolName)) {
@@ -72525,7 +72568,7 @@ class PivotUIPlugin extends CoreViewPlugin {
         const definition = deepCopy(this.getters.getPivotCoreDefinition(pivotId));
         if (!(pivotId in this.pivots)) {
             const Pivot = withPivotPresentationLayer(pivotRegistry.get(definition.type).ui);
-            this.pivots[pivotId] = new Pivot(this.custom, { definition, getters: this.getters });
+            this.pivots[pivotId] = new Pivot(pivotId, this.custom, { definition, getters: this.getters });
         }
         else if (recreate) {
             this.pivots[pivotId].onDefinitionChange(definition);
@@ -74417,7 +74460,7 @@ class Session extends EventBus {
         }
         delete this.clients[this.clientId];
         this.transportService.leave(this.clientId);
-        this.transportService.sendMessage({
+        this.sendToTransport({
             type: "CLIENT_LEFT",
             clientId: this.clientId,
             version: MESSAGE_VERSION,
@@ -74431,7 +74474,7 @@ class Session extends EventBus {
             return;
         }
         const snapshotId = this.uuidGenerator.uuidv4();
-        await this.transportService.sendMessage({
+        await this.sendToTransport({
             type: "SNAPSHOT",
             nextRevisionId: snapshotId,
             serverRevisionId: this.serverRevisionId,
@@ -74479,10 +74522,14 @@ class Session extends EventBus {
         const type = currentPosition ? "CLIENT_MOVED" : "CLIENT_JOINED";
         const client = this.getCurrentClient();
         this.clients[this.clientId] = { ...client, position };
-        this.transportService.sendMessage({
+        this.sendToTransport({
             type,
             version: MESSAGE_VERSION,
             client: { ...client, position },
+        }).then(() => {
+            if (this.pendingMessages.length > 0 && !this.waitingAck) {
+                this.sendPendingMessage();
+            }
         });
     }
     /**
@@ -74563,7 +74610,7 @@ class Session extends EventBus {
             if (client) {
                 const { position } = client;
                 if (position) {
-                    this.transportService.sendMessage({
+                    this.sendToTransport({
                         type: "CLIENT_MOVED",
                         version: MESSAGE_VERSION,
                         client: { ...client, position },
@@ -74584,6 +74631,10 @@ class Session extends EventBus {
         }
         this.sendPendingMessage();
     }
+    async sendToTransport(message) {
+        // wrap in an async function to ensure it returns a promise
+        return this.transportService.sendMessage(message);
+    }
     /**
      * Send the next pending message
      */
@@ -74612,9 +74663,14 @@ class Session extends EventBus {
       ${JSON.stringify(message)}`);
         }
         this.waitingAck = true;
-        this.transportService.sendMessage({
+        this.sendToTransport({
             ...message,
             serverRevisionId: this.serverRevisionId,
+        }).catch((e) => {
+            if (!(e instanceof ClientDisconnectedError)) {
+                throw e.cause || e;
+            }
+            this.waitingAck = false;
         });
     }
     acknowledge(message) {
@@ -82120,10 +82176,8 @@ class SpreadsheetDashboard extends Component {
         });
     }
     get gridContainer() {
-        const sheetId = this.env.model.getters.getActiveSheetId();
-        const { right } = this.env.model.getters.getSheetZone(sheetId);
-        const { end } = this.env.model.getters.getColDimensions(sheetId, right);
-        return cssPropertiesToCss({ "max-width": `${end}px` });
+        const maxWidth = this.getMaxSheetWidth();
+        return cssPropertiesToCss({ "max-width": `${maxWidth}px` });
     }
     get gridOverlayDimensions() {
         return cssPropertiesToCss({
@@ -82155,10 +82209,12 @@ class SpreadsheetDashboard extends Component {
     onClosePopover() {
         this.cellPopovers.close();
     }
-    onGridResized({ height, width }) {
+    onGridResized() {
+        const { height, width } = this.props.getGridSize();
+        const maxWidth = this.getMaxSheetWidth();
         this.env.model.dispatch("RESIZE_SHEETVIEW", {
-            width: width,
-            height: height,
+            width: Math.min(maxWidth, width),
+            height,
             gridOffsetX: 0,
             gridOffsetY: 0,
         });
@@ -82176,6 +82232,11 @@ class SpreadsheetDashboard extends Component {
             ...this.env.model.getters.getSheetViewDimensionWithHeaders(),
         };
     }
+    getMaxSheetWidth() {
+        const sheetId = this.env.model.getters.getActiveSheetId();
+        const { right } = this.env.model.getters.getSheetZone(sheetId);
+        return this.env.model.getters.getColDimensions(sheetId, right).end;
+    }
 }
 
 css /* scss */ `
@@ -84417,22 +84478,20 @@ class Spreadsheet extends Component {
         return this.env.model.getters.getVisibleGroupLayers(sheetId, "COL");
     }
     getGridSize() {
-        const topBarHeight = this.spreadsheetRef.el
-            ?.querySelector(".o-spreadsheet-topbar-wrapper")
-            ?.getBoundingClientRect().height || 0;
-        const bottomBarHeight = this.spreadsheetRef.el
-            ?.querySelector(".o-spreadsheet-bottombar-wrapper")
-            ?.getBoundingClientRect().height || 0;
-        const gridWidth = this.spreadsheetRef.el?.querySelector(".o-grid")?.getBoundingClientRect().width || 0;
-        const gridHeight = (this.spreadsheetRef.el?.getBoundingClientRect().height || 0) -
-            (this.spreadsheetRef.el?.querySelector(".o-column-groups")?.getBoundingClientRect().height ||
-                0) -
-            topBarHeight -
-            bottomBarHeight;
-        return {
-            width: Math.max(gridWidth - SCROLLBAR_WIDTH, 0),
-            height: Math.max(gridHeight - SCROLLBAR_WIDTH, 0),
-        };
+        const el = this.spreadsheetRef.el;
+        if (!el) {
+            return { width: 0, height: 0 };
+        }
+        const getHeight = (selector) => el.querySelector(selector)?.getBoundingClientRect().height || 0;
+        const getWidth = (selector) => el.querySelector(selector)?.getBoundingClientRect().width || 0;
+        const rect = el.getBoundingClientRect();
+        const topBarHeight = getHeight(".o-spreadsheet-topbar-wrapper");
+        const bottomBarHeight = getHeight(".o-spreadsheet-bottombar-wrapper");
+        const colGroupHeight = getHeight(".o-column-groups");
+        const gridWidth = getWidth(".o-grid");
+        const width = Math.max(gridWidth - SCROLLBAR_WIDTH, 0);
+        const height = Math.max(rect.height - topBarHeight - bottomBarHeight - colGroupHeight - SCROLLBAR_WIDTH, 0);
+        return { width, height };
     }
 }
 
@@ -84445,7 +84504,7 @@ class ReadonlyTransportFilter {
         if (message.type === "CLIENT_JOINED" ||
             message.type === "CLIENT_LEFT" ||
             message.type === "CLIENT_MOVED") {
-            this.transportService.sendMessage(message);
+            await this.transportService.sendMessage(message);
         }
         // ignore all other messages
     }
@@ -89064,6 +89123,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.15";
-__info__.date = "2025-12-26T10:19:23.408Z";
-__info__.hash = "fe625c9";
+__info__.version = "19.0.17";
+__info__.date = "2026-01-14T10:01:24.044Z";
+__info__.hash = "2165bad";