Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

You can now emag the gravity generator (and fix it) #2635

Merged
merged 6 commits into from
May 19, 2024

Conversation

DeltaFire15
Copy link
Contributor

About The Pull Request

Title!
Using an emag on an active gravity generator after a few seconds of interaction will short out the generator and reboot it with some... slightly altered settings.
Very hazardous to do.
Inconveniencing for just about anyone once the generator is done charging (you should be gone by then), somewhat dangerous for people weak to high gravity.

To fix, take the generator offline (nondestructively) and carefully reset its firmware with a multitool.

Almost entirely modularized save for some changes to the JS, because I felt like writing modular code.

Why It's Good For The Game

Ways for antagonists to antagonize without doing (almost) irreplaceable things ("space all the NACs haha!") are fun!

Testing Photographs and Procedure

No.

Changelog

馃啈
add: The gravity generator now has an emag interaction (and a method to fix said interaction).
/:cl:

we love ways to mess with things..
just to be sure..
@DeltaFire15 DeltaFire15 added Feature TGUI TGUI Changes/Crashlogs labels May 1, 2024
@Bokkiewokkie Bokkiewokkie merged commit c4263f8 into BeeStation:master May 19, 2024
9 checks passed
//Modular proc attachment
/obj/machinery/gravity_generator/main/ui_data(mob/user)
. = ..()
.["emagged"] = (obj_flags & EMAGGED) ? TRUE : FALSE //Trinary for safety because bitfield checks do not return just a pure TRUE / FALSE.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ternary

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bit late there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants