Create a gist now

Instantly share code, notes, and snippets.

var currencyValidator = {
format: function (number) {
return (Math.trunc(number * 100) / 100).toFixed(2)
},
parse: function (newString, oldNumber) {
var CleanParse = function (value) {
return { value: value }
}
var CurrencyWarning = function (warning, value) {
return {
warning: warning,
value: value,
attempt: newString
}
}
var NotAValidDollarAmountWarning = function (value) {
return new CurrencyWarning(newString + ' is not a valid dollar amount', value)
}
var AutomaticConversionWarning = function (value) {
return new CurrencyWarning(newString + ' was automatically converted to ' + value, value)
}
var newNumber = Number(newString)
var indexOfDot = newString.indexOf('.')
var indexOfE = newString.indexOf('e')
if (isNaN(newNumber)) {
if (
indexOfDot === -1 &&
indexOfE > 0 &&
indexOfE === newString.length - 1 &&
Number(newString.slice(0, indexOfE)) !== 0
) {
return new CleanParse(oldNumber)
} else {
return new NotAValidDollarAmountWarning(oldNumber)
}
}
var newCurrencyString = currencyValidator.format(newNumber)
var newCurrencyNumber = Number(newCurrencyString)
if (newCurrencyNumber === newNumber) {
if (indexOfE !== -1 && indexOfE === newString.length - 2) {
return new AutomaticConversionWarning(newNumber)
} else {
return new CleanParse(newNumber)
}
} else {
return new NotAValidDollarAmountWarning(
newNumber > newCurrencyNumber
? newCurrencyNumber
: oldNumber
)
}
}
}
@bcoppingHOF

Hi, I've been looking at the vue documentation where they utilise your currency validator here -
https://vuejs.org/v2/guide/migration.html#Filters-Outside-Text-Interpolations-removed

I noticed a weird bug when playing about with the currency inputs in the examples.
For example if I try entering 33.33 It will round down to 33.29, making it impossible to enter that amount.

It also happens trying to enter 33.80, rounds down to 33.79, haven't noticed any others :)

@shibamo
shibamo commented Feb 11, 2017

@bcoppingHOF,
Try change line 3 to:
return (Math.trunc(number * 1000) / 1000).toFixed(2)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment