You should now be able to declare false/true booleans within the where search cr…

…iteria for associations. Closes #820

lib/utils.js

@@ -197,7 +197,7 @@ var Utils = module.exports = {
       }
       }
     }
     }
 
 
-    return lodash.compact([text.join(' AND ')].concat(whereArgs))
+    return [text.join(' AND ')].concat(whereArgs.filter(function(w) { return w !== undefined && w !== null }))
   },
   },
   getWhereLogic: function(logic) {
   getWhereLogic: function(logic) {
     switch (logic) {
     switch (logic) {

test/dao-factory.test.js

@@ -21,7 +21,8 @@ describe(Support.getTestDialectTeaser("DAOFactory"), function () {
       secretValue:  DataTypes.STRING,
       secretValue:  DataTypes.STRING,
       data:         DataTypes.STRING,
       data:         DataTypes.STRING,
       intVal:       DataTypes.INTEGER,
       intVal:       DataTypes.INTEGER,
-      theDate:      DataTypes.DATE
+      theDate:      DataTypes.DATE,
+      aBool:        DataTypes.BOOLEAN
     })
     })
     this.User.sync({ force: true }).success(function() {
     this.User.sync({ force: true }).success(function() {
       done()
       done()
@@ -999,6 +1000,70 @@ describe(Support.getTestDialectTeaser("DAOFactory"), function () {
         })
         })
     })
     })
 
 
+    it('should be able to handle false/true values just fine...', function(done) {
+      var User = this.User
+      User.bulkCreate([
+        {username: 'boo5', aBool: false},
+        {username: 'boo6', aBool: true}
+      ]).success(function() {
+        User.all({where: ['"aBool" = ?', false]}).success(function(users) {
+          expect(users).to.have.length(1)
+          expect(users[0].username).to.equal('boo5')
+          User.all({where: ['"aBool" = ?', true]}).success(function(_users) {
+            expect(_users).to.have.length(1)
+            expect(_users[0].username).to.equal('boo6')
+            done()
+          })
+        })
+      })
+    })
+
+    it('should be able to handle false/true values through associations as well...', function(done) {
+      var User = this.User
+      var Passports = this.sequelize.define('Passports', {
+        isActive: Sequelize.BOOLEAN
+      })
+
+      User.hasMany(Passports)
+      Passports.belongsTo(User)
+
+      User.sync({ force: true }).success(function() {
+        Passports.sync({ force: true }).success(function() {
+          User.bulkCreate([
+            {username: 'boo5', aBool: false},
+            {username: 'boo6', aBool: true}
+          ]).success(function() {
+            Passports.bulkCreate([
+              {isActive: true},
+              {isActive: false}
+            ]).success(function() {
+              User.find(1).success(function(user) {
+                Passports.find(1).success(function(passport) {
+                  user.setPassports([passport]).success(function() {
+                    User.find(2).success(function(_user) {
+                      Passports.find(2).success(function(_passport) {
+                        _user.setPassports([_passport]).success(function() {
+                          _user.getPassports({where: ['"isActive" = ?', false]}).success(function(theFalsePassport) {
+                            user.getPassports({where: ['"isActive" = ?', true]}).success(function(theTruePassport) {
+                              expect(theFalsePassport).to.have.length(1)
+                              expect(theFalsePassport[0].isActive).to.be.false
+                              expect(theTruePassport).to.have.length(1)
+                              expect(theTruePassport[0].isActive).to.be.true
+                              done()
+                            })
+                          })
+                        })
+                      })
+                    })
+                  })
+                })
+              })
+            })
+          })
+        })
+      })
+    })
+
     it('should be able to retun a record with primaryKey being null for new inserts', function(done) {
     it('should be able to retun a record with primaryKey being null for new inserts', function(done) {
       var Session = this.sequelize.define('Session', {
       var Session = this.sequelize.define('Session', {
           token: { type: DataTypes.TEXT, allowNull: false },
           token: { type: DataTypes.TEXT, allowNull: false },