fix currency filter (fix #953)

Author: yyx990803

src/filters/index.js

@@ -58,20 +58,21 @@ exports.lowercase = function (value) {
  */
 
 var digitsRE = /(\d{3})(?=\d)/g
-
-exports.currency = function (value, sign) {
+exports.currency = function (value, currency) {
   value = parseFloat(value)
   if (!isFinite(value) || (!value && value !== 0)) return ''
-  sign = sign || '$'
-  var s = Math.floor(Math.abs(value)).toString(),
-    i = s.length % 3,
-    h = i > 0
-      ? (s.slice(0, i) + (s.length > 3 ? ',' : ''))
-      : '',
-    v = Math.abs(parseInt((value * 100) % 100, 10)),
-    f = '.' + (v < 10 ? ('0' + v) : v)
-  return (value < 0 ? '-' : '') +
-    sign + h + s.slice(i).replace(digitsRE, '$1,') + f
+  currency = currency || '$'
+  var stringified = Math.abs(value).toFixed(2)
+  var _int = stringified.slice(0, -3)
+  var i = _int.length % 3
+  var head = i > 0
+    ? (_int.slice(0, i) + (_int.length > 3 ? ',' : ''))
+    : ''
+  var _float = stringified.slice(-3)
+  var sign = value < 0 ? '-' : ''
+  return currency + sign + head +
+    _int.slice(i).replace(digitsRE, '$1,') +
+    _float
 }
 
 /**

test/unit/specs/filters/filters_spec.js

@@ -62,7 +62,9 @@ describe('Filters', function () {
     expect(filter(1234)).toBe('$1,234.00')
     expect(filter(1234.45)).toBe('$1,234.45')
     expect(filter(123443434.4343434)).toBe('$123,443,434.43')
-    expect(filter(0.99999)).toBe('$0.99')
+    expect(filter(0.99)).toBe('$0.99')
+    expect(filter(0.99999)).toBe('$1.00')
+    expect(filter(0.76)).toBe('$0.76')
     // sign arg
     expect(filter(2134, '@')).toBe('@2,134.00')
     // falsy, infinity and 0
@@ -72,9 +74,9 @@ describe('Filters', function () {
     expect(filter(undefined)).toBe('')
     expect(filter(Infinity)).toBe('')
     // negative numbers
-    expect(filter(-50)).toBe('-$50.00')
-    expect(filter(-150.43)).toBe('-$150.43')
-    expect(filter(-1500.4343434)).toBe('-$1,500.43')
+    expect(filter(-50)).toBe('$-50.00')
+    expect(filter(-150.43)).toBe('$-150.43')
+    expect(filter(-1500.4343434)).toBe('$-1,500.43')
   })
 
   it('key', function () {