From c4176beb3ee3ef23d0782c192c07dbe02cb0faf4 Mon Sep 17 00:00:00 2001
From: Cael_Aislinn <cael_aislinn@yahoo.com.au>
Date: Mon, 18 Feb 2013 00:56:53 +1000
Subject: [PATCH] some more useability tweaks to adminwho/modwho

Signed-off-by: Cael_Aislinn <cael_aislinn@yahoo.com.au>
---
 code/game/verbs/who.dm | 28 ++++++++++++++++++----------
 1 file changed, 18 insertions(+), 10 deletions(-)

diff --git a/code/game/verbs/who.dm b/code/game/verbs/who.dm
index a5ce1f7b792..1608a81a748 100644
--- a/code/game/verbs/who.dm
+++ b/code/game/verbs/who.dm
@@ -46,11 +46,12 @@
 	set category = "Admin"
 	set name = "Adminwho"
 
-	var/msg = "<b>Current Admins:</b>\n"
+	var/msg = ""
 	var/num_mods_online = 0
+	var/num_admins_online = 0
 	if(holder)
 		for(var/client/C in admins)
-			if(C.holder.rank != "Moderator")
+			if(R_ADMIN & C.holder.rights || !(R_MOD & C.holder.rights))
 				msg += "\t[C] is a [C.holder.rank]"
 
 				if(C.holder.fakekey)
@@ -66,16 +67,20 @@
 				if(C.is_afk())
 					msg += " (AFK)"
 				msg += "\n"
+
+				num_admins_online++
 			else
 				num_mods_online++
 	else
 		for(var/client/C in admins)
-			if(C.holder.rank != "Moderator")
+			if(R_ADMIN & C.holder.rights || !(R_MOD & C.holder.rights))
 				if(!C.holder.fakekey)
 					msg += "\t[C] is a [C.holder.rank]\n"
+					num_admins_online++
 			else
 				num_mods_online++
 
+	msg = "<b>Current Admins ([num_admins_online]):</b>\n" + msg
 	msg += "<b>There are also [num_mods_online] moderators online.</b> To view online moderators, type 'modwho'\n"
 	src << msg
 
@@ -83,11 +88,14 @@
 	set category = "Admin"
 	set name = "Modwho"
 
-	var/msg = "<b>Current Moderators:</b>\n"
+	var/msg = ""
 	var/num_admins_online = 0
+	var/num_mods_online = 0
 	if(holder)
 		for(var/client/C in admins)
-			if(C.holder.rank == "Moderator")
+			if(R_ADMIN & C.holder.rights || !(R_MOD & C.holder.rights))
+				num_admins_online++
+			else
 				msg += "\t[C] is a [C.holder.rank]"
 
 				if(isobserver(C.mob))
@@ -100,14 +108,14 @@
 				if(C.is_afk())
 					msg += " (AFK)"
 				msg += "\n"
-			else
-				num_admins_online++
+				num_mods_online++
 	else
 		for(var/client/C in admins)
-			if(C.holder.rank == "Moderator")
-				msg += "\t[C] is a [C.holder.rank]\n"
-			else
+			if(R_ADMIN & C.holder.rights || !(R_MOD & C.holder.rights))
 				num_admins_online++
+			else
+				msg += "\t[C] is a [C.holder.rank]\n"
 
+	msg = "<b>Current Moderators ([num_mods_online]):</b>\n" + msg
 	msg += "<b>There are also [num_admins_online] admins online.</b> To view online admins, type 'adminwho'\n"
 	src << msg