fixed bug where cid was hardcoded, and switched format back to storing group names, not slugs, #2887

public/src/admin/manage/category.js

@@ -202,7 +202,7 @@ define('admin/manage/category', [
 				privilege = checkboxEl.parent().attr('data-privilege'),
 				state = checkboxEl.prop('checked'),
 				rowEl = checkboxEl.parents('tr'),
-				member = rowEl.attr('data-group-slug') || rowEl.attr('data-uid');
+				member = rowEl.attr('data-group-name') || rowEl.attr('data-uid');
 
 			if (member) {
 				socket.emit('admin.categories.setPrivilege', {
@@ -225,7 +225,7 @@ define('admin/manage/category', [
 	};
 
 	Category.refreshPrivilegeTable = function() {
-		socket.emit('admin.categories.getPrivilegeSettings', 2, function(err, privileges) {
+		socket.emit('admin.categories.getPrivilegeSettings', ajaxify.variables.get('cid'), function(err, privileges) {
 			if (err) {
 				return app.alertError(err.message);
 			}

src/privileges/categories.js

@@ -92,12 +92,11 @@ module.exports = function(privileges) {
 							}).map(function(member) {
 								member.privileges = {};
 								for(var x=0,numPrivs=privileges.length;x<numPrivs;x++) {
-									member.privileges[privileges[x]] = memberSets[x].indexOf(member.slug) !== -1
+									member.privileges[privileges[x]] = memberSets[x].indexOf(member.name) !== -1
 								}
 
 								return {
 									name: member.name,
-									slug: member.slug,
 									memberCount: member.memberCount,
 									privileges: member.privileges,
 								};

src/views/admin/partials/categories/privileges.tpl

@@ -31,7 +31,7 @@
 								<!-- END privileges.labels.groups -->
 							</tr>
 							<!-- BEGIN privileges.groups -->
-							<tr data-group-slug="{privileges.groups.slug}">
+							<tr data-group-name="{privileges.groups.name}">
 								<td>{privileges.groups.name}</td>
 								{function.spawnPrivilegeStates, privileges}
 							</tr>