@odoo/o-spreadsheet 19.3.17 → 19.3.19

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.17
- * @date 2026-08-21T15:31:50.940Z
- * @hash cf8c614
+ * @version 19.3.19
+ * @date 2026-09-04T07:38:14.356Z
+ * @hash c8c526d
  */
 
 (function(exports, _odoo_owl) {
@@ -9594,16 +9594,16 @@ stores.inject(MyMetaStore, storeInstance);
 					drawLineOrBarOrRadarChartValues(chart, options, ctx);
 					break;
 				case "bar":
-					options.horizontal ? drawHorizontalBarChartValues(chart, options, ctx) : drawLineOrBarOrRadarChartValues(chart, options, ctx);
+					options.horizontal ? drawHorizontalBarChartValues(chart, options, ctx, { offsetFromAxisOrigin: true }) : drawLineOrBarOrRadarChartValues(chart, options, ctx);
 					break;
 				case "pyramid":
-					drawHorizontalBarChartValues(chart, options, ctx);
+					drawHorizontalBarChartValues(chart, options, ctx, { offsetFromAxisOrigin: true });
 					break;
 				case "calendar":
 					drawBarChartValues(chart, options, ctx);
 					break;
 				case "funnel":
-					drawHorizontalBarChartValues(chart, options, ctx);
+					drawHorizontalBarChartValues(chart, options, ctx, { offsetFromAxisOrigin: false });
 					break;
 			}
 			ctx.restore();
@@ -9673,7 +9673,7 @@ stores.inject(MyMetaStore, storeInstance);
 			}
 		}
 	}
-	function drawHorizontalBarChartValues(chart, options, ctx) {
+	function drawHorizontalBarChartValues(chart, options, ctx, args) {
 		const textsPositions = {};
 		for (const dataset of chart._metasets) {
 			if (isTrendLineAxis(dataset.xAxisID)) return;
@@ -9688,7 +9688,7 @@ stores.inject(MyMetaStore, storeInstance);
 				const distanceFromAxisOrigin = Math.abs(point.x - xZeroLine);
 				const PADDING = 3;
 				let xPosition;
-				if (distanceFromAxisOrigin < textWidth) xPosition = value < 0 ? xZeroLine - textWidth / 2 - PADDING : xZeroLine + textWidth / 2 + PADDING;
+				if (args.offsetFromAxisOrigin && distanceFromAxisOrigin < textWidth) xPosition = value < 0 ? xZeroLine - textWidth / 2 - PADDING : xZeroLine + textWidth / 2 + PADDING;
 				else xPosition = value < 0 ? point.x + point.width / 2 : point.x - point.width / 2;
 				if (!textsPositions[yPosition]) textsPositions[yPosition] = [];
 				for (const otherPosition of textsPositions[yPosition]) if (Math.abs(otherPosition - xPosition) < textWidth) xPosition = value < 0 ? otherPosition - textWidth - PADDING : otherPosition + textWidth + PADDING;
@@ -10487,6 +10487,7 @@ stores.inject(MyMetaStore, storeInstance);
 				hidden,
 				borderColor,
 				backgroundColor: borderColor,
+				pointBackgroundColor: borderColor,
 				pointRadius: definition.hideDataMarkers ? 0 : 3
 			};
 			if (fill) {
@@ -25446,9 +25447,10 @@ stores.inject(MyMetaStore, storeInstance);
 		compositionActive = false;
 		spreadsheetRect = useSpreadsheetRect();
 		lastHoveredTokenIndex = void 0;
-		debouncedHover = debounce((tokenIndex, hoveredRect) => {
+		debouncedHover = debounce((tokenIndex, composerContent, hoveredRect) => {
 			const selection = this.contentHelper.getCurrentSelection();
-			if (selection.start !== selection.end) return;
+			const currentContent = this.props.composerStore.currentContent;
+			if (selection.start !== selection.end || currentContent !== composerContent) return;
 			const currentHoveredContext = this.props.composerStore.hoveredTokens;
 			this.props.composerStore.hoverToken(tokenIndex);
 			if (!deepEquals(currentHoveredContext, this.props.composerStore.hoveredTokens)) this.composerState.hoveredRect = hoveredRect;
@@ -25766,7 +25768,8 @@ stores.inject(MyMetaStore, storeInstance);
 		onTokenHover(tokenIndex, hoveredRect) {
 			if (this.lastHoveredTokenIndex !== tokenIndex) {
 				this.lastHoveredTokenIndex = tokenIndex;
-				this.debouncedHover(tokenIndex, hoveredRect);
+				const composerContent = this.props.composerStore.currentContent;
+				this.debouncedHover(tokenIndex, composerContent, hoveredRect);
 			}
 		}
 		/**
@@ -40077,7 +40080,13 @@ stores.inject(MyMetaStore, storeInstance);
 		}
 		onSelectionChanged(ranges) {
 			this.state.rangeHasChanged = true;
-			this.state.range = ranges[0];
+			if (ranges.length === 0) {
+				this.state.range = void 0;
+				return;
+			}
+			const sheetId = this.env.model.getters.getActiveSheetId();
+			const range = this.env.model.getters.getRangeFromSheetXC(sheetId, ranges[0]);
+			this.state.range = this.env.model.getters.getRangeString(range, "forceSheetReference", { useBoundedReference: true });
 		}
 		onSelectionConfirmed() {
 			if (this.state.range) {
@@ -49736,6 +49745,7 @@ stores.inject(MyMetaStore, storeInstance);
 					return "Success";
 				case "UPDATE_CAROUSEL":
 					if (!this.carousels[cmd.sheetId]?.[cmd.figureId]) return "InvalidFigureId";
+					for (const item of cmd.definition.items) if (item.type === "chart" && !this.getters.getChart(item.chartId)) return "ChartDoesNotExist";
 					return "Success";
 			}
 			return "Success";
@@ -49840,6 +49850,7 @@ stores.inject(MyMetaStore, storeInstance);
 		baseFormulaWasTransformed = false;
 		rangeToStringOptions = { doNotSimplifyRange: true };
 		baseNumber = void 0;
+		baseNumberFormat = void 0;
 		constructor(getters) {
 			this.getters = getters;
 		}
@@ -49874,6 +49885,11 @@ stores.inject(MyMetaStore, storeInstance);
 				this.baseFormulaWasTransformed = false;
 			}
 		}
+		/** Reset the base number literal chain, so the next literal number cannot be squished as an offset of it */
+		resetBaseNumber() {
+			this.baseNumber = void 0;
+			this.baseNumberFormat = void 0;
+		}
 		/**
 		* Takes a cell and squish their formulas against the previous one (in the previous call).
 		* We should call this method for each cell in the sheet, in order from top to bottom, then from left to right (all cells of a columns, for each columns left to right).
@@ -49912,8 +49928,9 @@ stores.inject(MyMetaStore, storeInstance);
 			if (typeof cell.parsedValue === "number" && cell.parsedValue % 1 === 0) {
 				this.resetBaseFormula();
 				const numberValue = cell.parsedValue;
-				if (this.baseNumber === void 0) {
+				if (this.baseNumber === void 0 || cell.format !== this.baseNumberFormat) {
 					this.baseNumber = numberValue;
+					this.baseNumberFormat = cell.format;
 					return cell.content;
 				}
 				const numberOffset = numberValue - this.baseNumber;
@@ -49924,7 +49941,7 @@ stores.inject(MyMetaStore, storeInstance);
 				}
 			}
 			this.resetBaseFormula();
-			this.baseNumber = void 0;
+			this.resetBaseNumber();
 			return cell.content;
 		}
 		squishCommand(command) {
@@ -49933,6 +49950,8 @@ stores.inject(MyMetaStore, storeInstance);
 				const squished = this.squish(cell, command.sheetId);
 				return typeof squished === "string" ? command.content : squished;
 			}
+			this.resetBaseFormula();
+			this.resetBaseNumber();
 			return command.content;
 		}
 		/**
@@ -50086,8 +50105,10 @@ stores.inject(MyMetaStore, storeInstance);
 					continue;
 				}
 				const current = command.content;
-				if (current === void 0 || current === null || current === "") strategy = "NOT_A_FORMULA";
-				else strategy = this.chooseStrategy(current, strategy, command.sheetId, getters);
+				if (current === void 0 || current === null || current === "") {
+					this.rebase();
+					strategy = "NOT_A_FORMULA";
+				} else strategy = this.chooseStrategy(current, strategy, command.sheetId, getters);
 				let targetPositions;
 				if (command.type === "SQUISHED_UPDATE_CELL" && "targetRange" in command) {
 					const [start, end] = command.targetRange.split(":");
@@ -50132,8 +50153,11 @@ stores.inject(MyMetaStore, storeInstance);
 			let strategy;
 			for (const { key } of keys) {
 				const current = squished[key];
-				if (current === void 0 || current === null || current === "") continue;
-				else strategy = this.chooseStrategy(current, strategy, sheetId, getters);
+				if (current === void 0 || current === null || current === "") {
+					this.rebase();
+					strategy = void 0;
+					continue;
+				} else strategy = this.chooseStrategy(current, strategy, sheetId, getters);
 				const parts = key.split(":");
 				const targetPositions = parts.length === 1 ? expandXc(key) : expandRange(parts[0], parts[1]);
 				yield* this.applyStrategy(strategy, targetPositions, current, sheetId, getters);
@@ -50179,6 +50203,9 @@ stores.inject(MyMetaStore, storeInstance);
 					case "FIRST_OFFSET":
 						strategy = "COMBINE_OFFSET";
 						break;
+					case "COMBINE_OFFSET":
+					case "OFFSET_NUMBER": break;
+					default: throw new Error(`Cannot unsquish an offset without a preceding base cell: ${strategy}`);
 				}
 			}
 			return strategy;
@@ -54897,6 +54924,7 @@ stores.inject(MyMetaStore, storeInstance);
 		}
 		getCellsDependingOn(ranges, visited = new RangeSet()) {
 			visited = visited.copy();
+			let initialRangesToRemove = visited.copy();
 			const queue = Array.from(ranges).reverse();
 			while (queue.length > 0) {
 				const range = queue.pop();
@@ -54912,8 +54940,9 @@ stores.inject(MyMetaStore, storeInstance);
 					for (const interval of overlappingIntervals) impactedRanges.add(interval.dependents);
 				}
 				queue.push(...impactedRanges.difference(visited));
+				initialRangesToRemove = initialRangesToRemove.difference(impactedRanges);
 			}
-			for (const range of ranges) visited.delete(range);
+			for (const range of initialRangesToRemove) visited.delete(range);
 			return visited;
 		}
 		getOrCreateIntervalTree(sheetId, col) {
@@ -60777,6 +60806,7 @@ stores.inject(MyMetaStore, storeInstance);
 		transportService;
 		serverRevisionId;
 		commandSquisher;
+		shouldVerifySquish;
 		/**
 		* Positions of the others client.
 		*/
@@ -60811,13 +60841,15 @@ stores.inject(MyMetaStore, storeInstance);
 		* between all connected clients
 		* @param serverRevisionId
 		* @param commandSquisher used to squish and unsquish commands to reduce the size of messages sent to the server
+		* @param shouldVerifySquish either the commands should be unsquished and verified before being sent to the server
 		*/
-		constructor(revisions, transportService, serverRevisionId = DEFAULT_REVISION_ID, commandSquisher) {
+		constructor(revisions, transportService, serverRevisionId = DEFAULT_REVISION_ID, commandSquisher, shouldVerifySquish = true) {
 			super();
 			this.revisions = revisions;
 			this.transportService = transportService;
 			this.serverRevisionId = serverRevisionId;
 			this.commandSquisher = commandSquisher;
+			this.shouldVerifySquish = shouldVerifySquish;
 		}
 		canApplyOptimisticUpdate() {
 			return !this.waitingUndoRedoAck;
@@ -61073,10 +61105,20 @@ stores.inject(MyMetaStore, storeInstance);
 					this.revisions.rebase(revision.id);
 					revision = this.revisions.get(message.nextRevisionId);
 				}
-				message = {
+				const squishedCommands = this.commandSquisher.squish(revision.commands);
+				if (!squishedCommands.some((x) => x.type === "SQUISHED_UPDATE_CELL")) message = {
 					...message,
 					commands: revision.commands
 				};
+				else if (this.shouldVerifySquish && !deepEquals(revision.commands, this.commandSquisher.unsquish(squishedCommands))) message = {
+					...message,
+					commands: revision.commands,
+					squishedFailed: true
+				};
+				else message = {
+					...message,
+					commands: squishedCommands
+				};
 			}
 			if (this.isReplayingInitialRevisions) throw new Error(`Trying to send a new revision while replaying initial revision. This can lead to endless dispatches every time the spreadsheet is open.
       ${JSON.stringify(message)}`);
@@ -68773,7 +68815,7 @@ stores.inject(MyMetaStore, storeInstance);
 				if (!this.getters.isDashboard()) {
 					ctx.strokeStyle = GRAY_400;
 					ctx.lineWidth = 1;
-					ctx.strokeRect(x, y, figure.width, figure.height);
+					ctx.strokeRect(x + .5, y + .5, figure.width - 1, figure.height - 1);
 				}
 			}
 		}
@@ -84788,8 +84830,8 @@ exports.stores = stores;
 exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
-__info__.version = "19.3.17";
-__info__.date = "2026-08-21T15:31:50.940Z";
-__info__.hash = "cf8c614";
+__info__.version = "19.3.19";
+__info__.date = "2026-09-04T07:38:14.356Z";
+__info__.hash = "c8c526d";
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);