From d2f41a297bfe21c03c4710684cd43bbf34ef9739 Mon Sep 17 00:00:00 2001
From: Adri <33333517+Miraviel@users.noreply.github.com>
Date: Wed, 8 Mar 2023 05:43:47 +0100
Subject: [PATCH] Adds more robust labeller/rename logging (#20474)

* Adds more robust labeler logging

* Logs rename_interactive(), formatting

* Adds FLW link
---
 code/__DEFINES/misc_defines.dm        | 2 +-
 code/game/atoms.dm                    | 7 +++++++
 code/modules/paperwork/handlabeler.dm | 2 +-
 3 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/code/__DEFINES/misc_defines.dm b/code/__DEFINES/misc_defines.dm
index 92035487ad8..6dee5bb4c27 100644
--- a/code/__DEFINES/misc_defines.dm
+++ b/code/__DEFINES/misc_defines.dm
@@ -375,7 +375,7 @@
 #define EXPLOSION_BLOCK_PROC -1
 
 // Defines for investigate to prevent typos and for styling
-#define INVESTIGATE_LABEL "labels"
+#define INVESTIGATE_RENAME "renames"
 
 #define INVESTIGATE_BOMB "bombs"
 
diff --git a/code/game/atoms.dm b/code/game/atoms.dm
index bb329df6b24..50c6db6a583 100644
--- a/code/game/atoms.dm
+++ b/code/game/atoms.dm
@@ -1226,6 +1226,13 @@ GLOBAL_LIST_EMPTY(blood_splatter_icons)
 
 
 	t = sanitize(copytext(t, 1, MAX_NAME_LEN))
+
+	// Logging
+	var/logged_name = initial(name)
+	if(t)
+		logged_name = "[use_prefix ? "[prefix][t]" : t]"
+	investigate_log("[key_name(user)] ([ADMIN_FLW(user,"FLW")]) renamed \"[src]\" ([ADMIN_VV(src, "VV")]) as \"[logged_name]\".", INVESTIGATE_RENAME)
+
 	if(actually_rename)
 		if(t == "")
 			name = "[initial(name)]"
diff --git a/code/modules/paperwork/handlabeler.dm b/code/modules/paperwork/handlabeler.dm
index ff317465b18..e85bc133506 100644
--- a/code/modules/paperwork/handlabeler.dm
+++ b/code/modules/paperwork/handlabeler.dm
@@ -29,7 +29,7 @@
 
 	user.visible_message("<span class='notice'>[user] labels [A] as [label].</span>", \
 						"<span class='notice'>You label [A] as [label].</span>")
-	investigate_log("[key_name(user)] labelled [A] as [label].", INVESTIGATE_LABEL) // Investigate goes BEFORE rename so the original name is preserved in the log
+	investigate_log("[key_name(user)] ([ADMIN_FLW(user,"FLW")]) labelled \"[A]\" ([ADMIN_VV(A, "VV")]) with \"[label]\" at [COORD(A.loc)] [ADMIN_JMP(A)].", INVESTIGATE_RENAME) // Investigate goes BEFORE rename so the original name is preserved in the log
 	A.AddComponent(/datum/component/label, label)
 	playsound(A, 'sound/items/handling/component_pickup.ogg', 20, TRUE)
 	labels_left--