Fixed cleanQuery error. cleanQuery call moved after middleware initia…

…lization to allow usage of populate and other options even when we are not using middleware.
florianholzapfel · Jun 16, 2013 · 807f7ac · 807f7ac
1 parent bac2407
commit 807f7ac
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/express-restify-mongoose.js b/lib/express-restify-mongoose.js
@@ -38,18 +38,18 @@ var restify = function(app, model, options) {
 			var m = options.middleware;
 			options.middleware = [ m ];
 		}
-        options.middleware.unshift(cleanQuery);
 	} else {
 		options.middleware = [];
 	}
+    options.middleware.unshift(cleanQuery);
 
     var queryOptions = {protected: ["skip", "limit", "sort", "populate"], current: {}};
 
 	var uri_items = util.format('%s%s/%ss', options.prefix, options.version, model.modelName);
 	var uri_item = util.format('%s%s/%ss/:id', options.prefix, options.version, model.modelName);
 
     function cleanQuery(req, res, next) {
-		queryOptions.current = {};
+        queryOptions.current = {};
         for(var key in req.query) {
             if(!model.schema.paths.hasOwnProperty(key)) {
                 if(queryOptions.protected.indexOf(key) != -1) {
@@ -183,4 +183,4 @@ var restify = function(app, model, options) {
 	});
 };
 
-module.exports.serve = restify;
+module.exports.serve = restify;