ayo/lion
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(localize): for bg-BG locale, correct Intl output for group separator

Browse source
This commit is contained in:
Thijs Louisse 2019-08-07 12:13:28 +02:00 committed by Thomas Allmer
parent 92a548f677
commit c151f01b0b
2 changed files with 16 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
packages/localize/src/number/forceSpaceInsteadOfZeroForGroup.js Normal file
View file

@ -0,0 +1,14 @@
/**
* @desc Intl uses 0 as group separator for bg-BG locale.
* This should be a ' '
* @param {{type,value}[]} formattedParts
* @returns {{type,value}[]} corrected formatted parts
*/
export function forceSpaceInsteadOfZeroForGroup(formattedParts) {
return formattedParts.map(p => {
if (p.type === 'group' && p.value === '0') {
p.value = ' '; // eslint-disable-line no-param-reassign
}
return p;
});
}

2
packages/localize/src/number/normalizeIntl.js
View file

@ -4,6 +4,7 @@ import { forceCurrencyToEnd } from './forceCurrencyToEnd.js';
import { forceNormalSpaces } from './forceNormalSpaces.js';
import { forceSpaceBetweenCurrencyCodeAndNumber } from './forceSpaceBetweenCurrencyCodeAndNumber.js';
import { forceYenSymbol } from './forceYenSymbol.js';
import { forceSpaceInsteadOfZeroForGroup } from './forceSpaceInsteadOfZeroForGroup.js';
/**
* Function with all fixes on localize
@ -23,6 +24,7 @@ export function normalizeIntl(formattedParts, options, _locale) {
// Add group separator for Bulgarian locale
if (_locale === 'bg-BG') {
normalize = forceAddGroupSeparators(normalize, getGroupSeparator());
normalize = forceSpaceInsteadOfZeroForGroup(normalize);
}
// Force space between currency code and number
if (_locale === 'en-GB' || _locale === 'en-US' || _locale === 'en-AU') {