@odoo/o-spreadsheet 19.1.30 → 19.1.32

package/dist/o_spreadsheet.css

@@ -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.1.30
- * @date 2026-07-30T06:53:53.625Z
- * @hash b4f7ab8
+ * @version 19.1.32
+ * @date 2026-08-21T15:29:31.737Z
+ * @hash d34d0e7
  */
 :root {
   --os-gray-100: light-dark(#f9fafb, #1b1d26);
@@ -291,6 +291,7 @@
   .os-input {
     border-width: 0 0 1px 0;
     border-color: transparent;
+    border-style: solid;
     outline: none;
     text-overflow: ellipsis;
     color: var(--os-text-body);
@@ -542,6 +543,7 @@
     width: 100%;
     outline: none;
     border-color: var(--os-border-color);
+    border-style: solid;
     color: var(--os-gray-900);
 
     &::placeholder {

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.1.30
- * @date 2026-07-30T06:53:51.978Z
- * @hash b4f7ab8
+ * @version 19.1.32
+ * @date 2026-08-21T15:29:30.638Z
+ * @hash d34d0e7
  */
 
 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";
@@ -3726,6 +3726,18 @@ function toString(data) {
 		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, "");
@@ -3915,19 +3927,58 @@ function applyVectorization(formula, args, acceptToVectorize = void 0) {
 		}
 	}
 	if (countVectorizedCol === 1 && countVectorizedRow === 1) return formula(...args);
-	const getArgOffset = (i, j) => args.map((arg, index) => {
-		switch (vectorArgsType?.[index]) {
-			case "matrix": return arg[i][j];
-			case "horizontal": return arg[i][0];
-			case "vertical": return arg[0][j];
-			case void 0: return arg;
+	const argsBuffer = new Array(args.length);
+	const argGetters = [];
+	const vectorizedIndices = [];
+	for (let k = 0; k < args.length; k++) {
+		const arg = args[k];
+		switch (vectorArgsType?.[k]) {
+			case "matrix":
+				argGetters.push((i, j) => arg[i][j]);
+				vectorizedIndices.push(k);
+				break;
+			case "horizontal":
+				argGetters.push((i) => arg[i][0]);
+				vectorizedIndices.push(k);
+				break;
+			case "vertical":
+				argGetters.push((_i, j) => arg[0][j]);
+				vectorizedIndices.push(k);
+				break;
+			case void 0:
+				argsBuffer[k] = arg;
+				break;
 		}
-	});
-	return generateMatrix(countVectorizedCol, countVectorizedRow, (col, row) => {
-		if (col > vectorizedColLimit - 1 || row > vectorizedRowLimit - 1) return new NotAvailableError(_t("Array arguments to [[FUNCTION_NAME]] are of different size."));
-		const singleCellComputeResult = formula(...getArgOffset(col, row));
-		return isMatrix(singleCellComputeResult) ? singleCellComputeResult[0][0] : singleCellComputeResult;
-	});
+	}
+	const nbVectorized = vectorizedIndices.length;
+	let callFormula;
+	switch (argsBuffer.length) {
+		case 1:
+			callFormula = () => formula(argsBuffer[0]);
+			break;
+		case 2:
+			callFormula = () => formula(argsBuffer[0], argsBuffer[1]);
+			break;
+		case 3:
+			callFormula = () => formula(argsBuffer[0], argsBuffer[1], argsBuffer[2]);
+			break;
+		default: callFormula = () => formula(...argsBuffer);
+	}
+	const result = new Array(countVectorizedCol);
+	for (let col = 0; col < countVectorizedCol; col++) {
+		const column = new Array(countVectorizedRow);
+		result[col] = column;
+		for (let row = 0; row < countVectorizedRow; row++) {
+			if (col > vectorizedColLimit - 1 || row > vectorizedRowLimit - 1) {
+				column[row] = new NotAvailableError(_t("Array arguments to [[FUNCTION_NAME]] are of different size."));
+				continue;
+			}
+			for (let k = 0; k < nbVectorized; k++) argsBuffer[vectorizedIndices[k]] = argGetters[k](col, row);
+			const singleCellComputeResult = callFormula();
+			column[row] = isMatrix(singleCellComputeResult) ? singleCellComputeResult[0][0] : singleCellComputeResult;
+		}
+	}
+	return result;
 }
 /**
 * This function allows to visit arguments and stop the visit if necessary.
@@ -17264,7 +17315,7 @@ var FigureComponent = class extends Component {
 			case "ArrowLeft":
 			case "ArrowRight":
 			case "ArrowUp":
-				const { col, row, offset } = this.postionInBoundary(this.props.figureUI, ev.key);
+				const { col, row, offset } = this.postionInBoundary(this.env.model.getters.getActiveSheetId(), this.props.figureUI, ev.key);
 				this.env.model.dispatch("UPDATE_FIGURE", {
 					sheetId: this.env.model.getters.getActiveSheetId(),
 					figureId: this.props.figureUI.id,
@@ -17288,34 +17339,52 @@ var FigureComponent = class extends Component {
 				break;
 		}
 	}
-	postionInBoundary(position, key) {
-		const sheetId = this.env.model.getters.getActiveSheetId();
-		let { col, row, offset } = position;
+	postionInBoundary(sheetId, figure, key) {
+		let { col, row, offset } = figure;
 		offset = { ...offset };
+		const maxAnchor = this.env.model.getters.getMaxAnchorOffset(sheetId, figure.height, figure.width);
 		switch (key) {
 			case "ArrowUp":
 				if (offset.y === 0) {
 					row--;
+					while (row > 0 && this.env.model.getters.isRowHiddenByUser(sheetId, row)) row--;
 					offset.y = this.env.model.getters.getRowSize(sheetId, row) - 1;
 				} else offset.y--;
 				break;
 			case "ArrowLeft":
 				if (offset.x === 0) {
 					col--;
+					while (col > 0 && this.env.model.getters.isColHiddenByUser(sheetId, col)) col--;
 					offset.x = this.env.model.getters.getColSize(sheetId, col) - 1;
 				} else offset.x--;
 				break;
 			case "ArrowDown":
 				if (offset.y === this.env.model.getters.getRowSize(sheetId, row)) {
 					row++;
+					while (row <= maxAnchor.row && this.env.model.getters.isRowHiddenByUser(sheetId, row)) row++;
 					offset.y = 0;
 				} else offset.y++;
 				break;
 			case "ArrowRight": if (offset.x === this.env.model.getters.getColSize(sheetId, row)) {
 				col++;
+				while (col <= maxAnchor.col && this.env.model.getters.isColHiddenByUser(sheetId, col)) col++;
 				offset.x = 0;
 			} else offset.x++;
 		}
+		if (col < 0) {
+			col = 0;
+			offset.x = 0;
+		} else if (col > maxAnchor.col || col === maxAnchor.col && offset.x > maxAnchor.offset.x) {
+			col = maxAnchor.col;
+			offset.x = maxAnchor.offset.x;
+		}
+		if (row < 0) {
+			row = 0;
+			offset.y = 0;
+		} else if (row > maxAnchor.row || row === maxAnchor.row && offset.y > maxAnchor.offset.y) {
+			row = maxAnchor.row;
+			offset.y = maxAnchor.offset.y;
+		}
 		return {
 			col,
 			row,
@@ -18480,12 +18549,15 @@ function validateArguments(descr) {
 //#endregion
 //#region src/functions/create_compute_function.ts
 function createComputeFunction(descr) {
+	let currentArgDefinitions = [];
 	function vectorizedCompute(...args) {
 		const acceptToVectorize = [];
+		currentArgDefinitions = new Array(args.length);
 		const getArgToFocus = argTargeting(descr, args.length);
 		for (let i = 0; i < args.length; i++) {
 			const argIndex = getArgToFocus(i).index ?? -1;
 			const argDefinition = descr.args[argIndex];
+			currentArgDefinitions[i] = argDefinition;
 			const arg = args[i];
 			if (!isMatrix(arg) && argDefinition.acceptMatrixOnly) throw new BadExpressionError(_t("Function %s expects the parameter '%s' to be reference to a cell or range.", descr.name, (i + 1).toString()));
 			acceptToVectorize.push(!argDefinition.acceptMatrix);
@@ -18500,8 +18572,7 @@ function createComputeFunction(descr) {
 	function errorHandlingCompute(...args) {
 		for (let i = 0; i < args.length; i++) {
 			const arg = args[i];
-			const getArgToFocus = argTargeting(descr, args.length);
-			if (!descr.args[getArgToFocus(i).index ?? i].acceptErrors && !isMatrix(arg) && isEvaluationError(arg?.value)) return arg;
+			if (!currentArgDefinitions[i].acceptErrors && !isMatrix(arg) && isEvaluationError(arg?.value)) return arg;
 		}
 		try {
 			return computeFunctionToObject.apply(this, args);
@@ -19001,7 +19072,7 @@ const ARRAYTOTEXT = {
 		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"));
 	},
@@ -24987,7 +25058,7 @@ const CONCAT = {
 	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
 };
@@ -25819,7 +25890,7 @@ const CLEAN = {
 	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;
@@ -25830,7 +25901,7 @@ const CONCATENATE = {
 	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
 };
@@ -25838,7 +25909,7 @@ const EXACT = {
 	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
 };
@@ -25850,8 +25921,8 @@ const FIND = {
 		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));
@@ -25865,8 +25936,8 @@ const JOIN = {
 	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 = {
@@ -25875,7 +25946,7 @@ const LEFT = {
 	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
 };
@@ -25883,7 +25954,7 @@ const LEN = {
 	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
 };
@@ -25891,7 +25962,7 @@ const LOWER = {
 	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
 };
@@ -25903,7 +25974,7 @@ const MID = {
 		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()));
@@ -25916,7 +25987,7 @@ const PROPER = {
 	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();
 		});
 	},
@@ -25991,9 +26062,9 @@ const REPLACE = {
 	compute: function(text, position, length, newText) {
 		const _position = toNumber(position, this.locale);
 		if (_position < 1) return new EvaluationError(_t("The position (%s) must be greater than or equal to 1.", _position));
-		const _text = toString(text);
+		const _text = toLocaleString(text, this.locale);
 		const _length = toNumber(length, this.locale);
-		const _newText = toString(newText);
+		const _newText = toLocaleString(newText, this.locale);
 		return _text.substring(0, _position - 1) + _newText + _text.substring(_position - 1 + _length);
 	},
 	isExported: true
@@ -26004,7 +26075,7 @@ const RIGHT = {
 	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));
-		const _text = toString(text);
+		const _text = toLocaleString(text, this.locale);
 		const stringLength = _text.length;
 		return _text.substring(stringLength - _numberOfCharacters, stringLength);
 	},
@@ -26018,8 +26089,8 @@ const SEARCH = {
 		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,
@@ -26049,8 +26120,8 @@ const SPLIT = {
 		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));
@@ -26072,10 +26143,10 @@ const SUBSTITUTE = {
 	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;
@@ -26098,10 +26169,10 @@ const TEXTJOIN = {
 		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
 };
@@ -26154,7 +26225,7 @@ const TRIM = {
 	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
 };
@@ -26162,7 +26233,7 @@ const UPPER = {
 	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
 };
@@ -26259,7 +26330,7 @@ const HYPERLINK = {
 	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);
 	},
@@ -30583,12 +30654,13 @@ var FilterMenu = class extends Component {
 	}
 	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,
@@ -68941,6 +69013,10 @@ function inverseCreateFigure(cmd) {
 }
 function inverseCreateChart(cmd) {
 	return [{
+		type: "DELETE_CHART",
+		chartId: cmd.chartId,
+		sheetId: cmd.sheetId
+	}, {
 		type: "DELETE_FIGURE",
 		figureId: cmd.figureId,
 		sheetId: cmd.sheetId
@@ -79119,6 +79195,6 @@ const chartHelpers = {
 //#endregion
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, ClientDisconnectedError, CommandResult, CorePlugin, CoreViewPlugin, DEFAULT_LOCALE, DEFAULT_LOCALES, DispatchResult, EvaluationError, LocalTransportService, Model, PivotRuntimeDefinition, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, SpreadsheetPivotTable, UIPlugin, __info__, addFunction, addRenderingLayer, astToFormula, categories, chartHelpers, components, constants, convertAstNodes, coreTypes, createAutocompleteArgumentsProvider, 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.1.30";
-__info__.date = "2026-07-30T06:53:51.978Z";
-__info__.hash = "b4f7ab8";
+__info__.version = "19.1.32";
+__info__.date = "2026-08-21T15:29:30.638Z";
+__info__.hash = "d34d0e7";