fix(localize): remove unnecessary normalization for Belgium

Mikhail Bashkirov · Mikhail Bashkirov · commit 0afb0e127265 · 2019-06-27T10:46:39.000+02:00
Belgium does not need to be normalized always (e.g. not in "fr-BE").
But Dutch language does need normalization always, because this is where Intl really makes it wrong, both for "nl-Nl" and "nl-BE".
diff --git a/packages/localize/src/number/normalizeIntl.js b/packages/localize/src/number/normalizeIntl.js
@@ -17,7 +17,7 @@ export function normalizeIntl(formattedParts, options, _locale) {
   let normalize = forceNormalSpaces(formattedParts, options);
   // Dutch and Belgian currency must be moved to end of number
   if (options && options.style === 'currency') {
-    if (_locale === 'nl-NL' || _locale.slice(-2) === 'BE') {
+    if (_locale.slice(0, 2) === 'nl') {
       normalize = forceCurrencyToEnd(normalize);
     }
     // Add group separator for Bulgarian locale

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ export function normalizeIntl(formattedParts, options, _locale) {`
`17`	`17`	`let normalize = forceNormalSpaces(formattedParts, options);`
`18`	`18`	`// Dutch and Belgian currency must be moved to end of number`
`19`	`19`	`if (options && options.style === 'currency') {`
`20`		`- if (_locale === 'nl-NL' \|\| _locale.slice(-2) === 'BE') {`
	`20`	`+ if (_locale.slice(0, 2) === 'nl') {`
`21`	`21`	`normalize = forceCurrencyToEnd(normalize);`
`22`	`22`	`}`
`23`	`23`	`// Add group separator for Bulgarian locale`