From cbb05429847ae882c71a5a4f0324b62f9454f002 Mon Sep 17 00:00:00 2001
From: Julian Lam <julian@designcreateplay.com>
Date: Tue, 26 May 2015 10:51:23 -0400
Subject: [PATCH] changed behaviour of privilege table so that groups without
 explicit privileges are not shown in the privilege table

---
 src/privileges/categories.js | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/src/privileges/categories.js b/src/privileges/categories.js
index 59b2f79a1d..4572de2e92 100644
--- a/src/privileges/categories.js
+++ b/src/privileges/categories.js
@@ -101,19 +101,25 @@ module.exports = function(privileges) {
 							groupNames.splice(0, 0, groupNames.splice(groupNames.indexOf('registered-users'), 1)[0]);
 							groupNames.splice(groupNames.indexOf('administrators'), 1);
 
-							var memberData = groupNames.filter(function(member) {
+							var memberPrivs, boolSet,
+								memberData = groupNames.filter(function(member) {
 								return member.indexOf(':privileges:') === -1;
 							}).map(function(member) {
-								var memberPrivs = {};
+								memberPrivs = {};
+								boolSet = [];	// Here, the boolSet is used as a quick way to determine whether a given group's privilege set is empty or not (see below)
 								for(var x=0,numPrivs=privileges.length;x<numPrivs;x++) {
-									memberPrivs[privileges[x]] = memberSets[x].indexOf(member) !== -1;
+									boolSet.push(memberPrivs[privileges[x]] = memberSets[x].indexOf(member) !== -1);
 								}
 
-								return {
-									name: member,
-									privileges: memberPrivs,
-								};
-							});
+								if (boolSet.some(Boolean)) {
+									return {
+										name: member,
+										privileges: memberPrivs,
+									};
+								} else {
+									return null;
+								}
+							}).filter(Boolean);
 
 							next(null, memberData);
 						});