removed extra isAdminOrMod check

src/categories/topics.js

@@ -5,57 +5,49 @@ var async = require('async');
 var db = require('../database');
 var topics = require('../topics');
 var plugins = require('../plugins');
-var privileges = require('../privileges');
 
 module.exports = function(Categories) {
 
 	Categories.getCategoryTopics = function(data, callback) {
-		async.parallel({
+		async.waterfall([
-			isAdminOrMod: function(next) {
+			function (next) {
-				privileges.categories.isAdminOrMod(data.cid, data.uid, next);
+				plugins.fireHook('filter:category.topics.prepare', data, next);
 			},
-			topics: function(next) {
+			function (data, next) {
-				async.waterfall([
+				Categories.getTopicIds(data.set, data.reverse, data.start, data.stop, next);
-					function(next) {
+			},
-						plugins.fireHook('filter:category.topics.prepare', data, next);
+			function (tids, next) {
-					},
+				topics.getTopicsByTids(tids, data.uid, next);
-					function(data, next) {
+			},
-						Categories.getTopicIds(data.set, data.reverse, data.start, data.stop, next);
+			function (topics, next) {
-					},
+				if (!Array.isArray(topics) || !topics.length) {
-					function(tids, next) {
+					return next(null, {topics: [], uid: data.uid});
-						topics.getTopicsByTids(tids, data.uid, next);
+				}
-					},
-					function(topics, next) {
-						if (!Array.isArray(topics) || !topics.length) {
-							return next(null, {topics: [], uid: data.uid});
-						}
 
-						for (var i=0; i<topics.length; ++i) {
+				for (var i=0; i<topics.length; ++i) {
-							topics[i].index = data.start + i;
+					topics[i].index = data.start + i;
-						}
+				}
 
-						plugins.fireHook('filter:category.topics.get', {topics: topics, uid: data.uid}, next);
+				plugins.fireHook('filter:category.topics.get', {topics: topics, uid: data.uid}, next);
-					},
+			},
-					function(results, next) {
+			function (results, next) {
-						next(null, results.topics);
+				next(null, {topics: results.topics, nextStart: data.stop + 1});
-					}
-				], next);
-			}
-		}, function(err, results) {
-			if (err) {
-				return callback(err);
 			}
+		], callback);
+	};
 
-			results.topics.forEach(function(topic) {
+	Categories.modifyTopicsByPrivilege = function(topics, privileges) {
-				if (!(!topic.deleted || results.isAdminOrMod || topic.isOwner)) {
+		if (!Array.isArray(topics) || !topics.length || privileges.isAdminOrMod) {
-					topic.title = '[[topic:topic_is_deleted]]';
+			return;
-					topic.slug = topic.tid;
+		}
-					topic.teaser = null;
-					topic.noAnchor = true;
-				}
-			});
 
-			callback(null, {topics: results.topics, nextStart: data.stop + 1});
+		topics.forEach(function(topic) {
+			if (topic.deleted && !topic.isOwner) {
+				topic.title = '[[topic:topic_is_deleted]]';
+				topic.slug = topic.tid;
+				topic.teaser = null;
+				topic.noAnchor = true;
+			}
 		});
 	};
 

src/controllers/category.js

@@ -111,6 +111,9 @@ categoryController.get = function(req, res, callback) {
 			categories.getCategoryById(payload, next);
 		},
 		function (categoryData, next) {
+
+			categories.modifyTopicsByPrivilege(categoryData.topics, userPrivileges);
+
 			if (categoryData.link) {
 				db.incrObjectField('category:' + categoryData.cid, 'timesClicked');
 				return res.redirect(categoryData.link);

src/controllers/topics.js

@@ -116,7 +116,7 @@ topicsController.get = function(req, res, callback) {
 				return callback();
 			}
 
-			topics.modifyByPrivilege(topicData.posts, userPrivileges);
+			topics.modifyPostsByPrivilege(topicData.posts, userPrivileges);
 
 			plugins.fireHook('filter:controllers.topic.get', {topicData: topicData, uid: req.uid}, next);
 		},

src/routes/feeds.js

@@ -49,7 +49,7 @@ function generateForTopic(req, res, callback) {
 			return callback(err);
 		}
 
-		topics.modifyByPrivilege(topicData.posts, userPrivileges);
+		topics.modifyPostsByPrivilege(topicData.posts, userPrivileges);
 
 		var description = topicData.posts.length ? topicData.posts[0].content : '';
 		var image_url = topicData.posts.length ? topicData.posts[0].picture : '';

src/socket.io/categories.js

@@ -119,6 +119,8 @@ SocketCategories.loadMore = function(socket, data, callback) {
 				return callback(err);
 			}
 
+			categories.modifyTopicsByPrivilege(data.topics, results.privileges);
+
 			data.privileges = results.privileges;
 			data.template = {
 				category: true,

src/socket.io/topics/infinitescroll.js

@@ -81,7 +81,7 @@ module.exports = function(SocketTopics) {
 				topicData['reputation:disabled'] = parseInt(meta.config['reputation:disabled'], 10) === 1;
 				topicData['downvote:disabled'] = parseInt(meta.config['downvote:disabled'], 10) === 1;
 
-				topics.modifyByPrivilege(topicData.posts, results.privileges);
+				topics.modifyPostsByPrivilege(topicData.posts, results.privileges);
 				callback(null, topicData);
 			});
 		});

src/topics/posts.js

@@ -138,7 +138,7 @@ module.exports = function(Topics) {
 		});
 	};
 
-	Topics.modifyByPrivilege = function(postData, topicPrivileges) {
+	Topics.modifyPostsByPrivilege = function(postData, topicPrivileges) {
 		postData.forEach(function(post) {
 			if (post) {
 				post.display_moderator_tools = topicPrivileges.isAdminOrMod || post.selfPost;