fixed to only search string fields

nullivex · Jan 9, 2014 · b20eaf3 · b20eaf3
1 parent 1b7629a
commit b20eaf3
Show file tree

Hide file tree

Showing 5 changed files with 26 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -85,3 +85,11 @@ Uses a Mongoose style sort string eg: `+name -author'
 * Default `''`
 
 Filter results by value applying to the `searchFields`
+
+## Changelog
+
+### 0.1.1
+* Fixed bug with searching on non string fields
+
+### 0.1.0
+* Initial Release
diff --git a/lib/mongoose-list.js b/lib/mongoose-list.js
@@ -20,8 +20,8 @@ module.exports = exports = function list(schema,options){
           search.find.$or.push(obj)
         })
       } else {
-        Model.schema.eachPath(function(path){
-          if(0 !== path.indexOf('_')){
+        Model.schema.eachPath(function(path,type){
+          if(0 !== path.indexOf('_') && String === type.options.type){
             var obj = {}
             obj[path] = searchText
             search.find.$or.push(obj)

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "mongoose-list",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "List plugin for mongoose that allows pagination, filtering, and sorting.",
   "homepage": "https://github.com/snailjs/mongoose-list",
   "bugs": "https://github.com/snailjs/mongoose-list/issues",

diff --git a/test/model.js b/test/model.js
@@ -7,5 +7,20 @@ exports.model = mongoose.model('Test',new mongoose.Schema({
   name: {
     type: String,
     require: true
+  },
+  hits: {
+    type: Number,
+    require: false,
+    default: 10
+  },
+  foo: {
+    type: mongoose.Schema.Types.Mixed,
+    require: false,
+    default: 'fox'
+  },
+  dateCreated: {
+    type: Date,
+    require: false,
+    default: Date.now
   }
 }))
diff --git a/test/mongoose-list.test.js b/test/mongoose-list.test.js