Make comma's and period interchangeable when entering figures (#2672)

- Only allowed for formats not using period or comma as a thousand separator

packages/loot-core/src/shared/arithmetic.ts

 // @ts-strict-ignore
-import { getNumberFormat } from './util';
+import { currencyToAmount } from './util';
 
 function fail(state, msg) {
   throw new Error(
@@ -37,31 +37,16 @@ function parsePrimary(state) {
     next(state);
   }
 
-  if (/\p{Sc}/u.test(char(state))) {
-    next(state);
-  }
-
-  // TODO: The regex should probably respect the number format better,
-  // and we should do more strict parsing
   let numberStr = '';
-  while (char(state) && char(state).match(/[0-9,.]/)) {
-    const thousandsSep = getNumberFormat().separator === ',' ? '.' : ',';
-
-    // Don't include the thousands separator
-    if (char(state) === thousandsSep) {
-      next(state);
-    } else {
-      numberStr += next(state);
-    }
+  while (char(state) && char(state).match(/[0-9,. ]|\p{Sc}/u)) {
+    numberStr += next(state);
   }
 
   if (numberStr === '') {
     fail(state, 'Unexpected character');
   }
 
-  const number = parseFloat(
-    numberStr.replace(getNumberFormat().separator, '.'),
-  );
+  const number = currencyToAmount(numberStr);
   return isNegative ? -number : number;
 }
 

packages/loot-core/src/shared/util.ts

@@ -255,13 +255,14 @@ export function getNumberFormat({
   format?: NumberFormats;
   hideFraction: boolean;
 } = numberFormatConfig) {
-  let locale, regex, separator;
+  let locale, regex, separator, separatorRegex;
 
   switch (format) {
     case 'space-comma':
       locale = 'en-SE';
-      regex = /[^-0-9,]/g;
+      regex = /[^-0-9,.]/g;
       separator = ',';
+      separatorRegex = /[,.]/g;
       break;
     case 'dot-comma':
       locale = 'de-DE';
@@ -270,8 +271,9 @@ export function getNumberFormat({
       break;
     case 'space-dot':
       locale = 'dje';
-      regex = /[^-0-9.]/g;
+      regex = /[^-0-9,.]/g;
       separator = '.';
+      separatorRegex = /[,.]/g;
       break;
     case 'comma-dot-in':
       locale = 'en-IN';
@@ -293,6 +295,7 @@ export function getNumberFormat({
       maximumFractionDigits: hideFraction ? 0 : 2,
     }),
     regex,
+    separatorRegex,
   };
 }
 
@@ -342,11 +345,20 @@ export function amountToCurrencyNoDecimal(n) {
 }
 
 export function currencyToAmount(str: string) {
-  const amount = parseFloat(
-    str
-      .replace(getNumberFormat().regex, '')
-      .replace(getNumberFormat().separator, '.'),
-  );
+  let amount;
+  if (getNumberFormat().separatorRegex) {
+    amount = parseFloat(
+      str
+        .replace(getNumberFormat().regex, '')
+        .replace(getNumberFormat().separatorRegex, '.'),
+    );
+  } else {
+    amount = parseFloat(
+      str
+        .replace(getNumberFormat().regex, '')
+        .replace(getNumberFormat().separator, '.'),
+    );
+  }
   return isNaN(amount) ? null : amount;
 }
 

upcoming-release-notes/2672.md

+---
+category: Enhancements
+authors: [Wizmaster]
+---
+
+Comma and period decimal separator can both be used for number format not using those as thousand separator.