Merge pull request #165 from toddbranch/filter-by-strict-mode

feat(filter-by): add strict mode; fixes #163

Author: a8m

src/_filter/collection/filter-by.js

@@ -8,7 +8,7 @@
  */
 angular.module('a8m.filter-by', [])
   .filter('filterBy', ['$parse', function( $parse ) {
-    return function(collection, properties, search) {
+    return function(collection, properties, search, strict) {
       var comparator;
 
       search = (isString(search) || isNumber(search)) ?
@@ -32,16 +32,19 @@ angular.module('a8m.filter-by', [])
           if(!~prop.indexOf('+')) {
             comparator = $parse(prop)(elm)
           } else {
-            var propList = prop.replace(new RegExp('\\s', 'g'), '').split('+');
-            comparator = propList.reduce(function(prev, cur, index) {
-              return (index === 1) ? $parse(prev)(elm) + ' ' + $parse(cur)(elm) :
-                prev + ' ' + $parse(cur)(elm);
-            });
+            var propList = prop.replace(/\s+/g, '').split('+');
+            comparator = propList
+              .map(function(prop) { return $parse(prop)(elm); })
+              .join(' ');
           }
 
-          return (isString(comparator) || isNumber(comparator))
-            ? String(comparator).toLowerCase().contains(search)
-            : false;
+          if (!isString(comparator) && !isNumber(comparator)) {
+            return false;
+          }
+
+          comparator = String(comparator).toLowerCase();
+
+          return strict ? comparator === search : comparator.contains(search);
         });
       });
     }

test/spec/filter/collection/filter-by.js

@@ -92,4 +92,41 @@ describe('filterByFilter', function() {
 
   });
 
+  it('should not coerce non-string/number properties', function() {
+    var users = [
+      { id: [1, 2], user: { first_name: 'foo', last_name: 'bar',  mobile: 4444 } }
+    ];
+
+    expect(filter(users, ['id'], 1)).toEqual([]);
+  });
+
+  describe('strict mode', function() {
+
+    var users = [
+      { id: 1, user: { first_name: 'foo', last_name: 'bar',  mobile: 4444 } }
+    ];
+
+    it('should only return exact matches', function() {
+
+      expect(filter(users, ['user.first_name'], 'fo', true)).toEqual([]);
+      expect(filter(users, ['user.first_name'], 'foo', true)).toEqual(users);
+
+    });
+
+    it('should handle multiple properties', function() {
+
+      expect(filter(users, ['user.first_name', 'user.last_name'], 'ba', true)).toEqual([]);
+      expect(filter(users, ['user.first_name', 'user.last_name'], 'bar', true)).toEqual(users);
+
+    });
+
+    it('should handle concatenation', function() {
+
+      expect(filter(users, ['user.first_name + user.last_name'], 'foo ba', true)).toEqual([]);
+      expect(filter(users, ['user.first_name + user.last_name'], 'foo bar', true)).toEqual(users);
+
+    });
+
+  });
+
 });