calculate topic post indices instead of querying db

This commit is contained in:
barisusakli
2015-02-07 00:12:47 -05:00
parent ff5eeb3a85
commit c859586269
3 changed files with 42 additions and 32 deletions

View File

@@ -29,15 +29,28 @@ module.exports = function(Topics) {
], callback);
};
Topics.getTopicPosts = function(tid, set, start, end, uid, reverse, callback) {
callback = callback || function() {};
posts.getPostsByTid(tid, set, start, end, uid, reverse, function(err, postData) {
async.parallel({
posts: function(next) {
posts.getPostsByTid(tid, set, start, end, uid, reverse, next);
},
postCount: function(next) {
Topics.getTopicField(tid, 'postcount', next);
}
}, function(err, results) {
if (err) {
return callback(err);
}
Topics.addPostData(postData, uid, callback);
var indices = Topics.calculatePostIndices(start, end, results.postCount, reverse);
results.posts.forEach(function(post, index) {
if (post) {
post.index = indices[index];
}
});
Topics.addPostData(results.posts, uid, callback);
});
};
@@ -103,9 +116,6 @@ module.exports = function(Topics) {
},
privileges: function(next) {
privileges.posts.get(pids, uid, next);
},
indices: function(next) {
posts.getPostIndices(postData, uid, next);
}
}, function(err, results) {
if (err) {
@@ -114,7 +124,6 @@ module.exports = function(Topics) {
postData.forEach(function(postObj, i) {
if (postObj) {
postObj.index = results.indices[i];
postObj.deleted = parseInt(postObj.deleted, 10) === 1;
postObj.user = parseInt(postObj.uid, 10) ? results.userData[postObj.uid] : _.clone(results.userData[postObj.uid]);
postObj.editor = postObj.editor ? results.editors[postObj.editor] : null;
@@ -141,6 +150,19 @@ module.exports = function(Topics) {
});
};
Topics.calculatePostIndices = function(start, end, postCount, reverse) {
var indices = [];
var count = end - start + 1;
for(var i=0; i<count; ++i) {
if (reverse) {
indices.push(postCount - (start + i + 1));
} else {
indices.push(start + i + 1);
}
}
return indices;
};
Topics.getLatestUndeletedPid = function(tid, callback) {
Topics.getLatestUndeletedReply(tid, function(err, pid) {
if (err) {