maverick/flex-template-web
1
0
Fork 0
mirror of https://github.com/kingomarnajjar/flex-template-web.git synced 2026-07-30 18:16:48 +10:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix initial value handlign with zero as currency

Browse source
This commit is contained in:
Kimmo Puputti 2017-08-24 10:58:54 +03:00
parent 19f66eef6b
commit 98a913a4c5
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/components/CurrencyInputField/CurrencyInputField.js
View file

@ -51,6 +51,7 @@ class CurrencyInputComponent extends Component {
const initialValue = input.value instanceof types.Money
? convertMoneyToNumber(input.value, currencyConfig.subUnitDivisor)
: defaultValue;
const hasInitialValue = typeof initialValue === 'number' && !isNaN(initialValue);
// We need to handle number format - some locales use dots and some commas as decimal separator
// TODO Figure out if this could be digged from React-Intl directly somehow
@ -60,7 +61,7 @@ class CurrencyInputComponent extends Component {
try {
// whatever is passed as a default value, will be converted to currency string
// Unformatted value is digits + localized sub unit separator ("9,99")
const unformattedValue = initialValue
const unformattedValue = hasInitialValue
? truncateToSubUnitPrecision(
ensureSeparator(initialValue.toString(), usesComma),
currencyConfig.subUnitDivisor,
@ -68,7 +69,7 @@ class CurrencyInputComponent extends Component {
)
: '';
// Formatted value fully localized currency string ("$1,000.99")
const formattedValue = initialValue
const formattedValue = hasInitialValue
? intl.formatNumber(ensureDotSeparator(unformattedValue), currencyConfig)
: '';
@ -198,7 +199,7 @@ class CurrencyInputComponent extends Component {
CurrencyInputComponent.defaultProps = {
className: null,
currencyConfig: null,
defaultValue: 0,
defaultValue: null,
input: null,
placeholder: null,
};