Merge pull request #5458 from brendonboshell/master

empty array  in where should return no data

lib/dialects/abstract/query-generator.js

@@ -2004,9 +2004,20 @@ var QueryGenerator = {
           return item && item.length;
         });
 
+        // $or: [] should return no data.
+        // $not of no restriction should also return no data
+        if ((key === '$or' || key === '$not') && value.length === 0) {
+          return '0 = 1';
+        }
+
         return value.length ? outerBinding + '('+value.join(binding)+')' : undefined;
       } else {
         value = self.whereItemsQuery(value, options, binding);
+
+        if ((key === '$or' || key === '$not') && !value) {
+          return '0 = 1';
+        }
+
         return value ? outerBinding + '('+value+')' : undefined;
       }
     }

test/unit/sql/where.test.js

@@ -248,6 +248,20 @@ suite(Support.getTestDialectTeaser('SQL'), function() {
             mssql: "([group_id] = 1 OR ([user_id] = 2 AND [role] = N'admin'))"
           });
         });
+
+        testsql('$or', [], {
+          default: "0 = 1"
+        });
+
+        testsql('$or', {}, {
+          default: "0 = 1"
+        });
+
+        test("sequelize.or()", function () {
+          expectsql(sql.whereItemQuery(undefined, this.sequelize.or()), {
+            default: "0 = 1"
+          });
+        });
       });
 
       suite('$and', function () {
@@ -313,6 +327,14 @@ suite(Support.getTestDialectTeaser('SQL'), function() {
         }, {
           default: 'NOT ([shared] = 1 AND ([group_id] = 1 OR [user_id] = 2))'
         });
+
+        testsql('$not', [], {
+          default: "0 = 1"
+        });
+
+        testsql('$not', {}, {
+          default: "0 = 1"
+        });
       });
     });