Merge branch 'master' into tgs-dmapi-update

.vscode/settings.json

@@ -17,14 +17,14 @@
 		"**/.pnp.*": true
 	},
 	"editor.codeActionsOnSave": {
-		"source.fixAll.eslint": true
+		"source.fixAll.eslint": "explicit"
 	},
 	"files.insertFinalNewline": true,
 	"gitlens.advanced.blame.customArguments": ["-w"],
 	"tgstationTestExplorer.project.resultsType": "json",
 	"[python]": {
 		"gitlens.codeLens.symbolScopes": ["!Module"],
-		"editor.wordBasedSuggestions": false,
+		"editor.wordBasedSuggestions": "off",
 		"editor.insertSpaces": true,
 		"editor.tabSize": 4
 	},

_maps/configs/independent_shetland.json

@@ -22,11 +22,6 @@
 			"officer": true,
 			"slots": 1
 		},
-		"Lieutenant": {
-			"outfit": "/datum/outfit/job/lieutenant",
-			"officer": true,
-			"slots": 1
-		},
 		"First Mate": {
 			"outfit": "/datum/outfit/job/head_of_personnel",
 			"officer": true,

_maps/shuttles/independent/independent_beluga.dmm

@@ -641,7 +641,6 @@
 /obj/item/clothing/head/beret/hop,
 /obj/item/radio/headset/heads/head_of_personnel,
 /obj/item/clothing/glasses/sunglasses/big,
-/obj/item/clothing/suit/toggle/lieutenant,
 /obj/item/clothing/suit/toggle/lawyer/burgundy,
 /obj/item/clothing/suit/toggle/lawyer/navy,
 /obj/item/clothing/head/hopcap,

_maps/shuttles/independent/independent_shetland.dmm

@@ -64,7 +64,6 @@
 /obj/structure/chair/comfy/shuttle{
 	dir = 4
 	},
-/obj/effect/landmark/start/lieutenant,
 /turf/open/floor/plasteel/dark,
 /area/ship/bridge)
 "aG" = (
@@ -1240,18 +1239,21 @@
 /obj/machinery/button/door{
 	id = "amogusdoors";
 	name = "Blast Door Control";
-	pixel_x = 5;
-	pixel_y = 5
+	pixel_x = 6;
+	pixel_y = 7
 	},
 /obj/structure/table/reinforced,
 /obj/machinery/button/door{
 	id = "amoguswindows";
 	name = "Window Lockdown";
-	pixel_x = -5;
-	pixel_y = -5
+	pixel_x = -7;
+	pixel_y = 7
 	},
 /obj/effect/turf_decal/corner/opaque/neutral/half,
 /obj/item/radio/intercom/directional/north,
+/obj/item/phone{
+	pixel_y = -4
+	},
 /turf/open/floor/plasteel/dark,
 /area/ship/bridge)
 "mG" = (
@@ -1395,16 +1397,22 @@
 /turf/open/floor/plasteel/dark,
 /area/ship/hallway/port)
 "oB" = (
-/obj/structure/table/reinforced,
-/obj/effect/turf_decal/corner/opaque/neutral/three_quarters{
-	dir = 4
-	},
-/obj/item/phone,
-/obj/item/megaphone/command{
-	pixel_x = 10
-	},
 /obj/machinery/airalarm/directional/south,
-/turf/open/floor/plasteel/dark,
+/obj/structure/closet/secure_closet{
+	icon_state = "hop";
+	name = "\proper first officer's locker";
+	req_access_txt = "57"
+	},
+/obj/item/clothing/under/rank/command/head_of_personnel,
+/obj/item/clothing/under/rank/command/head_of_personnel/skirt,
+/obj/item/clothing/suit/armor/vest/hop,
+/obj/item/clothing/head/beret/hop,
+/obj/item/gun/energy/e_gun/mini,
+/obj/item/clothing/shoes/laceup,
+/obj/item/radio/headset/headset_com/alt,
+/obj/item/storage/box/ids,
+/obj/item/storage/briefcase,
+/turf/open/floor/carpet/blue,
 /area/ship/bridge)
 "oC" = (
 /obj/structure/cable{
@@ -1615,15 +1623,18 @@
 "qE" = (
 /obj/machinery/light/small/directional/south,
 /obj/structure/table/reinforced,
-/obj/effect/turf_decal/corner/opaque/neutral/half{
-	dir = 1
-	},
 /obj/item/paper_bin,
 /obj/item/spacecash/bundle/c1000,
 /obj/item/spacecash/bundle/c1000,
 /obj/item/spacecash/bundle/c1000,
 /obj/item/spacecash/bundle/c1000,
 /obj/item/spacecash/bundle/c1000,
+/obj/item/megaphone/command{
+	pixel_x = 10
+	},
+/obj/effect/turf_decal/corner/opaque/neutral/three_quarters{
+	dir = 4
+	},
 /turf/open/floor/plasteel/dark,
 /area/ship/bridge)
 "qJ" = (
@@ -3621,11 +3632,22 @@
 	icon_state = "0-2"
 	},
 /obj/machinery/power/apc/auto_name/directional/north,
-/obj/structure/closet/secure_closet/true_lieutenant,
 /obj/machinery/light_switch{
 	pixel_x = 11;
 	pixel_y = 23
 	},
+/obj/structure/closet/secure_closet{
+	icon_state = "cap";
+	name = "\proper captain's locker";
+	req_access_txt = "20"
+	},
+/obj/item/clothing/under/rank/command/captain/suit,
+/obj/item/clothing/under/rank/command/captain/skirt,
+/obj/item/storage/backpack/captain,
+/obj/item/radio/headset/heads/captain/alt,
+/obj/item/clothing/gloves/color/captain,
+/obj/item/gun/energy/e_gun,
+/obj/item/clothing/shoes/laceup,
 /turf/open/floor/carpet/blue,
 /area/ship/bridge)
 "KN" = (
@@ -4063,12 +4085,6 @@
 /obj/effect/spawner/structure/window/reinforced,
 /turf/open/floor/plating,
 /area/ship/medical)
-"OD" = (
-/obj/structure/chair/comfy/shuttle{
-	dir = 4
-	},
-/turf/open/floor/plasteel/dark,
-/area/ship/bridge)
 "OE" = (
 /obj/machinery/atmospherics/components/unary/vent_pump/on/layer2{
 	dir = 1
@@ -6769,9 +6785,9 @@ dl
 LL
 TY
 Ft
-OD
+aw
 Lr
-OD
+aw
 xq
 TY
 LL

_maps/shuttles/nanotrasen/nanotrasen_delta.dmm

@@ -1095,10 +1095,6 @@
 /obj/item/clothing/suit/hooded/wintercoat/captain,
 /obj/item/clothing/under/rank/command/captain/nt,
 /obj/item/clothing/under/rank/command/captain/nt/skirt,
-/obj/item/clothing/suit/toggle/lieutenant{
-	desc = "Threadbare and uncharacteristically casual for Nanotrasen. Probably rebadged surplus from some colonial militia.";
-	name = "tattered captain's jacket"
-	},
 /obj/item/clothing/shoes/laceup,
 /obj/item/clothing/head/caphat/nt,
 /obj/item/stock_parts/cell/gun,

_maps/shuttles/nanotrasen/nanotrasen_gecko.dmm

@@ -3391,10 +3391,6 @@
 /obj/item/clothing/under/rank/command/captain/nt,
 /obj/item/clothing/head/caphat/nt,
 /obj/item/clothing/shoes/laceup,
-/obj/item/clothing/suit/toggle/lieutenant{
-	desc = "Threadbare and uncharacteristically casual for Nanotrasen. Probably rebadged surplus from some colonial militia.";
-	name = "tattered captain's jacket"
-	},
 /obj/item/clothing/glasses/sunglasses,
 /obj/effect/turf_decal/borderfloor{
 	dir = 1

_maps/shuttles/nanotrasen/nanotrasen_heron.dmm

@@ -4337,7 +4337,6 @@
 /obj/item/clothing/under/rank/security/officer/military/eng,
 /obj/item/clothing/suit/jacket/leather/duster,
 /obj/item/clothing/suit/jacket/miljacket,
-/obj/item/clothing/head/beret/lt,
 /obj/item/clothing/mask/bandana/skull,
 /obj/item/clothing/suit/armor/vest/marine,
 /obj/item/instrument/piano_synth/headphones/spacepods{
@@ -9392,7 +9391,6 @@
 /obj/item/clothing/under/rank/security/officer/military/eng,
 /obj/item/clothing/suit/jacket/leather/duster,
 /obj/item/clothing/suit/jacket/miljacket,
-/obj/item/clothing/head/beret/lt,
 /obj/item/clothing/suit/armor/vest/marine,
 /obj/item/instrument/piano_synth/headphones/spacepods{
 	pixel_x = -5;

_maps/shuttles/nanotrasen/nanotrasen_ranger.dmm

@@ -3009,7 +3009,6 @@
 /obj/item/clothing/shoes/jackboots,
 /obj/item/clothing/suit/jacket/miljacket,
 /obj/item/clothing/suit/jacket/leather/duster/command,
-/obj/item/clothing/head/beret/lt,
 /obj/item/storage/backpack/messenger/com,
 /obj/item/storage/backpack/satchel/cap,
 /obj/item/storage/backpack/captain,

code/__DEFINES/misc.dm

@@ -474,3 +474,7 @@ GLOBAL_LIST_INIT(ghost_others_options, list(GHOST_OTHERS_SIMPLE, GHOST_OTHERS_DE
 #define CUSTOM_HOLODECK_ONE (1<<1)
 #define CUSTOM_HOLODECK_TWO (1<<2)
 #define HOLODECK_DEBUG (1<<3)//you should never see this
+
+#define ROUND_END_NOT_DELAYED 0
+#define ROUND_END_DELAYED 1
+#define ROUND_END_TGS 2

code/__HELPERS/chat.dm

@@ -47,18 +47,17 @@ In TGS3 it will always be sent to all connected designated game chats.
 
 	var/datum/tgs_version/version = world.TgsVersion()
 	if(channel_tag == "" || version.suite == 3)
-		world.TgsTargetedChatBroadcast(message, FALSE)
+		world.TgsTargetedChatBroadcast(new /datum/tgs_message_content(message), FALSE)
 		return
 
 	var/list/channels_to_use = list()
-	for(var/I in world.TgsChatChannelInfo())
-		var/datum/tgs_chat_channel/channel = I
+	for(var/datum/tgs_chat_channel/channel as anything in world.TgsChatChannelInfo())
 		var/list/applicable_tags = splittext(channel.custom_tag, ",")
 		if(channel_tag in applicable_tags)
 			channels_to_use += channel
 
-	if(channels_to_use.len)
-		world.TgsChatBroadcast(message, channels_to_use)
+	if(length(channels_to_use))
+		world.TgsChatBroadcast(new /datum/tgs_message_content(message), channels_to_use)
 
 /**
  * Sends a message to TGS admin chat channels.
@@ -69,4 +68,4 @@ In TGS3 it will always be sent to all connected designated game chats.
 /proc/send2adminchat(category, message)
 	category = replacetext(replacetext(category, "\proper", ""), "\improper", "")
 	message = replacetext(replacetext(message, "\proper", ""), "\improper", "")
-	world.TgsTargetedChatBroadcast("[category] | [message]", TRUE)
+	world.TgsTargetedChatBroadcast(new /datum/tgs_message_content("[category] | [message]"), TRUE)

code/controllers/subsystem/ping.dm

@@ -13,8 +13,8 @@ SUBSYSTEM_DEF(ping)
 
 	var/list/currentrun = list()
 
-/datum/controller/subsystem/ping/stat_entry()
-	..("P:[GLOB.clients.len]")
+/datum/controller/subsystem/ping/stat_entry(msg)
+	return ..("P:[GLOB.clients.len]")
 
 /datum/controller/subsystem/ping/fire(resumed = FALSE)
 	// Prepare the new batch of clients

code/datums/components/storage/concrete/pockets.dm

@@ -117,16 +117,6 @@
 	max_items = 2
 	max_w_class = WEIGHT_CLASS_BULKY
 
-/datum/component/storage/concrete/pockets/holster/lt/Initialize()
-	original_parent = parent
-	. = ..()
-	can_hold = typecacheof(list(
-		/obj/item/gun/energy/e_gun/adv_stopping,
-		/obj/item/gun/energy/e_gun/mini,
-		/obj/item/gun/energy/disabler,
-		/obj/item/stock_parts/cell/gun
-	))
-
 /datum/component/storage/concrete/pockets/holster/nukie/Initialize()
 	original_parent = parent
 	. = ..()

code/datums/tgs_event_handler.dm

@@ -1,5 +1,6 @@
 /datum/tgs_event_handler/impl
 	var/datum/timedevent/reattach_timer
+	var/datum/timedevent/delay_end_timer
 
 /datum/tgs_event_handler/impl/HandleEvent(event_code, ...)
 	switch(event_code)
@@ -14,28 +15,44 @@
 			message_admins("TGS: Instance renamed to from [world.TgsInstanceName()] to [args[2]]")
 		if(TGS_EVENT_COMPILE_START)
 			message_admins("TGS: Deployment started, new game version incoming...")
+			SSticker.delay_end = ROUND_END_TGS
+			SSticker.admin_delay_notice = "TGS Deployment in progress, please wait..."
+			delay_end_timer = addtimer(CALLBACK(src, PROC_REF(end_delay)), 5 MINUTES, TIMER_STOPPABLE)
 		if(TGS_EVENT_COMPILE_CANCELLED)
 			message_admins("TGS: Deployment cancelled!")
+			end_delay()
 		if(TGS_EVENT_COMPILE_FAILURE)
 			message_admins("TGS: Deployment failed!")
+			end_delay()
 		if(TGS_EVENT_DEPLOYMENT_COMPLETE)
 			message_admins("TGS: Deployment complete!")
-			to_chat(world, "<span class='boldannounce'>Server updated, changes will be applied on the next round...</span>")
+			to_chat(world, span_boldannounce("Server updated, changes will be applied on the next round..."))
+			end_delay()
 		if(TGS_EVENT_WATCHDOG_DETACH)
 			message_admins("TGS restarting...")
-			reattach_timer = addtimer(CALLBACK(src, PROC_REF(LateOnReattach)), 1 MINUTES)
+			reattach_timer = addtimer(CALLBACK(src, PROC_REF(LateOnReattach)), 1 MINUTES, TIMER_STOPPABLE)
 		if(TGS_EVENT_WATCHDOG_REATTACH)
 			var/datum/tgs_version/old_version = world.TgsVersion()
 			var/datum/tgs_version/new_version = args[2]
 			if(!old_version.Equals(new_version))
-				to_chat(world, "<span class='boldannounce'>TGS updated to v[new_version.deprefixed_parameter]</span>")
+				to_chat(world, span_boldannounce("TGS updated to v[new_version.deprefixed_parameter]"))
 			else
 				message_admins("TGS: Back online")
 			if(reattach_timer)
 				deltimer(reattach_timer)
 				reattach_timer = null
 		if(TGS_EVENT_WATCHDOG_SHUTDOWN)
-			to_chat_immediate(world, "<span class='boldannounce'>Server is shutting down!</span>")
+			to_chat_immediate(world, span_boldannounce("Server is shutting down!"))
 
 /datum/tgs_event_handler/impl/proc/LateOnReattach()
 	message_admins("Warning: TGS hasn't notified us of it coming back for a full minute! Is there a problem?")
+
+/datum/tgs_event_handler/impl/proc/end_delay()
+	if(SSticker.delay_end == ROUND_END_TGS)
+		SSticker.delay_end = ROUND_END_NOT_DELAYED
+		SSticker.admin_delay_notice = null
+		if(delay_end_timer)
+			deltimer(delay_end_timer)
+			delay_end_timer = null
+		if(SSticker.ready_for_reboot && !SSticker.delay_end) //we undelayed after standard reboot would occur
+			SSticker.standard_reboot()

code/game/objects/effects/landmarks.dm

@@ -219,10 +219,6 @@ INITIALIZE_IMMEDIATE(/obj/effect/landmark)
 	name = "Brig Physician"
 	icon_state = "Brig Physician"
 
-/obj/effect/landmark/start/lieutenant
-	name = "SolGov Representative"		//WS Edit - SolGov Rep
-	icon_state = "SolGov Representative"		//WS Edit - SolGov Rep
-
 //Department Security spawns
 
 /obj/effect/landmark/start/depsec

code/game/objects/items/devices/PDA/PDA_types.dm

@@ -239,8 +239,3 @@
 	name = "brig physician PDA"
 	default_cartridge = /obj/item/cartridge/medical
 	icon_state = "pda-brig_phys"
-
-/obj/item/pda/lieutenant
-	name = "lieutenant PDA"
-	default_cartridge = /obj/item/cartridge/hos
-	icon_state = "pda-h"

code/game/objects/items/devices/radio/headset.dm

@@ -411,20 +411,6 @@ GLOBAL_LIST_INIT(channel_tokens, list(
 	desc = "A headset allowing the wearer to communicate with medbay and service."
 	icon_state = "med_headset"
 
-/obj/item/radio/headset/heads/lieutenant
-	name = "lieutenant's headset"
-	desc = "A lieutenant's headset."
-	icon_state = "com_headset"
-
-/obj/item/radio/headset/heads/lieutenant/alt
-	name = "lieutenant's bowman headset"
-	desc = "A lieutenant's headset. Protects ears from flashbangs."
-	icon_state = "com_headset_alt"
-
-/obj/item/radio/headset/heads/lieutenant/alt/ComponentInitialize()
-	. = ..()
-	AddComponent(/datum/component/wearertargeting/earprotection, list(ITEM_SLOT_EARS))
-
 /obj/item/radio/headset/heads/rd
 	name = "\proper the research director's headset"
 	desc = "Headset of the fellow who keeps society marching towards technological singularity."