From b3e7e256314c50112d527db9cff62f247bdbf5d0 Mon Sep 17 00:00:00 2001
From: Jordie <Jordie0608@gmail.com>
Date: Mon, 1 Sep 2014 20:39:12 +1000
Subject: [PATCH 1/2] Logging to add/remove memo

---
 code/modules/admin/admin_memo.dm | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/code/modules/admin/admin_memo.dm b/code/modules/admin/admin_memo.dm
index b3f9e451ee3..c5a259103f9 100644
--- a/code/modules/admin/admin_memo.dm
+++ b/code/modules/admin/admin_memo.dm
@@ -21,13 +21,15 @@
 			if(null)
 				return
 			if("")
+				message_admins(<span class='admin'>"[src.ckey] removed their own Memo</span>")
+				log_admin("[src.ckey] removed their own Memo")
 				F.dir.Remove(ckey)
-				src << "<b>Memo removed</b>"
 				return
 		if( findtext(memo,"<script",1,0) )
 			return
 		F[ckey] << "[key] on [time2text(world.realtime,"(DDD) DD MMM hh:mm")]<br>[memo]"
 		message_admins("[key] set an admin memo:<br>[memo]")
+		log_admin("[key] set an admin memo:[memo]")
 
 //show all memos
 /client/proc/admin_memo_show()
@@ -35,7 +37,7 @@
 		var/savefile/F = new(MEMOFILE)
 		if(F)
 			for(var/ckey in F.dir)
-				src << "<center><span class='motd'><b>Admin Memo</b><i> by [F[ckey]]</i></span></center>"
+				src << "<center><span class='motd'><span class='prefix'>Admin Memo</span><span class='emote'> by [F[ckey]]</span></span></center>"
 
 //delete your own or somebody else's memo
 /client/proc/admin_memo_delete()
@@ -47,8 +49,9 @@
 		else
 			ckey = src.ckey
 		if(ckey)
+			message_admins("<span class='admin'>[src.ckey] removed [ckey]'s Memo.</span>")
+			log_admin("[src.ckey] removed Memo created by [ckey]: [memo].")
 			F.dir.Remove(ckey)
-			src << "<b>Removed Memo created by [ckey].</b>"
 
 #undef MEMOFILE
-#undef ENABLE_MEMOS
\ No newline at end of file
+#undef ENABLE_MEMOS

From a72b0dddadaf239c0090e86b2560d85f0b6ce262 Mon Sep 17 00:00:00 2001
From: Jordie0608 <Jordie0608@gmail.com>
Date: Mon, 1 Sep 2014 22:27:03 +1000
Subject: [PATCH 2/2] Made this actually work

---
 code/modules/admin/admin_memo.dm | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/code/modules/admin/admin_memo.dm b/code/modules/admin/admin_memo.dm
index c5a259103f9..18b9b030ed1 100644
--- a/code/modules/admin/admin_memo.dm
+++ b/code/modules/admin/admin_memo.dm
@@ -21,7 +21,7 @@
 			if(null)
 				return
 			if("")
-				message_admins(<span class='admin'>"[src.ckey] removed their own Memo</span>")
+				message_admins("<span class='admin'>[src.ckey] removed their own Memo</span>")
 				log_admin("[src.ckey] removed their own Memo")
 				F.dir.Remove(ckey)
 				return
@@ -49,9 +49,10 @@
 		else
 			ckey = src.ckey
 		if(ckey)
-			message_admins("<span class='admin'>[src.ckey] removed [ckey]'s Memo.</span>")
-			log_admin("[src.ckey] removed Memo created by [ckey]: [memo].")
-			F.dir.Remove(ckey)
+			for(var/memo in F.dir)
+				message_admins("<span class='admin'>[src.ckey] removed [ckey]'s Memo.</span>")
+				log_admin("[src.ckey] removed Memo created by [F[memo]].")
+				F.dir.Remove(ckey)
 
 #undef MEMOFILE
 #undef ENABLE_MEMOS