Fix issue #447

lib/dao-factory.js

@@ -140,12 +140,13 @@ module.exports = (function() {
   }
 
   DAOFactory.prototype.findAll = function(options) {
-    var hasJoin = false;
+    var hasJoin = false
+    var options = Utils._.clone(options)
 
     if ((typeof options === 'object') && (options.hasOwnProperty('include'))) {
       var includes = options.include
 
-      hasJoin = true;
+      hasJoin = true
       options.include = {}
 
       includes.forEach(function(daoName) {
@@ -154,10 +155,10 @@ module.exports = (function() {
         if (!options.include[daoName]) {
           options.include[daoName] = this.getAssociationByAlias(daoName).target
         }
-      }.bind(this));
+      }.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
     }
 
     return this.QueryInterface.select(this, this.tableName, options, { type: 'SELECT', hasJoin: hasJoin })
@@ -168,7 +169,7 @@ module.exports = (function() {
     var optcpy = Utils._.clone(options)
     optcpy.attributes = optcpy.attributes || [Utils.addTicks(this.tableName)+".*"]
 
-    // whereCollection is used for  non-primary key updates
+    // whereCollection is used for non-primary key updates
     this.options.whereCollection = optcpy.where || null;
 
     return this.QueryInterface.select(this, [this.tableName, joinTableName], optcpy, { type: 'SELECT' })

spec/dao.spec.js

@@ -168,5 +168,20 @@ describe(Helpers.getTestDialectTeaser("DAO"), function() {
         }.bind(this))
       }.bind(this))
     })
+
+    it("can reuse query option objects", function(done) {
+      this.User.create({ username: 'fnord' }).success(function() {
+        var query = { where: { username: 'fnord' }}
+
+        this.User.findAll(query).success(function(users) {
+          expect(users[0].username).toEqual('fnord')
+
+          this.User.findAll(query).success(function(users) {
+	        expect(users[0].username).toEqual('fnord')
+	        done()
+	      }.bind(this))
+        }.bind(this))
+      }.bind(this))
+    })
   })
 })