@odoo/o-spreadsheet 19.3.10 → 19.3.12

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.3.10
- * @date 2026-07-01T05:04:42.999Z
- * @hash 040ae04
+ * @version 19.3.12
+ * @date 2026-07-20T11:02:44.416Z
+ * @hash 2b42707
  */
 
 import { App, Component, blockDom, markRaw, onMounted, onPatched, onWillPatch, onWillStart, onWillUnmount, onWillUpdateProps, status, toRaw, useChildSubEnv, useComponent, useEffect, useEnv, useExternalListener, useRef, useState, useSubEnv, whenReady, xml } from "@odoo/owl";
@@ -5567,7 +5567,7 @@ function getRangeString(range, forSheetId, getSheetName, options = {
 	if (prefixSheet && !sheetName) return CellErrorType.InvalidReference;
 	let rangeString = getRangePartString(range, 0, options);
 	if (range.parts && range.parts.length === 2) {
-		if (range.zone.top !== range.zone.bottom || range.zone.left !== range.zone.right || range.parts[0].rowFixed || range.parts[0].colFixed || range.parts[1].rowFixed || range.parts[1].colFixed) {
+		if (range.zone.top !== range.zone.bottom || range.zone.left !== range.zone.right || range.parts[0].rowFixed || range.parts[0].colFixed || range.parts[1].rowFixed || range.parts[1].colFixed || options.doNotSimplifyRange) {
 			rangeString += ":";
 			rangeString += getRangePartString(range, 1, options);
 		}
@@ -6289,6 +6289,11 @@ var CompiledFormula = class CompiledFormula {
 		const params = compileTokens(rangeTokenize(formula));
 		return new CompiledFormula(sheetId, params.tokens, params.literalValues, params.symbols, params.dependencies.map((xc) => getters.getRangeFromSheetXC(sheetId, xc)), params.isBadExpression, params.normalizedFormula, params.execute);
 	}
+	areRangeShapesInLineWithNormalizedFormula() {
+		return Array.from(this.normalizedFormula.matchAll(/\|C\||\|R\|/g)).every((value, index) => {
+			return this.rangeDependencies[index].parts.length === 1 && value[0] === "|C|" || this.rangeDependencies[index].parts.length === 2 && value[0] === "|R|" || this.rangeDependencies[index].invalidXc || this.rangeDependencies[index].invalidSheetName;
+		});
+	}
 };
 function compileTokens(tokens) {
 	try {
@@ -14140,7 +14145,7 @@ function filterInvalidCalendarDataPoints(labels, datasets) {
 */
 function filterInvalidHierarchicalPoints(values, hierarchy) {
 	const numberOfDataPoints = Math.max(values.length, ...hierarchy.map((dataset) => dataset.data?.length || 0));
-	const isEmpty = (value) => value === null || value === "";
+	const isEmpty = (value) => value === void 0 || value === null || value === "";
 	const dataPointsIndexes = range(0, numberOfDataPoints).filter((dataPointIndex) => {
 		const groups = hierarchy.map((dataset) => dataset.data?.[dataPointIndex]);
 		if (isEmpty(groups[0]?.value)) return false;
@@ -33914,7 +33919,7 @@ var ScorecardChartConfigPanel = class extends Component {
 		return !!this.state.keyValueDispatchResult?.isCancelledBecause("InvalidScorecardKeyValue");
 	}
 	get isBaselineInvalid() {
-		return !!this.state.keyValueDispatchResult?.isCancelledBecause("InvalidScorecardBaseline");
+		return !!this.state.baselineDispatchResult?.isCancelledBecause("InvalidScorecardBaseline");
 	}
 	onKeyValueRangeChanged(ranges) {
 		this.keyValue = ranges[0];
@@ -49717,6 +49722,7 @@ var Squisher = class {
 	alreadyAppliedNumberOffsets = [];
 	previousStrings = [];
 	baseFormulaWasTransformed = false;
+	rangeToStringOptions = { doNotSimplifyRange: true };
 	baseNumber = void 0;
 	constructor(getters) {
 		this.getters = getters;
@@ -49772,9 +49778,9 @@ var Squisher = class {
 			let references = [];
 			if (!this.baseFormula || this.baseFormula.normalizedFormula !== cell.compiledFormula.normalizedFormula) {
 				this.resetBaseTo(cell.compiledFormula);
-				return cell.compiledFormula.toFormulaString(this.getters);
+				return cell.compiledFormula.toFormulaString(this.getters, this.rangeToStringOptions);
 			} else {
-				if (!this.baseFormulaWasTransformed && deepEquals(cell.compiledFormula.literalValues, this.baseFormula.literalValues) && deepEquals(cell.compiledFormula.rangeDependencies, this.baseFormula.rangeDependencies)) return cell.compiledFormula.toFormulaString(this.getters);
+				if (!this.baseFormulaWasTransformed && deepEquals(cell.compiledFormula.literalValues, this.baseFormula.literalValues) && deepEquals(cell.compiledFormula.rangeDependencies, this.baseFormula.rangeDependencies)) return cell.compiledFormula.toFormulaString(this.getters, this.rangeToStringOptions);
 				numbers = this.squishNumbers(cell.compiledFormula.literalValues.numbers);
 				strings = this.squishStrings(cell.compiledFormula.literalValues.strings);
 				references = this.squishReferences(cell.compiledFormula.rangeDependencies, forSheetId);
@@ -49851,23 +49857,23 @@ var Squisher = class {
 	squishOneReference(reference, previousReferences, index, forSheetId) {
 		const previousReference = previousReferences[index];
 		if (deepEquals(previousReference, reference)) return "=";
-		if (previousReference.sheetId !== reference.sheetId || previousReference.prefixSheet !== reference.prefixSheet || previousReference.invalidSheetName !== reference.invalidSheetName || previousReference.invalidXc !== reference.invalidXc) {
+		if (previousReference.sheetId !== reference.sheetId || previousReference.prefixSheet !== reference.prefixSheet || previousReference.invalidSheetName !== reference.invalidSheetName || previousReference.invalidXc !== reference.invalidXc || previousReference.parts.length !== reference.parts.length) {
 			previousReferences[index] = deepCopy(reference);
-			return getRangeString(reference, forSheetId, this.getters.getSheetName);
+			return getRangeString(reference, forSheetId, this.getters.getSheetName, this.rangeToStringOptions);
 		}
 		if (previousReference.unboundedZone.bottom === void 0 || previousReference.unboundedZone.right === void 0 || reference.unboundedZone.bottom === void 0 || reference.unboundedZone.right === void 0) {
 			previousReferences[index] = deepCopy(reference);
-			return getRangeString(reference, forSheetId, this.getters.getSheetName);
+			return getRangeString(reference, forSheetId, this.getters.getSheetName, this.rangeToStringOptions);
 		}
 		for (let i = 0; i < reference.parts.length; i++) if (previousReference.parts[i].colFixed !== reference.parts[i].colFixed || previousReference.parts[i].rowFixed !== reference.parts[i].rowFixed) {
 			previousReferences[index] = deepCopy(reference);
-			return getRangeString(reference, forSheetId, this.getters.getSheetName);
+			return getRangeString(reference, forSheetId, this.getters.getSheetName, this.rangeToStringOptions);
 		}
 		const currentZone = reference.zone;
 		const previousZone = previousReference.zone;
 		if (currentZone.top !== currentZone.bottom || currentZone.left !== currentZone.right || previousZone.top !== previousZone.bottom || previousZone.left !== previousZone.right) {
 			previousReferences[index] = deepCopy(reference);
-			return getRangeString(reference, forSheetId, this.getters.getSheetName);
+			return getRangeString(reference, forSheetId, this.getters.getSheetName, this.rangeToStringOptions);
 		}
 		const diffCol = reference.zone.left - previousReference.zone.left;
 		const diffRow = reference.zone.top - previousReference.zone.top;
@@ -49875,7 +49881,7 @@ var Squisher = class {
 		previousReference.unboundedZone = deepCopy(reference.unboundedZone);
 		if (diffCol !== 0 && diffRow === 0) return `${diffCol > 0 ? "+" : "-"}C${Math.abs(diffCol)}`;
 		else if (diffRow !== 0 && diffCol === 0) return `${diffRow > 0 ? "+" : "-"}R${Math.abs(diffRow)}`;
-		return getRangeString(reference, forSheetId, this.getters.getSheetName);
+		return getRangeString(reference, forSheetId, this.getters.getSheetName, this.rangeToStringOptions);
 	}
 	/**
 	* Squish the number parameters. Result for each parameter should be compared the previous formula.
@@ -50084,10 +50090,19 @@ var Unsquisher = class {
 			case "FIRST_OFFSET": {
 				const currentOffset = current;
 				this.previousOffset = currentOffset;
-				for (const position of targetPositions) yield {
-					position,
-					compiled: this.unsquishFormula(currentOffset, sheetId, getters)
-				};
+				for (const position of targetPositions) {
+					const result = this.unsquishFormula(currentOffset, sheetId, getters);
+					if (!result.areRangeShapesInLineWithNormalizedFormula()) {
+						const formulaString = result.toFormulaString(getters);
+						yield {
+							position,
+							compiled: CompiledFormula.Compile(formulaString, sheetId, getters)
+						};
+					} else yield {
+						position,
+						compiled: result
+					};
+				}
 				break;
 			}
 			case "COMBINE_OFFSET": {
@@ -50096,10 +50111,19 @@ var Unsquisher = class {
 				this.previousOffset.N = currentOffset.N ?? this.previousOffset.N;
 				this.previousOffset.S = currentOffset.S ?? this.previousOffset.S;
 				this.previousOffset.R = currentOffset.R ?? this.previousOffset.R;
-				for (const position of targetPositions) yield {
-					position,
-					compiled: this.unsquishFormula(this.previousOffset, sheetId, getters)
-				};
+				for (const position of targetPositions) {
+					const result = this.unsquishFormula(this.previousOffset, sheetId, getters);
+					if (!result.areRangeShapesInLineWithNormalizedFormula()) {
+						const formulaString = result.toFormulaString(getters);
+						yield {
+							position,
+							compiled: CompiledFormula.Compile(formulaString, sheetId, getters)
+						};
+					} else yield {
+						position,
+						compiled: result
+					};
+				}
 				break;
 			}
 			case "NEW_NUMBER":
@@ -56941,13 +56965,13 @@ var EvaluationConditionalFormatPlugin = class extends CoreViewPlugin {
 		const minValue = this.parsePoint(sheetId, range, rule.minimum, "min");
 		const midValue = rule.midpoint ? this.parsePoint(sheetId, range, rule.midpoint) : null;
 		const maxValue = this.parsePoint(sheetId, range, rule.maximum, "max");
-		if (minValue === null || maxValue === null || minValue >= maxValue || midValue && (minValue >= midValue || midValue >= maxValue)) return;
+		if (minValue === null || maxValue === null || minValue >= maxValue || midValue !== null && (minValue >= midValue || midValue >= maxValue)) return;
 		const zone = this.getters.getRangeFromSheetXC(sheetId, range).zone;
 		const colorThresholds = [{
 			value: minValue,
 			color: rule.minimum.color
 		}];
-		if (rule.midpoint && midValue) colorThresholds.push({
+		if (rule.midpoint && midValue !== null) colorThresholds.push({
 			value: midValue,
 			color: rule.midpoint.color
 		});
@@ -57860,6 +57884,8 @@ var PivotUIPlugin = class extends CoreViewPlugin {
 	pivots = {};
 	unusedPivotsInFormulas;
 	custom;
+	pivotPositionCache = new PositionMap();
+	shouldInvalidateCache = false;
 	constructor(config) {
 		super(config);
 		this.custom = config.custom;
@@ -57870,7 +57896,11 @@ var PivotUIPlugin = class extends CoreViewPlugin {
 		}
 	}
 	handle(cmd) {
-		if (invalidateEvaluationCommands.has(cmd.type)) for (const pivotId of this.getters.getPivotIds()) this.setupPivot(pivotId, { recreate: true });
+		if (invalidateEvaluationCommands.has(cmd.type)) {
+			this.shouldInvalidateCache = true;
+			for (const pivotId of this.getters.getPivotIds()) this.setupPivot(pivotId, { recreate: true });
+		}
+		if (cmd.type === "UPDATE_CELL") this.shouldInvalidateCache = true;
 		switch (cmd.type) {
 			case "REFRESH_PIVOT":
 				this.refreshPivot(cmd.id);
@@ -57910,6 +57940,12 @@ var PivotUIPlugin = class extends CoreViewPlugin {
 				break;
 		}
 	}
+	finalize() {
+		if (this.shouldInvalidateCache) {
+			this.pivotPositionCache = new PositionMap();
+			this.shouldInvalidateCache = false;
+		}
+	}
 	/**
 	* Get the id of the first pivot in the formula at the given position. Returns undefined if there
 	* is no pivot at this position
@@ -57921,9 +57957,14 @@ var PivotUIPlugin = class extends CoreViewPlugin {
 	* Get all of the ids of the pivot present in the formula at the given position.
 	*/
 	getPivotIdsFromPosition(position) {
-		const cell = this.getters.getCorrespondingFormulaCell(position);
-		if (cell && cell.isFormula) return this.getPivotIdsFromFormula(position.sheetId, cell.compiledFormula);
-		return [];
+		if (!this.pivotPositionCache.has(position)) {
+			const cell = this.getters.getCorrespondingFormulaCell(position);
+			if (cell && cell.isFormula) {
+				const pivotIds = this.getPivotIdsFromFormula(position.sheetId, cell.compiledFormula);
+				this.pivotPositionCache.set(position, pivotIds);
+			}
+		}
+		return this.pivotPositionCache.get(position) || [];
 	}
 	getPivotIdsFromFormula(sheetId, formula) {
 		return this.getPivotFunctions(sheetId, formula).map((pivotFunction) => {
@@ -59678,7 +59719,12 @@ var TableComputedStylePlugin = class extends UIPlugin {
 	computeTableStyle(sheetId, table) {
 		return lazy(() => {
 			const style = this.getters.getTableStyle(table.config.styleId);
-			const { tableMetaData, config } = this.getTableMetaData(sheetId, table);
+			const tableMetaDataAndConfig = this.getTableMetaData(sheetId, table);
+			if (!tableMetaDataAndConfig) return {
+				borders: {},
+				styles: {}
+			};
+			const { tableMetaData, config } = tableMetaDataAndConfig;
 			const relativeTableStyle = getComputedTableStyle(config, style, tableMetaData);
 			const mapping = this.getTableMapping(sheetId, table);
 			const absoluteTableStyle = {
@@ -59716,6 +59762,7 @@ var TableComputedStylePlugin = class extends UIPlugin {
 		const pivotInfo = this.getters.getPivotStyleAtPosition(mainPosition);
 		if (!pivotInfo) throw new Error("No dynamic pivot info found at pivot table position");
 		const pivot = this.getters.getPivot(pivotInfo.pivotId);
+		if (!pivot.isValid()) return;
 		const pivotStyle = pivotInfo.pivotStyle;
 		const maxRowDepth = pivot.getExpandedTableStructure().getNumberOfRowGroupBys();
 		const pivotTable = pivot.getCollapsedTableStructure();
@@ -64260,11 +64307,23 @@ var GridSelectionPlugin = class extends UIPlugin {
 		ctx.strokeStyle = SELECTION_BORDER_COLOR;
 		ctx.lineWidth = 1.5 * thinLineWidth;
 		for (const zone of zones) {
+			if (!viewports.isZoneVisibleInViewport(sheetId, zone)) continue;
 			const { x, y, width, height } = viewports.getVisibleRect(sheetId, zone);
 			ctx.globalCompositeOperation = "multiply";
-			ctx.fillRect(x, y, width, height);
-			ctx.globalCompositeOperation = "source-over";
-			ctx.strokeRect(x, y, width, height);
+			const currentLineWidth = ctx.lineWidth;
+			if (height === 0 || width === 0) ctx.lineWidth = 3 * thinLineWidth;
+			if (height === 0 && width === 0) {
+				ctx.beginPath();
+				ctx.arc(x, y, 3, 0, 2 * Math.PI);
+				ctx.fill();
+				ctx.globalCompositeOperation = "source-over";
+				ctx.stroke();
+			} else {
+				ctx.fillRect(x, y, width, height);
+				ctx.globalCompositeOperation = "source-over";
+				ctx.strokeRect(x, y, width, height);
+			}
+			ctx.lineWidth = currentLineWidth;
 		}
 		ctx.globalCompositeOperation = "source-over";
 		const position = renderingContext.activePosition;
@@ -64411,6 +64470,9 @@ var InternalViewport = class {
 		this.adjustViewportZoneX();
 		this.adjustViewportZoneY();
 	}
+	isZoneVisible(zone) {
+		return intersection(zone, this) !== void 0;
+	}
 	/**
 	*
 	* Computes the visible coordinates & dimensions of a given zone inside the viewport
@@ -64691,6 +64753,9 @@ var ViewportCollection = class {
 	isVisibleInViewport({ sheetId, col, row }) {
 		return this.getSubViewports(sheetId).some((pane) => pane.isVisible(col, row));
 	}
+	isZoneVisibleInViewport(sheetId, zone) {
+		return this.getSubViewports(sheetId).some((pane) => pane.isZoneVisible(zone));
+	}
 	getScrollBarWidth() {
 		return 15 / this.zoomLevel;
 	}
@@ -69222,18 +69287,28 @@ var NamedRangeSelector = class extends Component {
 			return;
 		}
 		const activeSheetId = this.env.model.getters.getActiveSheetId();
-		if (activeSheetId !== sheetId) this.env.model.dispatch("ACTIVATE_SHEET", {
-			sheetIdFrom: activeSheetId,
-			sheetIdTo: sheetId
-		});
-		this.env.model.selection.selectCell(zone.right, zone.bottom);
+		if (activeSheetId !== sheetId) {
+			if (!this.env.model.getters.getSheet(sheetId).isVisible) {
+				this.env.notifyUser({
+					text: _t("The sheet on which the range is defined is hidden."),
+					type: "info",
+					sticky: false
+				});
+				return;
+			}
+			this.env.model.dispatch("ACTIVATE_SHEET", {
+				sheetIdFrom: activeSheetId,
+				sheetIdTo: sheetId
+			});
+		}
+		this.env.model.selection.selectCell(zone.right, zone.bottom, { allowsHiddenSelection: true });
 		this.env.model.selection.selectZone({
 			cell: {
 				col: zone.left,
 				row: zone.top
 			},
 			zone
-		});
+		}, { allowsHiddenSelection: true });
 	}
 	get selectionKey() {
 		return `${this.env.model.getters.getActiveSheetId()}-${zoneToXc(this.selectedZone)}`;
@@ -71642,7 +71717,7 @@ var SelectionStreamProcessorImpl = class {
 	/**
 	* Select a single cell as the new anchor.
 	*/
-	selectCell(col, row) {
+	selectCell(col, row, options) {
 		const zone = positionToZone({
 			col,
 			row
@@ -71653,7 +71728,10 @@ var SelectionStreamProcessorImpl = class {
 				col,
 				row
 			}
-		}, { scrollIntoView: true });
+		}, {
+			scrollIntoView: true,
+			...options
+		});
 	}
 	/**
 	* Set the selection to one of the cells adjacent to the current anchor cell.
@@ -72008,15 +72086,21 @@ var SelectionStreamProcessorImpl = class {
 		return DispatchResult.Success;
 	}
 	checkEventAnchorZone(event) {
-		return this.checkAnchorZone(event.anchor);
+		return this.checkAnchorZone(event.anchor, event.options);
 	}
-	checkAnchorZone(anchor) {
+	checkAnchorZone(anchor, options) {
 		const { cell, zone } = anchor;
 		if (!isInside(cell.col, cell.row, zone)) return "InvalidAnchorZone";
 		const { left, right, top, bottom } = zone;
 		const sheetId = this.getters.getActiveSheetId();
-		const refCol = this.getters.findVisibleHeader(sheetId, "COL", left, right);
-		if (this.getters.findVisibleHeader(sheetId, "ROW", top, bottom) === void 0 || refCol === void 0) return "SelectionOutOfBound";
+		if (!options?.allowsHiddenSelection) {
+			const refCol = this.getters.findVisibleHeader(sheetId, "COL", left, right);
+			if (this.getters.findVisibleHeader(sheetId, "ROW", top, bottom) === void 0 || refCol === void 0) return "SelectionOutOfBound";
+		} else {
+			const numberCols = this.getters.getNumberCols(sheetId);
+			const numberRows = this.getters.getNumberRows(sheetId);
+			if (left < 0 || right > numberCols - 1 || top < 0 || bottom > numberRows - 1) return "SelectionOutOfBound";
+		}
 		return "Success";
 	}
 	checkAnchorZoneOrThrow(anchor) {
@@ -84387,7 +84471,8 @@ const helpers = {
 	parseFormat,
 	isFormula,
 	domainToColRowDomain,
-	collapseHierarchicalDisplayName
+	collapseHierarchicalDisplayName,
+	PositionMap
 };
 const links = {
 	isMarkdownLink,
@@ -84509,6 +84594,6 @@ const chartHelpers = {
 //#endregion
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, ClientDisconnectedError, CommandResult, CompiledFormula, CorePlugin, CoreViewPlugin, DEFAULT_LOCALE, DEFAULT_LOCALES, DispatchResult, EvaluationError, LocalTransportService, Model, PivotRuntimeDefinition, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, SpreadsheetPivotTable, UIPlugin, __info__, addFunction, addRenderingLayer, astToFormula, canExecuteInReadonly, categories, chartHelpers, components, constants, convertAstNodes, coreTypes, createAutocompleteArgumentsProvider, findCellInNewZone, functionCache, getCaretDownSvg, getCaretUpSvg, helpers, hooks, invalidateCFEvaluationCommands, invalidateChartEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, isCoreCommand, isSheetDependent, iterateAstNodes, links, load, lockedSheetAllowedCommands, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, stores, tokenColors, tokenize };
 
-__info__.version = "19.3.10";
-__info__.date = "2026-07-01T05:04:42.999Z";
-__info__.hash = "040ae04";
+__info__.version = "19.3.12";
+__info__.date = "2026-07-20T11:02:44.416Z";
+__info__.hash = "2b42707";