diff --git a/src/database/redis/list.js b/src/database/redis/list.js index fb445573ff..f8108a194d 100644 --- a/src/database/redis/list.js +++ b/src/database/redis/list.js @@ -3,6 +3,9 @@ module.exports = function (redisClient, module) { module.listPrepend = function (key, value, callback) { callback = callback || function () {}; + if (!key) { + return callback(); + } redisClient.lpush(key, value, function (err) { callback(err); }); @@ -10,6 +13,9 @@ module.exports = function (redisClient, module) { module.listAppend = function (key, value, callback) { callback = callback || function () {}; + if (!key) { + return callback(); + } redisClient.rpush(key, value, function (err) { callback(err); }); @@ -17,11 +23,17 @@ module.exports = function (redisClient, module) { module.listRemoveLast = function (key, callback) { callback = callback || function () {}; + if (!key) { + return callback(); + } redisClient.rpop(key, callback); }; module.listRemoveAll = function (key, value, callback) { callback = callback || function () {}; + if (!key) { + return callback(); + } redisClient.lrem(key, 0, value, function (err) { callback(err); }); @@ -29,6 +41,9 @@ module.exports = function (redisClient, module) { module.listTrim = function (key, start, stop, callback) { callback = callback || function () {}; + if (!key) { + return callback(); + } redisClient.ltrim(key, start, stop, function (err) { callback(err); }); @@ -36,6 +51,9 @@ module.exports = function (redisClient, module) { module.getListRange = function (key, start, stop, callback) { callback = callback || function () {}; + if (!key) { + return callback(); + } redisClient.lrange(key, start, stop, callback); }; }; diff --git a/test/database/list.js b/test/database/list.js index 1f5d72c6c3..7d00df3e8d 100644 --- a/test/database/list.js +++ b/test/database/list.js @@ -98,7 +98,7 @@ describe('List methods', function () { }); }); - it('should not get anyhing if key is falsy', function (done) { + it('should not get anything if key is falsy', function (done) { db.getListRange(null, 0, -1, function (err, data) { assert.ifError(err); assert.equal(data, undefined);