fix regex search

Author: mathieudutour

bower.json

@@ -4,7 +4,7 @@
     "react-search-input.js",
     "react-search-input.css"
   ],
-  "version": "0.2.1",
+  "version": "0.2.2",
   "homepage": "https://github.com/enki-com/react-search-input",
   "description": "Simple react.js component for a search input, providing a filter function.",
   "keywords": [

example/index.html

@@ -47,7 +47,7 @@
       render() {
         var mails = [{
           user: {
-            name: 'Mathieu',
+            name: 'java',
             job: 'Software Engineer',
             company: 'Enki'
           },
@@ -66,7 +66,7 @@
           ]
         }, {
           user: {
-            name: 'foo'
+            name: 'javascript'
           },
           subject: 'bar',
           dest: [

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-search-input",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "description": "Simple react.js component for a search input, providing a filter function.",
   "main": "react-search-input.js",
   "dependencies": {

react-search-input.js

@@ -60,34 +60,7 @@
         return function(item) {
           if (term === '') {return true;}
           // escape special symbols to ensure `term` is a valid regex
-          term = term.replace(/[\-\[\]\/\{\}\(\)\*\+\?\.\\\^\$\|]/g, "\\$&");
-
-          var _getValuesForKey = function(key, _item) {
-            var keys = key.split('.');
-            var results = [_item];
-            keys.forEach(function(_key) {
-              var tmp = [];
-              results.forEach(function(result) {
-                if (result) {
-                  if(result instanceof Array) {
-                    result.forEach(function(res) {
-                      tmp.push(res[_key]);
-                    });
-                  } else {
-                    tmp.push(result[_key]);
-                  }
-                }
-              });
-              results = tmp;
-            });
-            return results.map(function(result) {
-              if (typeof result === 'string') {
-                return result.toLowerCase();
-              } else {
-                return null;
-              }
-            });
-          };
+          var escapedTerm = term.replace(/[\-\[\]\/\{\}\(\)\*\+\?\.\\\^\$\|]/g, "\\$&");
 
           if (keys) {
             if( typeof keys === 'string' ) {
@@ -97,8 +70,11 @@
               var values = _getValuesForKey(keys[i], item);
               var found = false;
               values.forEach(function(value) {
-                if (value && value.search(term) !== -1) {
-                  found = true;
+                try {
+                  if (value && value.search(term) !== -1) {
+                    found = true;
+                  }
+                } catch(e) {
                 }
               });
               if (found) {
@@ -107,10 +83,10 @@
             }
             return false;
           } else {
-            if( typeof item === 'string' ) {
+            try {
               var stringValue = item.toLowerCase();
               return (stringValue.search(term) !== -1);
-            } else {
+            } catch(e) {
               return false;
             }
           }
@@ -119,5 +95,32 @@
     }
   });
 
+  var _getValuesForKey = function(key, _item) {
+    var keys = key.split('.');
+    var results = [_item];
+    keys.forEach(function(_key) {
+      var tmp = [];
+      results.forEach(function(result) {
+        if (result) {
+          if(result instanceof Array) {
+            result.forEach(function(res) {
+              tmp.push(res[_key]);
+            });
+          } else {
+            tmp.push(result[_key]);
+          }
+        }
+      });
+      results = tmp;
+    });
+    return results.map(function(result) {
+      if (typeof result === 'string') {
+        return result.toLowerCase();
+      } else {
+        return null;
+      }
+    });
+  };
+
   return Search;
 });