diff --git a/code/modules/admin/topic.dm b/code/modules/admin/topic.dm
index 2aa11ed11ae5..a4503f34b79e 100644
--- a/code/modules/admin/topic.dm
+++ b/code/modules/admin/topic.dm
@@ -2043,15 +2043,15 @@
/atom/proc/extra_admin_link()
return
-/mob/extra_admin_link(var/source)
+/mob/extra_admin_link(source)
if(client && eyeobj)
- return "|EYE"
+ return "|EYE"
-/mob/observer/dead/extra_admin_link(var/source)
- if(mind && mind.current)
- return "|BDY"
+/mob/observer/dead/extra_admin_link(source)
+ if(mind?.current)
+ return "|BDY"
-/proc/admin_jump_link(var/atom/target, var/source)
+/proc/admin_jump_link(atom/target, source)
if(!target) return
// The way admin jump links handle their src is weirdly inconsistent...
if(istype(source, /datum/admins))
@@ -2059,5 +2059,5 @@
else
source = "_src_=holder"
- . = "JMP"
+ . = "JMP"
. += target.extra_admin_link(source)
diff --git a/code/modules/client/verbs/ooc.dm b/code/modules/client/verbs/ooc.dm
index 4bb061d0d29f..da022b865618 100644
--- a/code/modules/client/verbs/ooc.dm
+++ b/code/modules/client/verbs/ooc.dm
@@ -81,15 +81,15 @@
if(!mob)
return
- if(!holder)
+ if(isnull(holder))
if(!config_legacy.ooc_allowed)
- to_chat(src, "OOC is globally muted.")
+ to_chat(src, SPAN_DANGER("OOC is globally muted."))
return
if(!config_legacy.dooc_allowed && (mob.stat == DEAD))
- to_chat(usr, "OOC for dead mobs has been turned off.")
+ to_chat(usr, SPAN_DANGER("OOC for dead mobs has been turned off."))
return
if(prefs.muted & MUTE_OOC)
- to_chat(src, "You cannot use OOC (muted).")
+ to_chat(src, SPAN_DANGER("You cannot use OOC (muted)."))
return
if(QDELETED(src))
@@ -103,7 +103,7 @@
msg = emoji_parse(msg)
- if(((msg[1] in list(".",";",":","#")) || findtext_char(msg, "say", 1, 5))) //SSticker.HasRoundStarted() &&
+ if(SSticker.HasRoundStarted() && ((msg[1] in list(".",";",":","#")) || findtext_char(msg, "say", 1, 5)))
if(alert("Your message \"[raw_msg]\" looks like it was meant for in game communication, say it in OOC?", "Meant for OOC?", "No", "Yes") != "Yes")
return
@@ -111,14 +111,14 @@
if(handle_spam_prevention(MUTE_OOC))
return
if(findtext(msg, "byond://"))
- to_chat(src, "Advertising other servers is not allowed.")
+ to_chat(src, SPAN_BOLDANNOUNCE("Advertising other servers is not allowed."))
log_admin("[key_name(src)] has attempted to advertise in OOC: [msg]")
message_admins("[key_name_admin(src)] has attempted to advertise in OOC: [msg]")
return
if(!get_preference_toggle(/datum/game_preference_toggle/chat/ooc))
- to_chat(src, "You have OOC muted.")
+ to_chat(src, SPAN_DANGER("You have OOC muted."))
return
log_ooc(raw_msg, src)
@@ -142,7 +142,7 @@
var/effective_color = holder && preferences.get_entry(/datum/game_preference_entry/simple_color/admin_ooc_color)
- for(var/client/target in GLOB.clients)
+ for(var/client/target as anything in GLOB.clients)
if(!target.initialized)
continue
@@ -204,7 +204,7 @@
to_chat(src, "You cannot use OOC (muted).")
return
if(findtext(msg, "byond://"))
- to_chat(src, "Advertising other servers is not allowed.")
+ to_chat(src, SPAN_BOLDANNOUNCE("Advertising other servers is not allowed."))
log_admin("[key_name(src)] has attempted to advertise in OOC: [msg]")
message_admins("[key_name_admin(src)] has attempted to advertise in OOC: [msg]")
return