From 9794feef4bb41b8109eb77bfb2d47426334e39b3 Mon Sep 17 00:00:00 2001
From: harryob <55142896+harryob@users.noreply.github.com>
Date: Sat, 10 Feb 2024 13:36:40 +0000
Subject: [PATCH] eek, sdmm didn't pick this up (though it's obvious)

---
 code/datums/entities/player.dm | 10 +++++-----
 code/modules/admin/NewBan.dm   |  6 +++---
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/code/datums/entities/player.dm b/code/datums/entities/player.dm
index 831f4a1fd8a5..c4e5669fd808 100644
--- a/code/datums/entities/player.dm
+++ b/code/datums/entities/player.dm
@@ -317,12 +317,12 @@ BSQL_PROTECT_DATUM(/datum/entity/player)
 	if(banner)
 		permaban_admin_id = banner.id
 		message_admins("[key_name_admin(banner.owning_client)] has permanently banned [ckey] for '[reason]'.")
-		var/datum/tgs_chat_embed/field/reason
+		var/datum/tgs_chat_embed/field/reason_embed
 		if(internal_reason)
-			reason = new()
-			reason.name = "Permaban Reason"
-			reason.value = internal_reason
-		important_message_external("[banner.owning_client] has permanently banned [ckey] for '[reason]'.", "Permaban Placed", list(reason))
+			reason_embed = new()
+			reason_embed.name = "Permaban Reason"
+			reason_embed.value = internal_reason
+		important_message_external("[banner.owning_client] has permanently banned [ckey] for '[reason]'.", "Permaban Placed", reason ? list(reason_embed) : null)
 
 		add_note("Permanently banned | [reason]", FALSE, NOTE_ADMIN, TRUE)
 		if(internal_reason)
diff --git a/code/modules/admin/NewBan.dm b/code/modules/admin/NewBan.dm
index e0452ac024a5..b231fad0ceb0 100644
--- a/code/modules/admin/NewBan.dm
+++ b/code/modules/admin/NewBan.dm
@@ -323,7 +323,7 @@ GLOBAL_DATUM(Banlist, /savefile)
 
 	log_admin("STICKYBAN: Identifier: [identifier] Reason: [reason] Message: [message] CKEYs: [english_list(impacted_ckeys)] IPs: [english_list(impacted_ips)] CIDs: [english_list(impacted_cids)]")
 	message_admins("[key_name_admin(src)] has added a new stickyban with the identifier '[identifier]'.")
-	var/datum/tgs_chat_embed/field/reason = new()
-	reason.name = "Stickyban Reason"
-	reason.value = reason
+	var/datum/tgs_chat_embed/field/reason_embed = new()
+	reason_embed.name = "Stickyban Reason"
+	reason_embed.value = reason
 	important_message_external("[src] has added a new stickyban with the identifier '[identifier]'.", "Stickyban Placed", list(reason))