Merge branch 'master' into develop

This commit is contained in:
Julian Lam
2017-02-09 12:53:07 -05:00
11 changed files with 96 additions and 34 deletions

View File

@@ -350,11 +350,11 @@ var privileges = require('./privileges');
Categories.filterIgnoringUids = function (cid, uids, callback) {
async.waterfall([
function (next) {
db.sortedSetScores('cid:' + cid + ':ignorers', uids, next);
db.isSortedSetMembers('cid:' + cid + ':ignorers', uids, next);
},
function (scores, next) {
function (isIgnoring, next) {
var readingUids = uids.filter(function (uid, index) {
return uid && !!scores[index];
return uid && !isIgnoring[index];
});
next(null, readingUids);
}

View File

@@ -153,9 +153,9 @@ module.exports = function (Topics) {
function (next) {
db.isSetMembers('tid:' + tid + ':ignorers', uids, next);
},
function (isMembers, next) {
function (isIgnoring, next) {
var readingUids = uids.filter(function (uid, index) {
return uid && isMembers[index];
return uid && !isIgnoring[index];
});
next(null, readingUids);
}