@odoo/o-spreadsheet 19.3.11 → 19.3.13

package/dist/o_spreadsheet.iife.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.11
- * @date 2026-07-13T06:26:28.422Z
- * @hash 35c06d3
+ * @version 19.3.13
+ * @date 2026-07-23T07:41:06.182Z
+ * @hash 2059bd0
  */
 
 (function(exports, _odoo_owl) {
@@ -5569,7 +5569,7 @@ stores.inject(MyMetaStore, storeInstance);
 		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);
 			}
@@ -6291,6 +6291,11 @@ stores.inject(MyMetaStore, storeInstance);
 			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 {
@@ -33916,7 +33921,7 @@ stores.inject(MyMetaStore, storeInstance);
 			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];
@@ -44152,7 +44157,8 @@ stores.inject(MyMetaStore, storeInstance);
 			const { x, y } = this.state.handler ? this.state.position : this.props.position;
 			return cssPropertiesToCss({
 				top: `${y}px`,
-				left: `${x}px`
+				left: `${x}px`,
+				visibility: this.props.isVisible ? "visible" : "hidden"
 			});
 		}
 		get styleNextValue() {
@@ -44573,7 +44579,7 @@ stores.inject(MyMetaStore, storeInstance);
 						this._cancelEdition();
 						this.resetContent();
 					}
-					if (cmd.sheetIdFrom !== cmd.sheetIdTo) {
+					if (this.model.selection.isListening(this) && cmd.sheetIdFrom !== cmd.sheetIdTo) {
 						const activePosition = this.getters.getActivePosition();
 						const { col, row } = this.getters.getNextVisibleCellPosition({
 							sheetId: cmd.sheetIdTo,
@@ -48715,6 +48721,10 @@ stores.inject(MyMetaStore, storeInstance);
 		get displaySelectionHandler() {
 			return this.env.isMobile() && this.composerFocusStore.activeComposer.editionMode === "inactive";
 		}
+		get clientsToDisplay() {
+			const sheetId = this.env.model.getters.getActiveSheetId();
+			return this.env.model.getters.getClientsToDisplay(sheetId);
+		}
 	};
 
 //#endregion
@@ -49719,6 +49729,7 @@ stores.inject(MyMetaStore, storeInstance);
 		alreadyAppliedNumberOffsets = [];
 		previousStrings = [];
 		baseFormulaWasTransformed = false;
+		rangeToStringOptions = { doNotSimplifyRange: true };
 		baseNumber = void 0;
 		constructor(getters) {
 			this.getters = getters;
@@ -49774,9 +49785,9 @@ stores.inject(MyMetaStore, storeInstance);
 				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);
@@ -49853,23 +49864,23 @@ stores.inject(MyMetaStore, storeInstance);
 		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;
@@ -49877,7 +49888,7 @@ stores.inject(MyMetaStore, storeInstance);
 			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.
@@ -50086,10 +50097,19 @@ stores.inject(MyMetaStore, storeInstance);
 				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": {
@@ -50098,10 +50118,19 @@ stores.inject(MyMetaStore, storeInstance);
 					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":
@@ -57862,6 +57891,8 @@ stores.inject(MyMetaStore, storeInstance);
 		pivots = {};
 		unusedPivotsInFormulas;
 		custom;
+		pivotPositionCache = new PositionMap();
+		shouldInvalidateCache = false;
 		constructor(config) {
 			super(config);
 			this.custom = config.custom;
@@ -57872,7 +57903,11 @@ stores.inject(MyMetaStore, storeInstance);
 			}
 		}
 		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);
@@ -57912,6 +57947,12 @@ stores.inject(MyMetaStore, storeInstance);
 					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
@@ -57923,9 +57964,14 @@ stores.inject(MyMetaStore, storeInstance);
 		* 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) => {
@@ -59670,17 +59716,32 @@ stores.inject(MyMetaStore, storeInstance);
 		getCellTableStyle(position) {
 			const table = this.getters.getTable(position);
 			if (!table) return;
-			return this.tableStyles[position.sheetId][table.id]().styles[position.col]?.[position.row];
+			try {
+				return this.tableStyles[position.sheetId][table.id]().styles[position.col]?.[position.row];
+			} catch (e) {
+				if (isEvaluationError(e) || e instanceof EvaluationError) return;
+				throw e;
+			}
 		}
 		getCellTableBorder(position) {
 			const table = this.getters.getTable(position);
 			if (!table) return;
-			return this.tableStyles[position.sheetId][table.id]().borders[position.col]?.[position.row];
+			try {
+				return this.tableStyles[position.sheetId][table.id]().borders[position.col]?.[position.row];
+			} catch (e) {
+				if (isEvaluationError(e) || e instanceof EvaluationError) return;
+				throw e;
+			}
 		}
 		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 = {
@@ -59718,6 +59779,7 @@ stores.inject(MyMetaStore, storeInstance);
 			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();
@@ -61012,14 +61074,13 @@ stores.inject(MyMetaStore, storeInstance);
 		* Get the list of others connected clients which are present in the same sheet
 		* and with a valid position
 		*/
-		getClientsToDisplay() {
+		getClientsToDisplay(sheetId) {
 			try {
 				this.getters.getCurrentClient();
 			} catch (e) {
 				if (e instanceof ClientDisconnectedError) return [];
 				else throw e;
 			}
-			const sheetId = this.getters.getActiveSheetId();
 			const clients = [];
 			for (const client of this.getters.getConnectedClients()) if (client.id !== this.getters.getCurrentClient().id && client.position && client.position.sheetId === sheetId && this.isPositionValid(client.position)) clients.push({
 				...client,
@@ -61030,7 +61091,7 @@ stores.inject(MyMetaStore, storeInstance);
 		drawLayer(renderingContext) {
 			if (this.getters.isDashboard()) return;
 			const { ctx, thinLineWidth, viewports, sheetId } = renderingContext;
-			for (const client of this.getClientsToDisplay()) {
+			for (const client of this.getClientsToDisplay(sheetId)) {
 				const { row, col } = client.position;
 				const zone = this.getters.expandZone(sheetId, {
 					top: row,
@@ -61333,8 +61394,8 @@ stores.inject(MyMetaStore, storeInstance);
 		getNextVisibleCellPosition({ sheetId, col, row }) {
 			return {
 				sheetId,
-				col: this.findVisibleHeader(sheetId, "COL", col, this.getters.getNumberCols(sheetId) - 1),
-				row: this.findVisibleHeader(sheetId, "ROW", row, this.getters.getNumberRows(sheetId) - 1)
+				col: this.findVisibleHeader(sheetId, "COL", col, this.getters.getNumberCols(sheetId) - 1) || this.findVisibleHeader(sheetId, "COL", col, 0) || 0,
+				row: this.findVisibleHeader(sheetId, "ROW", row, this.getters.getNumberRows(sheetId) - 1) || this.findVisibleHeader(sheetId, "ROW", row, 0) || 0
 			};
 		}
 		/**
@@ -84426,7 +84487,8 @@ stores.inject(MyMetaStore, storeInstance);
 		parseFormat,
 		isFormula,
 		domainToColRowDomain,
-		collapseHierarchicalDisplayName
+		collapseHierarchicalDisplayName,
+		PositionMap
 	};
 	const links = {
 		isMarkdownLink,
@@ -84606,8 +84668,8 @@ exports.stores = stores;
 exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
-__info__.version = "19.3.11";
-__info__.date = "2026-07-13T06:26:28.422Z";
-__info__.hash = "35c06d3";
+__info__.version = "19.3.13";
+__info__.date = "2026-07-23T07:41:06.182Z";
+__info__.hash = "2059bd0";
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);