diff --git a/code/controllers/subsystem/non_firing/SSlate_mapping.dm b/code/controllers/subsystem/non_firing/SSlate_mapping.dm
index ccd435f6242..e3b34f58e83 100644
--- a/code/controllers/subsystem/non_firing/SSlate_mapping.dm
+++ b/code/controllers/subsystem/non_firing/SSlate_mapping.dm
@@ -14,6 +14,10 @@ SUBSYSTEM_DEF(late_mapping)
GLOB.air_alarms = sortAtom(GLOB.air_alarms)
GLOB.apcs = sortAtom(GLOB.apcs)
+ for(var/obj/machinery/computer/shuttle/console in GLOB.machines)
+ if(console.find_destinations_in_late_mapping)
+ console.connect()
+
if(length(maze_generators))
var/watch = start_watch()
log_startup_progress("Generating mazes...")
diff --git a/code/modules/shuttle/dock_generator.dm b/code/modules/shuttle/dock_generator.dm
index d6769dfeca3..132e66a13d3 100644
--- a/code/modules/shuttle/dock_generator.dm
+++ b/code/modules/shuttle/dock_generator.dm
@@ -3,17 +3,26 @@
desc = "A signaling device for the NT expeditionary vessel, used to designate new docking areas."
icon = 'icons/obj/device.dmi'
icon_state = "gangtool-white"
+ new_attack_chain = TRUE
var/list/placed_docks = list()
- var/max_docks = 3
+ var/max_docks = 6
+ var/shuttleId = "whiteship"
+ var/possible_destinations
/obj/item/whiteship_port_generator/examine(mob/user)
. = ..()
var/count = max_docks - length(placed_docks)
var/plural = count > 1
- . += "There [plural ? "are" : "is"] [count] use[plural ? "s" : ""] left."
+ . += "There [plural ? "are" : "is"] [count] use[plural ? "s" : ""] left."
+ . += "Alt-Click to call the ship to an existing docking area."
+
+/obj/item/whiteship_port_generator/activate_self(mob/user)
+ . = FINISH_ATTACK
+
+ if(..())
+ return
-/obj/item/whiteship_port_generator/attack_self__legacy__attackchain(mob/living/user)
if(is_station_level(user.z))
log_admin("[key_name(user)] attempted to create a whiteship dock in the station's sector at [COORD(user)].")
to_chat(user, "New docking areas cannot be designated within the station's sector!")
@@ -29,10 +38,15 @@
to_chat(user, "New docking areas cannot be designated planet side!")
return
+ var/list/docks_per_z = list()
for(var/obj/placed_dock in placed_docks)
- if(placed_dock.z == user.z)
- to_chat(user, "A docking area has already been placed in this sector!")
- return
+ if(!("[placed_dock.z]" in docks_per_z))
+ docks_per_z["[placed_dock.z]"] = 0
+ docks_per_z["[placed_dock.z]"]++
+
+ if(docks_per_z["[user.z]"] >= 3)
+ to_chat(user, "This sector cannot support any more docking areas!")
+ return
var/list/dir_choices = list("North" = NORTH, "East" = EAST, "South" = SOUTH, "West" = WEST)
var/dir_choice = tgui_input_list(user, "Select the new docking area orientation.", "Dock Orientation", dir_choices)
@@ -99,11 +113,76 @@
for(var/obj/machinery/computer/shuttle/white_ship/S in GLOB.machines)
S.possible_destinations = null
S.connect()
+ possible_destinations = S.possible_destinations
log_admin("[key_name(user)] created a whiteship dock named '[name]' at [COORD(port)].")
+ to_chat(user, "Landing zone set.")
- if(dock_count < max_docks)
- to_chat(user, "Landing zone set.")
- else
- to_chat(user, "Landing zone set. The signaller vanishes!")
- qdel(src)
+/obj/item/whiteship_port_generator/AltClick(mob/user, modifiers)
+ for(var/obj/machinery/computer/shuttle/white_ship/S in GLOB.machines)
+ possible_destinations = S.possible_destinations
+ break
+
+ ui_interact(user)
+
+/obj/item/whiteship_port_generator/ui_state(mob/user)
+ return GLOB.default_state
+
+/obj/item/whiteship_port_generator/ui_interact(mob/user, datum/tgui/ui = null)
+ ui = SStgui.try_update_ui(user, src, ui)
+ if(!ui)
+ ui = new(user, src, "ShuttleConsole", name)
+ ui.open()
+
+/// Pretty much a straight copy-paste of [/obj/machinery/computer/shuttle/proc/ui_data].
+/// Yes it could be refactored but tearing all of the "make the shuttle move" code out of
+/// the console whose job it is to exclusively do that is unsurprisingly messy.
+/obj/item/whiteship_port_generator/ui_data(mob/user)
+ var/list/data = list()
+ var/obj/docking_port/mobile/M = SSshuttle.getShuttle(shuttleId)
+ data["status"] = M ? M.getStatusText() : null
+ if(M)
+ data["shuttle"] = TRUE //this should just be boolean, right?
+ var/list/docking_ports = list()
+ data["docking_ports"] = docking_ports
+ var/list/options = params2list(possible_destinations)
+ for(var/obj/docking_port/stationary/S in SSshuttle.stationary_docking_ports)
+ if(!options.Find(S.id))
+ continue
+ if(!M.check_dock(S))
+ continue
+ docking_ports[++docking_ports.len] = list("name" = S.name, "id" = S.id)
+ data["docking_ports_len"] = length(docking_ports)
+ return data
+
+/// Pretty much a straight copy-paste of [/obj/machinery/computer/shuttle/proc/ui_act].
+/// Yes it could be refactored but tearing all of the "make the shuttle move" code out of
+/// the console whose job it is to exclusively do that is unsurprisingly messy.
+/obj/item/whiteship_port_generator/ui_act(action, params)
+ if(..()) //we can't actually interact, so no action
+ return TRUE
+ if(!allowed(usr))
+ to_chat(usr, "Access denied.")
+ return TRUE
+ var/list/options = params2list(possible_destinations)
+ if(action == "move")
+ var/destination = params["move"]
+ if(!options.Find(destination))//figure out if this translation works
+ message_admins("EXPLOIT: [ADMIN_LOOKUPFLW(usr)] attempted to move [src] to an invalid location! [ADMIN_COORDJMP(src)]")
+ return
+ switch(SSshuttle.moveShuttle(shuttleId, destination, TRUE, usr))
+ if(0)
+ atom_say("Shuttle en route.")
+ usr.create_log(MISC_LOG, "used [src] to call the [shuttleId] shuttle")
+ add_fingerprint(usr)
+ return TRUE
+ if(1)
+ to_chat(usr, "Invalid shuttle requested.")
+ if(2)
+ to_chat(usr, "Unable to comply.")
+ if(3)
+ atom_say("Shuttle is regenerating fuel. Please wait...")
+ if(4)
+ atom_say("Shuttle is currently en-route. The shuttle cannot be rerouted at this time.")
+ if(5)
+ atom_say("Shuttle is currently departing. Please wait...")
diff --git a/code/modules/shuttle/shuttle.dm b/code/modules/shuttle/shuttle.dm
index 2bf5377eb01..3c8eb4cd0bf 100644
--- a/code/modules/shuttle/shuttle.dm
+++ b/code/modules/shuttle/shuttle.dm
@@ -939,8 +939,8 @@
var/admin_controlled
var/max_connect_range = 7
var/moved = FALSE //workaround for nukie shuttle, hope I find a better way to do this...
- /// Do we want to search for shuttle destinations as part of Initialize (fixed) or LateInitialize (variable)
- var/find_destinations_in_late_init = FALSE
+ /// Do we want to search for shuttle destinations as part of Initialize (fixed) or when SSlate_mapping fires (variable)
+ var/find_destinations_in_late_mapping = FALSE
/obj/machinery/computer/shuttle/New(location, obj/item/circuitboard/shuttle/C)
..()
@@ -950,14 +950,11 @@
/obj/machinery/computer/shuttle/Initialize(mapload)
. = ..()
- if(find_destinations_in_late_init && mapload) // We only care about this in mapload, if its mid round its fine
- return INITIALIZE_HINT_LATELOAD
+ if(find_destinations_in_late_mapping && mapload) // We only care about this in mapload, if its mid round its fine
+ return
connect()
-/obj/machinery/computer/shuttle/LateInitialize()
- connect()
-
/obj/machinery/computer/shuttle/proc/connect()
var/obj/docking_port/mobile/M
if(!shuttleId)
@@ -1094,7 +1091,7 @@
circuit = /obj/item/circuitboard/white_ship
shuttleId = "whiteship"
possible_destinations = null // Set at runtime
- find_destinations_in_late_init = TRUE
+ find_destinations_in_late_mapping = TRUE
/obj/machinery/computer/shuttle/admin
name = "admin shuttle console"