cyberpresidentvanellope/Bubberstation
1
0
Fork 0
mirror of https://github.com/Bubberstation/Bubberstation.git synced 2026-07-10 15:45:05 +01:00
Code Issues Packages Projects Releases Wiki Activity

adminhelp logs are now sanitized correctly (#72501)

Browse Source 
## About The Pull Request
Properly sanitizes adminhelp replies to match the sanitization given to
the initial adminhelp


![image](https://user-images.githubusercontent.com/41448081/210929570-e1d3c29b-93aa-4140-a52e-e15c4222c2e6.png)

## Why It's Good For The Game
LogBuddy breaks if any admin replies to a message over multiple lines,
so this fixes that

## Changelog
🆑
admin: Logbuddy should no longer fail to parse logs with multi-line
adminhelp replies
/🆑
This commit is contained in:
Zonespace
2023-01-08 13:09:12 -08:00
committed by GitHub
parent 53595ac96f
commit 862c079d4e
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
code/modules/admin/verbs/adminpm.dm
+4 -4
View File
@@ -549,9 +549,9 @@
if(ambiguious_recipient == EXTERNAL_PM_USER)
// Guard against the possibility of a null, since it'll runtime and spit out the contents of what should be a private ticket.
if(ticket)
log_admin_private("PM: Ticket #[ticket_id]: [our_name]->External: [raw_message]")
log_admin_private("PM: Ticket #[ticket_id]: [our_name]->External: [sanitize_text(trim(raw_message))]")
else
log_admin_private("PM: [our_name]->External: [raw_message]")
log_admin_private("PM: [our_name]->External: [sanitize_text(trim(raw_message))]")
for(var/client/lad in GLOB.admins)
to_chat(lad,
type = MESSAGE_TYPE_ADMINPM,
@@ -575,9 +575,9 @@
window_flash(recipient, ignorepref = TRUE)
if(ticket)
log_admin_private("PM: Ticket #[ticket_id]: [our_name]->[recipient_name]: [raw_message]")
log_admin_private("PM: Ticket #[ticket_id]: [our_name]->[recipient_name]: [sanitize_text(trim(raw_message))]")
else
log_admin_private("PM: [our_name]->[recipient_name]: [raw_message]")
log_admin_private("PM: [our_name]->[recipient_name]: [sanitize_text(trim(raw_message))]")
//we don't use message_admins here because the sender/receiver might get it too
for(var/client/lad in GLOB.admins)
if(lad.key == key || lad.key == recipient_key) //check to make sure client/lad isn't the sender or recipient