From 9be47c6374310f177d79610394b1fbbd965367d8 Mon Sep 17 00:00:00 2001
From: GinjaNinja32 <ginjaninja32@gmail.com>
Date: Wed, 23 Sep 2015 13:03:11 +0100
Subject: [PATCH] Permabans do not expire, and we don't need to show expired
 when it's been manually lifted

---
 code/modules/admin/DB ban/functions.dm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/code/modules/admin/DB ban/functions.dm b/code/modules/admin/DB ban/functions.dm
index d07386c447..8e1ad3aa79 100644
--- a/code/modules/admin/DB ban/functions.dm	
+++ b/code/modules/admin/DB ban/functions.dm	
@@ -422,7 +422,7 @@ datum/admins/proc/DB_ban_unban_by_id(var/id)
 				var/cid = select_query.item[15]
 
 				// true if this ban has expired
-				var/auto = now > expiration // oh how I love ISO 8601 (ish) date strings
+				var/auto = (bantype in list("TEMPBAN", "JOB_TEMPBAN")) && now > expiration // oh how I love ISO 8601 (ish) date strings
 
 				var/lcolor = blcolor
 				var/dcolor = bdcolor
@@ -469,7 +469,7 @@ datum/admins/proc/DB_ban_unban_by_id(var/id)
 					output += "<tr bgcolor='[dcolor]'>"
 					output += "<td align='center' colspan='5' bgcolor=''><b>UNBANNED by admin [unbanckey] on [unbantime]</b></td>"
 					output += "</tr>"
-				if(auto)
+				else if(auto)
 					output += "<tr bgcolor='[dcolor]'>"
 					output += "<td align='center' colspan='5' bgcolor=''><b>EXPIRED at [expiration]</b></td>"
 					output += "</tr>"