-
-
Notifications
You must be signed in to change notification settings - Fork 538
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Meteor "Shield" #3371
Meteor "Shield" #3371
Conversation
…nto meteor-defenses
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ill probally refactor these to use a computer in the helm at some point but this is fine for now
if(W.tool_behaviour == TOOL_WRENCH) | ||
if(!anchored && !isinspace()) | ||
W.play_tool_sound(src, 100) | ||
if (W.use_tool(src, user, 20)) | ||
to_chat(user, span_notice("You secure \the [src] to the floor!")) | ||
set_anchored(TRUE) | ||
else if(anchored) | ||
W.play_tool_sound(src, 100) | ||
if (W.use_tool(src, user, 20)) | ||
to_chat(user, span_notice("You unsecure \the [src] from the floor!")) | ||
if(active) | ||
to_chat(user, span_notice("\The [src] shuts off!")) | ||
set_anchored(FALSE) | ||
if(W.tool_behaviour == TOOL_MULTITOOL) | ||
var/a = stripped_input(usr, "Please enter desired ID.", name, id, 20) | ||
if (!a) | ||
return | ||
id = a | ||
. = TRUE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these should probably use tool_acts but i forget if we actually like those
code/modules/station_goals/shield.dm
Outdated
/obj/machinery/meteor_shield/proc/los(meteor) | ||
for(var/turf/T in getline(src,meteor)) | ||
return TRUE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to do nothing rn
code/modules/station_goals/shield.dm
Outdated
var/kill_range = 6 | ||
var/fire_delay = 7 SECONDS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These seem a bit strict for something you're spending probably 5k+ on; high delay means you'd chance losing a covered area because the turret's cooling (unless you have overlap) and low range means you'll need quite a few turrets even disregarding that
code/modules/station_goals/shield.dm
Outdated
/obj/machinery/computer/meteor_shield/ui_data() | ||
var/list/data = list() | ||
|
||
data["satellites"] = list() | ||
for(var/obj/machinery/satellite/S in GLOB.machines) | ||
for(var/obj/machinery/meteor_shield/S in GLOB.machines) | ||
data["satellites"] += list(list( | ||
"id" = S.id, | ||
"active" = S.active, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This might need some work to get the console to only show turrets on the same ship, otherwise you'll get a list of everyone's turrets regardless of if you can touch them & I think you can touch turrets if the ships are docked to the same location
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll add them to a list on the ship. Shouldn't be too hard.
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
heartbreaking: I am lazy |
About The Pull Request
Adds purchasable anti-meteor turrets to the outpost.
Why It's Good For The Game
Sometimes you gotta go fast.
Changelog
🆑
add: Ships can now purchase anti-meteor grids at the outpost. Installation not included.
/:cl: