Enable reusing query option objects in find

lib/dao-factory.js

@@ -176,7 +176,7 @@ module.exports = (function() {
   }
   }
 
 
   DAOFactory.prototype.find = function(options) {
   DAOFactory.prototype.find = function(options) {
-    var hasJoin = false;
+    var hasJoin = false
     // no options defined?
     // no options defined?
     // return an emitter which emits null
     // return an emitter which emits null
     if ([null, undefined].indexOf(options) !== -1) {
     if ([null, undefined].indexOf(options) !== -1) {
@@ -185,7 +185,7 @@ module.exports = (function() {
       }).run()
       }).run()
     }
     }
 
 
-    var primaryKeys = this.primaryKeys;
+    var primaryKeys = this.primaryKeys
 
 
     // options is not a hash but an id
     // options is not a hash but an id
     if (typeof options === 'number') {
     if (typeof options === 'number') {
@@ -201,30 +201,34 @@ module.exports = (function() {
       })
       })
 
 
       options = { where: where }
       options = { where: where }
-    } else if ((typeof options === 'string') && (parseInt(options, 10).toString() === options)) {
-      var parsedId = parseInt(options, 10);
+    } else if (typeof options === 'string' && parseInt(options, 10).toString() === options) {
+      var parsedId = parseInt(options, 10)
 
 
       if (!Utils._.isFinite(parsedId)) {
       if (!Utils._.isFinite(parsedId)) {
         throw new Error('Invalid argument to find(). Must be an id or an options object.')
         throw new Error('Invalid argument to find(). Must be an id or an options object.')
       }
       }
 
 
       options = { where: parsedId }
       options = { where: parsedId }
-    } else if ((typeof options === 'object') && (options.hasOwnProperty('include'))) {
-      var includes = options.include
-      hasJoin = true;
-
-      options.include = {}
-
-      includes.forEach(function(daoName) {
-        options.include[daoName] = this.daoFactoryManager.getDAO(daoName)
-
-        if (!options.include[daoName]) {
-          options.include[daoName] = this.getAssociationByAlias(daoName).target
-        }
-      }.bind(this));
+    } else if (typeof options === 'object') {
+      var options = Utils._.clone(options)
+      var includes
+      
+      if (options.hasOwnProperty('include')) {
+        hasJoin = true
+        includes = options.include
+        options.include = {}
+        
+        includes.forEach(function(daoName) {
+          options.include[daoName] = this.daoFactoryManager.getDAO(daoName)
+
+          if (!options.include[daoName]) {
+            options.include[daoName] = this.getAssociationByAlias(daoName).target
+          }
+        }.bind(this))
+      }
 
 
-      // whereCollection is used for  non-primary key updates
-      this.options.whereCollection = options.where || null;
+      // whereCollection is used for non-primary key updates
+      this.options.whereCollection = options.where || null
     }
     }
 
 
     options.limit = 1
     options.limit = 1

spec/dao.spec.js

@@ -184,4 +184,21 @@ describe(Helpers.getTestDialectTeaser("DAO"), function() {
       }.bind(this))
       }.bind(this))
     })
     })
   })
   })
+  
+  describe('find', function find() {
+    it("can reuse query option objects", function(done) {
+      this.User.create({ username: 'fnord' }).success(function() {
+        var query = { where: { username: 'fnord' }}
+
+        this.User.find(query).success(function(user) {
+          expect(user.username).toEqual('fnord')
+
+          this.User.find(query).success(function(user) {
+	        expect(user.username).toEqual('fnord')
+	        done()
+	      }.bind(this))
+        }.bind(this))
+      }.bind(this))
+    })
+  })
 })
 })