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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hegemony Energy Rifle Bugfix #19649

Merged
merged 2 commits into from
Jul 25, 2024

Conversation

Crosarius
Copy link
Contributor

@Crosarius Crosarius commented Jul 16, 2024

Fixes Bug #19648

The Hegemony Energy Rifle has the E-Gun as a parent, instead of the laser rifle, so it doesn't inherit all the single-fire-mode stuff that the laser rifle has. I gave it all those attributes to make sure you can't accidentally change the fire mode.

Edit: To clarify, this fixes the bug where you can change the fire mode and it causes the gun to vanish.

@Crosarius
Copy link
Contributor Author

!review

@Crosarius
Copy link
Contributor Author

!bugfix

@BotBOREALIS BotBOREALIS added the Bugfix The PR is fixing an issue and requires less reviews and time to be ready for merging. label Jul 20, 2024
can_turret = TRUE
turret_is_lethal = TRUE
projectile_type = /obj/item/projectile/beam/midlaser/hegemony
can_switch_modes = 0
Copy link
Contributor

Choose a reason for hiding this comment

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

Booleans have to use the macros (TRUE / FALSE)

Signed-off-by: Crosarius <[email protected]>
@NonQueueingMatt NonQueueingMatt added this pull request to the merge queue Jul 25, 2024
Merged via the queue into Aurorastation:master with commit 26e8846 Jul 25, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Merge Bugfix The PR is fixing an issue and requires less reviews and time to be ready for merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants