Merge pull request #29 from mansuleman/master
Fix bug with sorting by id in redis adapter
This commit is contained in:
commit
bc814b435c
|
@ -243,9 +243,11 @@ BridgeToRedis.prototype.all = function all(model, filter, callback) {
|
||||||
orders = [filter.order];
|
orders = [filter.order];
|
||||||
}
|
}
|
||||||
orders.forEach(function (key) {
|
orders.forEach(function (key) {
|
||||||
|
if (key !== 'id') {
|
||||||
if (props[key].type.name !== 'Number' && props[key].type.name !== 'Date') {
|
if (props[key].type.name !== 'Number' && props[key].type.name !== 'Date') {
|
||||||
allNumeric = false;
|
allNumeric = false;
|
||||||
}
|
}
|
||||||
|
}
|
||||||
sortCmd.push("BY", model + ":*->" + key);
|
sortCmd.push("BY", model + ":*->" + key);
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue