Repository navigation
Decimal and grouping separator for en-ZA does not align with in-country usage #48120
Description
What is the problem this feature will solve?
Currently, the built-in number and currency formatting libraries in Node.js return "," (comma) as decimal separator and " " (whitespace) as grouping separator for formatted numbers and currencies in en-ZA (English South Africa) locale.
console.log(new Intl.NumberFormat('en-ZA', { maximumSignificantDigits: 3 }).format(4.56));
// Expected output: "4.56"
// Actual output: "4,56"
console.log(new Intl.NumberFormat('en-ZA', { style: 'currency', currency: 'SAR' }).format(1234.56));
// Expected output: "SAR 1,234.56"
// Actual output: "SAR 1 234,56"
console.log(new IntlMessageFormat(
'The price is: {price, number, ::currency/SAR}',
'en-ZA'
).format({price: 1234.56}));
// Expected output: "The price is: SAR 1,234.56"
// Actual output: "The price is: SAR 1 234,56"Officially, comma is the decimal separator and space is the grouping separator for numbers in en-ZA (https://www.sadev.co.za/content/how-correctly-format-currency-south-africa), however in common usage, price labels in shops, bank documents and statements use period (.) as decimal separator and comma (,) as grouping separator.
For context, check out following CLDR issues:
Decimal separator - South Africa
Decimal and grouping separator for en_ZA does not align with in-country usage
What is the feature you are proposing to solve the problem?
Upgrade CLDR data in NodeJS. The fix for numbers and currencies in en-ZA locale from CLDR is set to be released as part of CLDR 43.1. This CLDR update will be integrated in an upcoming minor release for ICU4C/ICU4J. Node 20 currently uses ICU 73, while Node 18 (current stable release) uses ICU 72.
- Node 18 - Upgrade to ICU4C 73 and pick up the next minor release (when it is available) including the fix for numbers and currencies in en-ZA locale in CLDR 43.1.
- Node 20 - Pick up the next minor release for ICU4C 73 (when it is available) including the fix for numbers and currencies in en-ZA locale in CLDR 43.1.
What alternatives have you considered?
Create overrides in our consumer library. Unfortunately, it is not feasible for us to create overrides for every number and price formatting use case.
Activity
aduh95 commentedon May 22, 2023
@nodejs/i18n
srl295 commentedon May 22, 2023
@abster Hi! Good to see you here. Yeah, can pick up the ICU dot release when it's available…
Antonius-S commentedon Jun 1, 2023
@abster meanwhile won't this help? https://nodejs.org/docs/latest-v19.x/api/intl.html#providing-icu-data-at-runtime
abster commentedon Jun 16, 2023
@Antonius-S From my understanding CLDR data can be updated on a specific ICU major release but updating CLDR data on a previous ICU release may not always be compatible, since CLDR releases are tied to specific ICU major releases.
abster commentedon Jun 16, 2023
Just as a heads-up, ICU 73.2 has been released: https://github.com/unicode-org/icu/releases/tag/release-73-2. Updating node.js to use ICU4C-73.2 should include the fix for en-ZA (addressed in CLDR issue).
richardlau commentedon Jun 16, 2023
To set expectations, our automation should pick up the new ICU release over the weekend. There's no reason to rush this any earlier as we won't be able to run the CI on it until the security releases are done next week.
Antonius-S commentedon Jun 20, 2023
@abster
According to docs, for external ICU to work, the binary should be built with at most small-icu option so this is just a temporary workaround to solve the issue until new build is ready
abster commentedon Jul 7, 2023
@richardlau - I see that icu version is updated to 73.2 in the main branch: https://github.com/nodejs/node/blob/main/tools/icu/current_ver.dep
The latest minor release (v18.6.1) for node 18 is however still on 72.1. Do you know when node 18 would be updated to ICU 73.2?
https://github.com/nodejs/node/blob/v18.16.1/tools/icu/current_ver.dep
richardlau commentedon Jul 7, 2023
We usually want changes in a current release for at least two weeks before landing in LTS -- I see #48502 only just recently went out in Node.js 20.4.0 so it might be a bit soon for the being prepared 18.17.0 (#48694).
abster commentedon Jul 26, 2023
@richardlau - The latest minor release (v18.17.0) for node 18 is on ICU 73.1. Do you know when you expect node 18 to be updated to ICU 73.2?
richardlau commentedon Jul 26, 2023
Sometime in August according to the current plan in nodejs/Release#737. May be a little clearer after tomorrow's working group meeting where we usually look at the schedules and see who's available to do releases nodejs/Release#886.
srl295 commentedon Jul 26, 2023
@abster wrote:
Hi…
Correct, prior ICUs cannot be upgraded to arbitrary CLDR versions. If there's enough interest/available hands to help, a CLDR fix could be back-ported to a prior CLDR maint line, which could then be picked up by the corresponding ICU maint line.
abster commentedon Sep 1, 2023
(v18.17.1) is on ICU 73.1. Is there a planned release for node 18, which will include update to ICU 73.2? Also, is there a target release date?
richardlau commentedon Sep 1, 2023
FYI @ruyadorno perhaps we can consider the ICU update (#48502) for #49220.
alexmojaki commentedon Oct 17, 2023
Looks like there was a change recently, and I see the two PRs above are merged. Is this done?
aduh95 commentedon Oct 17, 2023
The ICU update has landed in all supported release lines, I’m going to assume this is fixed. Please comment or reopen if the issue still remains.
icyJoseph commentedon Feb 14, 2025
Hi,
Using Node 22.12.0, and I still see this issue:
Whereas, running the same on, say Chrome, returns,
SAR 1,234.56. Other browsers, as well asDeno, return the same.How can I debug/fix this?
michaelwiles commentedon Jul 24, 2025
So this is still an issue...
It's challenging as my tests now use a different separator to running in the browser as chrome formats with the dot as the decimal separator in en-ZA...