@odoo/o-spreadsheet 19.4.6 → 19.4.8

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.4.6
- * @date 2026-08-10T12:33:32.728Z
- * @hash 5c667fe
+ * @version 19.4.8
+ * @date 2026-08-19T09:25:13.754Z
+ * @hash 9dd828e
  */
 
 (function(exports, _odoo_owl) {
@@ -4027,6 +4027,18 @@ set(value) {
 			default: return "";
 		}
 	}
+	/**
+	* Only converts the decimal separator, ignore number format such as % or dates
+	*/
+	function toLocaleString(data, locale) {
+		const value = toValue(data);
+		switch (typeof value) {
+			case "string": return value;
+			case "number": return value.toString().replace(".", locale.decimalSeparator);
+			case "boolean": return value ? "TRUE" : "FALSE";
+			default: return "";
+		}
+	}
 	/** Normalize string by setting it to lowercase and replacing accent letters with plain letters */
 	const normalizeString = memoize(function normalizeString(str) {
 		return str.toLowerCase().normalize("NFD").replace(/[\u0300-\u036f]/g, "");
@@ -5959,6 +5971,9 @@ set(value) {
 		}
 	};
 	const functionRegistry = new FunctionRegistry();
+	/** Formulas using one of these functions are never squished: they are always exported
+	* as a full formula string, and they don't become the base formula of the next cells. */
+	const NonSquishableFunctionRegistry = new Registry();
 
 //#endregion
 //#region src/helpers/references.ts
@@ -15477,18 +15492,21 @@ set(value) {
 	figureRegistry.add("chart", {
 		Component: ChartFigure,
 		SidePanelComponent: "ChartPanel",
-		menuBuilder: getChartMenuActions
+		menuBuilder: getChartMenuActions,
+		isThemeDependant: true
 	});
 	figureRegistry.add("image", {
 		Component: ImageFigure,
 		keepRatio: true,
 		minFigSize: 20,
 		borderWidth: 0,
-		menuBuilder: getImageMenuActions
+		menuBuilder: getImageMenuActions,
+		isThemeDependant: false
 	});
 	figureRegistry.add("carousel", {
 		Component: CarouselFigure,
-		menuBuilder: getCarouselMenuActions
+		menuBuilder: getCarouselMenuActions,
+		isThemeDependant: true
 	});
 
 //#endregion
@@ -15517,6 +15535,9 @@ set(value) {
 		get isSelected() {
 			return this.env.model.getters.getSelectedFigureIds().includes(this.props.figureUI.id);
 		}
+		get isThemeDependant() {
+			return figureRegistry.get(this.props.figureUI.tag).isThemeDependant;
+		}
 		get figureRegistry() {
 			return figureRegistry;
 		}
@@ -51151,7 +51172,8 @@ set(value) {
 		* The result of this method is:
 		* - if the cell is not a formula, returns the content as is and resets the base formula
 		* - if the cell is a formula:
-		*   - if there is no previous formula, or the normalized formula is different from the previous one, resets the base formula to this one and returns the full formula string
+		*   - if there is no previous formula, or the normalized formula is different from the previous one or it contains blacklisted functions (see NonSquishableFunctionRegistry),
+		*        resets the base formula to this one and returns the full formula string
 		*   - else, compares the literal values and range dependencies to the previous formula, and for each parameter:
 		*     - for numbers: returns a relative change (+N or -N) if possible, else the full number or "=" if unchanged
 		*     - for strings: returns the full string if changed, else "="
@@ -51159,6 +51181,10 @@ set(value) {
 		* */
 		squish(cell, forSheetId) {
 			if (cell.isFormula) {
+				if (NonSquishableFunctionRegistry.getAll().some((functionName) => cell.compiledFormula.usesSymbol(functionName))) {
+					this.resetBaseFormula();
+					return cell.compiledFormula.toFormulaString(this.getters, this.rangeToStringOptions);
+				}
 				let numbers = [];
 				let strings = [];
 				let references = [];
@@ -62419,7 +62445,7 @@ set(value) {
 				}
 				message = {
 					...message,
-					commands: this.commandSquisher.squish(revision.commands)
+					commands: revision.commands
 				};
 			}
 			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.
@@ -77835,12 +77861,13 @@ set(value) {
 		}
 		sortFilterZone(sortDirection) {
 			const filterPosition = this.props.filterPosition;
-			const tableZone = this.table?.range.zone;
+			const table = this.table;
+			const tableZone = table?.range.zone;
 			if (!filterPosition || !tableZone || tableZone.top === tableZone.bottom) return;
 			const sheetId = this.env.model.getters.getActiveSheetId();
 			const contentZone = {
 				...tableZone,
-				top: tableZone.top + 1
+				top: tableZone.top + table.config.numberOfHeaders
 			};
 			const sortAnchor = {
 				col: filterPosition.col,
@@ -80097,7 +80124,7 @@ set(value) {
 			else if (_format === 0) {
 				const rowSeparator = this.locale.decimalSeparator === "," ? "/" : ",";
 				return transposeMatrix(_array).flatMap((row) => row.map((value) => {
-					return isEvaluationError(value.value) ? value.value : toString(value);
+					return isEvaluationError(value.value) ? value.value : toLocaleString(value, this.locale);
 				})).join(rowSeparator);
 			} else return new EvaluationError(_t("Format must be 0 or 1"));
 		},
@@ -85863,7 +85890,7 @@ set(value) {
 		description: _t("Concatenation of two values."),
 		args: [arg("value1 (string)", _t("The value to which value2 will be appended.")), arg("value2 (string)", _t("The value to append to value1."))],
 		compute: function(value1, value2) {
-			return toString(value1) + toString(value2);
+			return toLocaleString(value1, this.locale) + toLocaleString(value2, this.locale);
 		},
 		isExported: true
 	};
@@ -86746,7 +86773,7 @@ set(value) {
 		description: _t("Remove non-printable characters from a piece of text."),
 		args: [arg("text (string)", _t("The text whose non-printable characters are to be removed."))],
 		compute: function(text) {
-			const _text = toString(text);
+			const _text = toLocaleString(text, this.locale);
 			let cleanedStr = "";
 			for (const char of _text) if (char && char.charCodeAt(0) > 31) cleanedStr += char;
 			return cleanedStr;
@@ -86757,7 +86784,7 @@ set(value) {
 		description: _t("Appends strings to one another."),
 		args: [arg("string (string, range<string>, repeating)", _t("String to append in sequence."))],
 		compute: function(...datas) {
-			return reduceAny(datas, (acc, a) => acc + toString(a), "");
+			return reduceAny(datas, (acc, a) => acc + toLocaleString(a, this.locale), "");
 		},
 		isExported: true
 	};
@@ -86765,7 +86792,7 @@ set(value) {
 		description: _t("Tests whether two strings are identical."),
 		args: [arg("string1 (string)", _t("The first string to compare.")), arg("string2 (string)", _t("The second string to compare."))],
 		compute: function(string1, string2) {
-			return toString(string1) === toString(string2);
+			return toLocaleString(string1, this.locale) === toLocaleString(string2, this.locale);
 		},
 		isExported: true
 	};
@@ -86777,8 +86804,8 @@ set(value) {
 			arg(`starting_at (number, default=${DEFAULT_STARTING_AT})`, _t("The character within text_to_search at which to start the search."))
 		],
 		compute: function(searchFor, textToSearch, startingAt = { value: DEFAULT_STARTING_AT }) {
-			const _searchFor = toString(searchFor);
-			const _textToSearch = toString(textToSearch);
+			const _searchFor = toLocaleString(searchFor, this.locale);
+			const _textToSearch = toLocaleString(textToSearch, this.locale);
 			const _startingAt = toNumber(startingAt, this.locale);
 			if (_textToSearch === "") return new EvaluationError(_t("The text_to_search must be non-empty."));
 			if (_startingAt < 1) return new EvaluationError(_t("The starting_at (%s) must be greater than or equal to 1.", _startingAt));
@@ -86815,8 +86842,8 @@ set(value) {
 		description: _t("Concatenates elements of arrays with delimiter."),
 		args: [arg("delimiter (string)", _t("The character or string to place between each concatenated value.")), arg("value_or_array (string, range<string>, repeating)", _t("Value to be appended using delimiter."))],
 		compute: function(delimiter, ...valuesOrArrays) {
-			const _delimiter = toString(delimiter);
-			return reduceAny(valuesOrArrays, (acc, a) => (acc ? acc + _delimiter : "") + toString(a), "");
+			const _delimiter = toLocaleString(delimiter, this.locale);
+			return reduceAny(valuesOrArrays, (acc, a) => (acc ? acc + _delimiter : "") + toLocaleString(a, this.locale), "");
 		}
 	};
 	const LEFT = {
@@ -86825,7 +86852,7 @@ set(value) {
 		compute: function(text, ...args) {
 			const _numberOfCharacters = args.length ? toNumber(args[0], this.locale) : 1;
 			if (_numberOfCharacters < 0) return new EvaluationError(_t("The number_of_characters (%s) must be positive or null.", _numberOfCharacters));
-			return toString(text).substring(0, _numberOfCharacters);
+			return toLocaleString(text, this.locale).substring(0, _numberOfCharacters);
 		},
 		isExported: true
 	};
@@ -86833,7 +86860,7 @@ set(value) {
 		description: _t("Length of a string."),
 		args: [arg("text (string)", _t("The string whose length will be returned."))],
 		compute: function(text) {
-			return toString(text).length;
+			return toLocaleString(text, this.locale).length;
 		},
 		isExported: true
 	};
@@ -86841,7 +86868,7 @@ set(value) {
 		description: _t("Converts a specified string to lowercase."),
 		args: [arg("text (string)", _t("The string to convert to lowercase."))],
 		compute: function(text) {
-			return toString(text).toLowerCase();
+			return toLocaleString(text, this.locale).toLowerCase();
 		},
 		isExported: true
 	};
@@ -86853,7 +86880,7 @@ set(value) {
 			arg("extract_length (number)", _t("The length of the segment to extract."))
 		],
 		compute: function(text, starting_at, extract_length) {
-			const _text = toString(text);
+			const _text = toLocaleString(text, this.locale);
 			const _starting_at = toNumber(starting_at, this.locale);
 			const _extract_length = toNumber(extract_length, this.locale);
 			if (_starting_at < 1) return new EvaluationError(_t("The starting_at argument (%s) must be positive greater than one.", _starting_at.toString()));
@@ -86866,7 +86893,7 @@ set(value) {
 		description: _t("Capitalizes each word in a specified string."),
 		args: [arg("text_to_capitalize (string)", _t("The text which will be returned with the first letter of each word in uppercase and all other letters in lowercase."))],
 		compute: function(text) {
-			return toString(text).replace(wordRegex, (word) => {
+			return toLocaleString(text, this.locale).replace(wordRegex, (word) => {
 				return word.charAt(0).toUpperCase() + word.slice(1).toLowerCase();
 			});
 		},
@@ -87016,8 +87043,8 @@ set(value) {
 			if (_position < 1) return new EvaluationError(_t("The position (%s) must be greater than or equal to 1.", _position));
 			const _length = toNumber(length, this.locale);
 			if (_length < 0) return new EvaluationError(_t("The length (%s) must be positive or zero.", _length));
-			const _text = toString(text);
-			const _newText = toString(newText);
+			const _text = toLocaleString(text, this.locale);
+			const _newText = toLocaleString(newText, this.locale);
 			return _text.substring(0, _position - 1) + _newText + _text.substring(_position - 1 + _length);
 		},
 		isExported: true
@@ -87028,7 +87055,7 @@ set(value) {
 		compute: function(text, ...args) {
 			const _numberOfCharacters = args.length ? toNumber(args[0], this.locale) : 1;
 			if (_numberOfCharacters < 0) return new EvaluationError(_t("The number_of_characters (%s) must be positive or zero.", _numberOfCharacters));
-			const _text = toString(text);
+			const _text = toLocaleString(text, this.locale);
 			const stringLength = _text.length;
 			return _text.substring(stringLength - _numberOfCharacters, stringLength);
 		},
@@ -87042,8 +87069,8 @@ set(value) {
 			arg(`starting_at (number, default=${DEFAULT_STARTING_AT})`, _t("The character within text_to_search at which to start the search."))
 		],
 		compute: function(searchFor, textToSearch, startingAt = { value: DEFAULT_STARTING_AT }) {
-			const _searchFor = toString(searchFor).toLowerCase();
-			const _textToSearch = toString(textToSearch).toLowerCase();
+			const _searchFor = toLocaleString(searchFor, this.locale).toLowerCase();
+			const _textToSearch = toLocaleString(textToSearch, this.locale).toLowerCase();
 			const _startingAt = toNumber(startingAt, this.locale);
 			if (_textToSearch === "") return {
 				value: CellErrorType.GenericError,
@@ -87073,8 +87100,8 @@ set(value) {
 			arg(`remove_empty_text (boolean, default=${SPLIT_DEFAULT_REMOVE_EMPTY_TEXT})`, _t("Whether or not to remove empty text messages from the split results. The default behavior is to treat         consecutive delimiters as one (if TRUE). If FALSE, empty cells values are added between consecutive delimiters."))
 		],
 		compute: function(text, delimiter, splitByEach = { value: SPLIT_DEFAULT_SPLIT_BY_EACH }, removeEmptyText = { value: SPLIT_DEFAULT_REMOVE_EMPTY_TEXT }) {
-			const _text = toString(text);
-			const _delimiter = escapeRegExp(toString(delimiter));
+			const _text = toLocaleString(text, this.locale);
+			const _delimiter = escapeRegExp(toLocaleString(delimiter, this.locale));
 			const _splitByEach = toBoolean(splitByEach);
 			const _removeEmptyText = toBoolean(removeEmptyText);
 			if (_delimiter.length <= 0) return new EvaluationError(_t("The delimiter (%s) must be not be empty.", _delimiter));
@@ -87096,10 +87123,10 @@ set(value) {
 		compute: function(textToSearch, searchFor, replaceWith, occurrenceNumber) {
 			const _occurrenceNumber = toNumber(occurrenceNumber, this.locale);
 			if (_occurrenceNumber < 0) return new EvaluationError(_t("The occurrenceNumber (%s) must be positive or null.", _occurrenceNumber));
-			const _textToSearch = toString(textToSearch);
-			const _searchFor = toString(searchFor);
+			const _textToSearch = toLocaleString(textToSearch, this.locale);
+			const _searchFor = toLocaleString(searchFor, this.locale);
 			if (_searchFor === "") return _textToSearch;
-			const _replaceWith = toString(replaceWith);
+			const _replaceWith = toLocaleString(replaceWith, this.locale);
 			const reg = new RegExp(escapeRegExp(_searchFor), "g");
 			if (_occurrenceNumber === 0) return _textToSearch.replace(reg, _replaceWith);
 			let n = 0;
@@ -87122,10 +87149,10 @@ set(value) {
 			arg("texts (string, range<string>, repeating)", _t("Text item to join."))
 		],
 		compute: function(delimiter, ignoreEmpty = { value: TEXTJOIN_DEFAULT_IGNORE_EMPTY }, ...textsOrArrays) {
-			const _delimiter = toString(delimiter);
+			const _delimiter = toLocaleString(delimiter, this.locale);
 			const _ignoreEmpty = toBoolean(ignoreEmpty);
 			let n = 0;
-			return reduceAny(textsOrArrays, (acc, a) => !(_ignoreEmpty && toString(a) === "") ? (n++ ? acc + _delimiter : "") + toString(a) : acc, "");
+			return reduceAny(textsOrArrays, (acc, a) => !(_ignoreEmpty && toLocaleString(a, this.locale) === "") ? (n++ ? acc + _delimiter : "") + toLocaleString(a, this.locale) : acc, "");
 		},
 		isExported: true
 	};
@@ -87178,7 +87205,7 @@ set(value) {
 		description: _t("Removes space characters."),
 		args: [arg("text (string)", _t("The text or reference to a cell containing text to be trimmed."))],
 		compute: function(text) {
-			return trimContent(toString(text));
+			return trimContent(toLocaleString(text, this.locale));
 		},
 		isExported: true
 	};
@@ -87186,7 +87213,7 @@ set(value) {
 		description: _t("Converts a specified string to uppercase."),
 		args: [arg("text (string)", _t("The string to convert to uppercase."))],
 		compute: function(text) {
-			return toString(text).toUpperCase();
+			return toLocaleString(text, this.locale).toUpperCase();
 		},
 		isExported: true
 	};
@@ -87283,7 +87310,7 @@ set(value) {
 		args: [arg("url (string)", _t("The full URL of the link enclosed in quotation marks.")), arg("link_label (string, optional)", _t("The text to display in the cell, enclosed in quotation marks."))],
 		compute: function(url, linkLabel) {
 			const processedUrl = toString(url).trim();
-			const processedLabel = toString(linkLabel) || processedUrl;
+			const processedLabel = toLocaleString(linkLabel, this.locale) || processedUrl;
 			if (processedUrl === "") return processedLabel;
 			return markdownLink(processedLabel, processedUrl);
 		},
@@ -87429,7 +87456,8 @@ set(value) {
 		migrationStepRegistry,
 		chartJsExtensionRegistry,
 		onIterationEndEvaluationRegistry,
-		specificRangeTransformRegistry
+		specificRangeTransformRegistry,
+		NonSquishableFunctionRegistry
 	};
 	const helpers = {
 		arg,
@@ -87731,8 +87759,8 @@ exports.stores = stores;
 exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
-__info__.version = "19.4.6";
-__info__.date = "2026-08-10T12:33:32.728Z";
-__info__.hash = "5c667fe";
+__info__.version = "19.4.8";
+__info__.date = "2026-08-19T09:25:13.754Z";
+__info__.hash = "9dd828e";
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);