refactor userPrivileges and groupPrivileges

src/privileges/categories.js

@@ -350,59 +350,22 @@ module.exports = function (privileges) {
 	};
 
 	privileges.categories.userPrivileges = function (cid, uid, callback) {
-		async.parallel({
+		var tasks = {};
-			find: async.apply(groups.isMember, uid, 'cid:' + cid + ':privileges:find'),
+
-			read: function (next) {
+		privileges.userPrivilegeList.forEach(function (privilege) {
-				groups.isMember(uid, 'cid:' + cid + ':privileges:read', next);
+			tasks[privilege] = async.apply(groups.isMember, uid, 'cid:' + cid + ':privileges:' + privilege);
-			},
+		});
-			'topics:create': function (next) {
+
-				groups.isMember(uid, 'cid:' + cid + ':privileges:topics:create', next);
+		async.parallel(tasks, callback);
-			},
-			'topics:read': function (next) {
-				groups.isMember(uid, 'cid:' + cid + ':privileges:topics:read', next);
-			},
-			'topics:reply': function (next) {
-				groups.isMember(uid, 'cid:' + cid + ':privileges:topics:reply', next);
-			},
-			'posts:edit': function (next) {
-				groups.isMember(uid, 'cid:' + cid + ':privileges:posts:edit', next);
-			},
-			'posts:delete': function (next) {
-				groups.isMember(uid, 'cid:' + cid + ':privileges:posts:delete', next);
-			},
-			'topics:delete': function (next) {
-				groups.isMember(uid, 'cid:' + cid + ':privileges:topics:delete', next);
-			},
-			mods: function (next) {
-				user.isModerator(uid, cid, next);
-			},
-		}, callback);
 	};
 
 	privileges.categories.groupPrivileges = function (cid, groupName, callback) {
-		async.parallel({
+		var tasks = {};
-			'groups:find': async.apply(groups.isMember, groupName, 'cid:' + cid + ':privileges:groups:find'),
+
-			'groups:read': function (next) {
+		privileges.groupPrivilegeList.forEach(function (privilege) {
-				groups.isMember(groupName, 'cid:' + cid + ':privileges:groups:read', next);
+			tasks[privilege] = async.apply(groups.isMember, groupName, 'cid:' + cid + ':privileges:' + privilege);
-			},
+		});
-			'groups:topics:create': function (next) {
+
-				groups.isMember(groupName, 'cid:' + cid + ':privileges:groups:topics:create', next);
+		async.parallel(tasks, callback);
-			},
-			'groups:topics:reply': function (next) {
-				groups.isMember(groupName, 'cid:' + cid + ':privileges:groups:topics:reply', next);
-			},
-			'groups:posts:edit': function (next) {
-				groups.isMember(groupName, 'cid:' + cid + ':privileges:groups:posts:edit', next);
-			},
-			'groups:posts:delete': function (next) {
-				groups.isMember(groupName, 'cid:' + cid + ':privileges:groups:posts:delete', next);
-			},
-			'groups:topics:delete': function (next) {
-				groups.isMember(groupName, 'cid:' + cid + ':privileges:groups:topics:delete', next);
-			},
-			'groups:topics:read': function (next) {
-				groups.isMember(groupName, 'cid:' + cid + ':privileges:groups:topics:read', next);
-			},
-		}, callback);
 	};
 };

test/categories.js

@@ -640,7 +640,6 @@ describe('Categories', function () {
 				assert.ifError(err);
 				assert.deepEqual(data, {
 					find: false,
-					mods: false,
 					'posts:delete': false,
 					read: false,
 					'topics:reply': false,
@@ -648,6 +647,10 @@ describe('Categories', function () {
 					'topics:create': false,
 					'topics:delete': false,
 					'posts:edit': false,
+					'upload:post:file': false,
+					'upload:post:image': false,
+					purge: false,
+					moderate: false,
 				});
 
 				done();
@@ -666,6 +669,10 @@ describe('Categories', function () {
 					'groups:posts:delete': true,
 					'groups:read': true,
 					'groups:topics:read': true,
+					'groups:upload:post:file': false,
+					'groups:upload:post:image': true,
+					'groups:purge': false,
+					'groups:moderate': false,
 				});
 
 				done();