fix: disregard empty and/or fields in buildSelect

Author: abhilashmurthy

lib/postgresql.js

@@ -623,11 +623,13 @@ PostgreSQL.prototype._buildWhere = function(model, where) {
             branches.push(stmtForClause.sql);
           }
         }
-        stmt.merge({
-          sql: '(' + branches.join(' ' + key.toUpperCase() + ' ') + ')',
-          params: branchParams,
-        });
-        whereStmts.push(stmt);
+        if (branches.length > 0) {
+          stmt.merge({
+            sql: '(' + branches.join(' ' + key.toUpperCase() + ' ') + ')',
+            params: branchParams,
+          });
+          whereStmts.push(stmt);
+        }
         continue;
       }
       // The value is not an array, fall back to regular fields

test/postgresql.test.js

@@ -424,6 +424,15 @@ describe('postgresql connector', function() {
       });
     });
 
+    it('should disregard empty and/or fields', function(done) {
+      Post.find({where: {and: []}}, function(err, post) {
+        should.not.exist(err);
+        should.exist(post);
+        post.length.should.not.equal(0);
+        done();
+      });
+    });
+
     it('should preserve order of and/or in where', async function() {
       await Post.create({title: 'T3', content: 'C3', approved: false});
       // WHERE (title='T3' OR approved=false) AND (content='C2')