Fix for redis adapter when finding records filtered with multiple att…

…ributes
1602 · Oct 9, 2012 · 075699e · 075699e
1 parent 01b9307
commit 075699e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/lib/adapters/redis2.js b/lib/adapters/redis2.js
@@ -60,7 +60,7 @@ var commands = Object.keys(redis.Multi.prototype).filter(function (n) {
 commands.forEach(function (cmd) {
 
     Client.prototype[cmd] = function (args, callback) {
-        
+
         var c = this._client, log;
 
         if (typeof args === 'string') {
@@ -389,6 +389,7 @@ BridgeToRedis.prototype.all = function all(model, filter, callback) {
         if (indexes && indexes.length) {
             innerSetUsed = true;
             if (indexes.length > 1) {
+                indexes.unshift(dest);
                 trans.sinterstore(indexes);
             } else {
                 dest = indexes[0];

diff --git a/test/common_test.js b/test/common_test.js
@@ -418,6 +418,16 @@ function testOrm(schema) {
 
     });
 
+
+    it('should find records filtered with multiple attributes', function (test) {
+        Post.create({title: 'title', content: 'content', published: true, date: 1}, function (err, post) {
+            Post.all({where: {title: 'title', date: 1}}, function (err, res) {
+                test.ok(res.length > 0, 'Exact match with string returns dataset');
+                test.done();
+            });
+        });
+    });
+
     it('should handle hasMany relationship', function (test) {
         User.create(function (err, u) {
             if (err) return console.log(err);