From f37d6bd17344d1e30ab4cdfd6654362dc61bc12a 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
(grafted from dc438f32fa7887e5e0e241a6154a1c685e9cda20)

--HG--
branch : 6.0
---
 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 f511ea8f01..33d6a35300 100644
--- a/sao/src/common.js
+++ b/sao/src/common.js
@@ -1976,8 +1976,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
+                    .replace(new RegExp(thousandSeparator, 'g'), '')
+                    .replace(decimalSign, '.'));
             }
             var convert_selection = function() {
                 if (typeof value == 'string') {