fix #928: filterBy false

Author: yyx990803

src/filters/array-filters.js

@@ -15,7 +15,8 @@ exports.filterBy = function (arr, search, delimiter, dataKey) {
   if (delimiter && delimiter !== 'in') {
     dataKey = delimiter
   }
-  if (!search) {
+  /* jshint eqeqeq: false */
+  if (search == null) {
     return arr
   }
   // cast to lowercase string
@@ -82,4 +83,4 @@ function contains (val, search) {
   } else if (val != null) {
     return val.toString().toLowerCase().indexOf(search) > -1
   }
-}
+}

test/unit/specs/filters/filters_spec.js

@@ -126,6 +126,10 @@ describe('Filters', function () {
     res = filter(arr, 'yoyo')
     expect(res.length).toBe(1)
     expect(res[0]).toBe(arr[2])
+    // filter by false (#928)
+    res = filter([{a:false},{b:true}], false)
+    expect(res.length).toBe(1)
+    expect(res[0].a).toBe(false)
   })
 
   it('orderBy', function () {
@@ -201,4 +205,4 @@ function assertNumberAndFalsy (filter) {
   expect(filter(undefined)).toBe('')
   expect(filter(null)).toBe('')
   expect(filter(false)).toBe('')
-}
+}