From d10a8a243fc6f91e0a55ea08ee2b466093282833 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Fri, 23 May 2025 12:53:44 +0200
Subject: [PATCH] Remove thousand separator when converting domain string to
 float

Since c8d97fac the domain parser format number using locale but the atof
function does not remove the thousand separator which could prevent Number to
parse the string.

Closes #14024
---
 sao/src/common.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/sao/src/common.js b/sao/src/common.js
index fcc84cb833..e0e2b8b58f 100644
--- a/sao/src/common.js
+++ b/sao/src/common.js
@@ -2038,8 +2038,13 @@
                 }
                 let { format } = new Intl.NumberFormat(
                     Sao.i18n.BC47(Sao.i18n.getlang()));
+                // use 10000 because some language (ex: es) add thousand
+                // separator only after 9999
+                let [, thousandSeparator] = /^10(.)?000/.exec(format(10000));
                 let [, decimalSign] = /^0(.)1$/.exec(format(0.1));
-                return Number(string.replace(decimalSign, '.'));
+                return Number(string
+                    .replaceAll(thousandSeparator, '')
+                    .replace(decimalSign, '.'));
             }
             var convert_selection = function() {
                 if (typeof value == 'string') {