Okay in relation to the previous commit r2391 muting is now tied to clients and not mobs. So if you get muted as a human and then die and change into a ghost you will no longer 'loose' your mute.

git-svn-id: http://tgstation13.googlecode.com/svn/trunk@2392 316c924e-a436-60f5-8080-3fe189b3f50e

code/modules/admin/admin.dm

@@ -377,10 +377,13 @@
 				if ((M.client && M.client.holder && (M.client.holder.level >= src.level)))
 					alert("You cannot perform this action. You must be of a higher administrative rank!", null, null, null, null, null)
 					return
-				M.muted = !M.muted
-				log_admin("[key_name(usr)] has [(M.muted ? "muted" : "voiced")] [key_name(M)].")
-				message_admins("\blue [key_name_admin(usr)] has [(M.muted ? "muted" : "voiced")] [key_name_admin(M)].", 1)
-				M << "You have been [(M.muted ? "muted" : "voiced")]. Please resolve this in adminhelp."
+				if(!M.client)
+					src << "This mob doesn't have a client tied to it."
+					return
+				M.client.muted = !M.client.muted
+				log_admin("[key_name(usr)] has [(M.client.muted ? "muted" : "voiced")] [key_name(M)].")
+				message_admins("\blue [key_name_admin(usr)] has [(M.client.muted ? "muted" : "voiced")] [key_name_admin(M)].", 1)
+				M << "You have been [(M.client.muted ? "muted" : "voiced")]. Please resolve this in adminhelp."
 	if (href_list["mute_complete"])
 		if ((src.rank in list( "Moderator", "Temporary Admin", "Admin Candidate", "Trial Admin", "Badmin", "Game Admin", "Game Master"  )))
 			var/mob/M = locate(href_list["mute_complete"])
@@ -388,10 +391,13 @@
 				if ((M.client && M.client.holder && (M.client.holder.level >= src.level)))
 					alert("You cannot perform this action. You must be of a higher administrative rank!", null, null, null, null, null)
 					return
-				M.muted_complete = !M.muted_complete
-				log_admin("[key_name(usr)] has [(M.muted_complete ? "completely muted" : "voiced (complete)")] [key_name(M)].")
-				message_admins("\blue [key_name_admin(usr)] has [(M.muted_complete ? "completely muted" : "voiced (complete)")] [key_name_admin(M)].", 1)
-				M << "You have been [(M.muted_complete ? "completely muted" : "voiced (complete)")]. You are unable to speak or even adminhelp"
+				if(!M.client)
+					src << "This mob doesn't have a client tied to it."
+					return
+				M.client.muted_complete = !M.client.muted_complete
+				log_admin("[key_name(usr)] has [(M.client.muted_complete ? "completely muted" : "voiced (complete)")] [key_name(M)].")
+				message_admins("\blue [key_name_admin(usr)] has [(M.client.muted_complete ? "completely muted" : "voiced (complete)")] [key_name_admin(M)].", 1)
+				M << "You have been [(M.client.muted_complete ? "completely muted" : "voiced (complete)")]. You are unable to speak or even adminhelp"
 
 	if (href_list["c_mode"])
 		if ((src.rank in list( "Temporary Admin", "Admin Candidate", "Trial Admin", "Badmin", "Game Admin", "Game Master"  )))
@@ -1689,8 +1695,11 @@
 		else
 			foo += text("<B>Hasn't Entered Game</B> | ")
 		foo += text("<A href='?src=\ref[src];forcespeech=\ref[M]'>Forcesay</A> | ")
-		foo += text("<A href='?src=\ref[src];mute2=\ref[M]'>Mute: [(M.muted ? "Muted" : "Voiced")]</A> | ")
-		foo += text("<A href='?src=\ref[src];mute_complete=\ref[M]'>Complete mute: [(M.muted ? "Completely Muted" : "Voiced")]</A> | ")
+		if(M.client)
+			foo += text("<A href='?src=\ref[src];mute2=\ref[M]'>Mute: [(M.client.muted ? "Muted" : "Voiced")]</A> | ")
+			foo += text("<A href='?src=\ref[src];mute_complete=\ref[M]'>Complete mute: [(M.client.muted ? "Completely Muted" : "Voiced")]</A> | ")
+		else
+			foo += "Mute unavailable - no client"
 		foo += text("<A href='?src=\ref[src];boot2=\ref[M]'>Boot</A>")
 	foo += text("<br>")
 	foo += text("<A href='?src=\ref[src];jumpto=\ref[M]'>Jump to</A> | ")

code/modules/admin/verbs/adminhelp.dm

@@ -10,7 +10,7 @@
 	if (!msg)
 		return
 
-	if (usr.muted_complete)
+	if (usr.client && usr.client.muted_complete)
 		return
 
 	for (var/mob/M in world)

code/modules/admin/verbs/adminsay.dm

@@ -9,7 +9,8 @@
 		src << "Only administrators may use this command."
 		return
 
-	if (!src.mob || src.mob.muted || src.mob.muted_complete)
+	if (src.muted || src.muted_complete)
+		src << "You are muted."
 		return
 
 	msg = copytext(sanitize(msg), 1, MAX_MESSAGE_LEN)

code/modules/admin/verbs/deadsay.dm

@@ -8,7 +8,7 @@
 		return
 	if(!src.mob)
 		return
-	if(src.mob.muted || src.mob.muted_complete)
+	if(src.muted || src.muted_complete)
 		return
 
 	msg = copytext(sanitize(msg), 1, MAX_MESSAGE_LEN)

code/modules/admin/verbs/pray.dm

@@ -10,7 +10,7 @@
 	if (!msg)
 		return
 
-	if (usr.muted || src.muted_complete)
+	if (usr.client && (usr.client.muted || usr.client.muted_complete))
 		return
 
 	var/icon/cross = icon('storage.dmi',"bible")

code/modules/admin/verbs/randomverbs.dm

@@ -91,7 +91,7 @@
 		src << "Only administrators may use this command."
 		return
 	if(M)
-		if(src.mob.muted_complete)
+		if(src.muted_complete)
 			src << "You are muted have a nice day"
 			return
 		if (!( ismob(M) ))
@@ -142,12 +142,15 @@
 	if (M.client && M.client.holder && (M.client.holder.level >= holder.level))
 		alert("You cannot perform this action. You must be of a higher administrative rank!", null, null, null, null, null)
 		return
-	M.muted = !M.muted
+	if(!M.client)
+		src << "This mob doesn't have a client tied to it."
+		return
+	M.client.muted = !M.client.muted
 
-	log_admin("[key_name(src)] has [(M.muted ? "muted" : "voiced")] [key_name(M)].")
-	message_admins("[key_name_admin(src)] has [(M.muted ? "muted" : "voiced")] [key_name_admin(M)].", 1)
+	log_admin("[key_name(src)] has [(M.client.muted ? "muted" : "voiced")] [key_name(M)].")
+	message_admins("[key_name_admin(src)] has [(M.client.muted ? "muted" : "voiced")] [key_name_admin(M)].", 1)
 
-	M << "You have been [(M.muted ? "muted" : "voiced")]."
+	M << "You have been [(M.client.muted ? "muted" : "voiced")]."
 
 
 /client/proc/cmd_admin_add_random_ai_law()