[web] Handle error cause by currencyDisplay: 'narrowSymbol' in old browsers (#18060)

* Handle error cause by `currencyDisplay: 'narrowSymbol'` in old browsers

RangeError
Value narrowSymbol out of range for Number.prototype.toLocaleString options property currencyDisplay

* Make `formatCurrencyLocalized` bulletproof

GitOrigin-RevId: 26e8abc6f9fb7c06c2d14b9d86af2d84fb9f32e3

services/web/app/src/util/currency.js

@@ -14,19 +14,23 @@
  * @returns {string}
  */
 function formatCurrencyLocalized(amount, currency, locale, stripIfInteger) {
+  const options = { style: 'currency', currency }
   if (stripIfInteger && Number.isInteger(amount)) {
+    options.minimumFractionDigits = 0
+  }
+
+  try {
     return amount.toLocaleString(locale, {
-      style: 'currency',
+      ...options,
-      currency,
-      minimumFractionDigits: 0,
       currencyDisplay: 'narrowSymbol',
     })
-  }
+  } catch {}
-  return amount.toLocaleString(locale, {
+
-    style: 'currency',
+  try {
-    currency,
+    return amount.toLocaleString(locale, options)
-    currencyDisplay: 'narrowSymbol',
+  } catch {}
-  })
+
+  return `${currency} ${amount}`
 }
 
 module.exports = {

services/web/frontend/js/shared/utils/currency.ts

@@ -23,17 +23,21 @@ export function formatCurrencyLocalized(
   locale: string,
   stripIfInteger = false
 ): string {
+  const options: Intl.NumberFormatOptions = { style: 'currency', currency }
   if (stripIfInteger && Number.isInteger(amount)) {
+    options.minimumFractionDigits = 0
+  }
+
+  try {
     return amount.toLocaleString(locale, {
-      style: 'currency',
+      ...options,
-      currency,
-      minimumFractionDigits: 0,
       currencyDisplay: 'narrowSymbol',
     })
-  }
+  } catch {}
-  return amount.toLocaleString(locale, {
+
-    style: 'currency',
+  try {
-    currency,
+    return amount.toLocaleString(locale, options)
-    currencyDisplay: 'narrowSymbol',
+  } catch {}
-  })
+
+  return `${currency} ${amount}`
 }