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

Fixes gibtonite explosions producing flames and flashes #77344

Merged
merged 1 commit into from
Aug 9, 2023

Conversation

Sealed101
Copy link
Member

About The Pull Request

On the tin. Was introduced in #77118 and uhh yeah

Why It's Good For The Game

Restoring previous behavior.
To be quite honest I can live with the flames, as they mostly aren't that much of an issue for a competent miner. Avoiding them or bringing an extinguisher with you is easy enough; if anything, these flames may or may not cook loose ore and turn it into sheets, which isn't favorable for the miner. But for consistency's sake it's better if we restore it to the previous effects.
fikou shield activate

Changelog

🆑
fix: fixed gibtonite explosions having flames and flash ranges
/:cl:

set gibby's flash and flame ranges to 0
@Sealed101 Sealed101 requested a review from Fikou as a code owner August 3, 2023 09:26
@tgstation-server tgstation-server added the Fix Rewrites a bug so it appears in different circumstances label Aug 3, 2023
@MrMelbert
Copy link
Contributor

Do you know if it's REALLY "previous behavior" though

Because the point of the fix pr, and the ORIGINAL pr, was to give explosions which do not explicitly set these flames and flashes

So did gibtonite originally intend to not have flames or flashes?

@Sealed101
Copy link
Member Author

been playing since 2020 and yeah gibby didn't ever produce flames, neither did it produce flames a few weeks before.
if it did, i'm crazy and in a room of rats

also considering that planetary atmos can still roll plasma sometimes it's probably a good idea to not make flames

flashes are in around the same boat, have been blastes by gibby a few times point blank over the years and i don't recall being flashed by it

in any case if it was intended for em to have flames and flashing, but they didn't for the longest time and nobody seemed to care, then lol add that to the list of stuff that didn't work for ages

@MrMelbert
Copy link
Contributor

It seems like it was broken in #58015 2 years ago originally

@Sealed101
Copy link
Member Author

in the very first iteration of gibtonite flash act was just not working apparently
as for the flame range it seems to have been either intended to default to 1, or it didn't work much like flash range
https://github.com/tgstation/tgstation/blob/e71f98bbb807e40db46d24d57548702eb33154c5/code/game/objects/explosion.dm

the pr does not set any values to flames and flash ranges
#2312

@MrMelbert MrMelbert merged commit 07d34fe into tgstation:master Aug 9, 2023
22 checks passed
comfyorange added a commit that referenced this pull request Aug 9, 2023
github-actions bot added a commit that referenced this pull request Aug 9, 2023
@Sealed101 Sealed101 deleted the gibtonite-flames-fix branch August 9, 2023 17:28
Jolly-66 pushed a commit to TaleStation/TaleStation that referenced this pull request Aug 9, 2023
… flashes (#7217)

Original PR: tgstation/tgstation#77344
-----
## About The Pull Request
On the tin. Was introduced in #77118 and uhh yeah

## Why It's Good For The Game

Restoring previous behavior.
To be quite honest I can live with the flames, as they mostly aren't
that much of an issue for a competent miner. Avoiding them or bringing
an extinguisher with you is easy enough; if anything, these flames may
or may not cook loose ore and turn it into sheets, which isn't favorable
for the miner. But for consistency's sake it's better if we restore it
to the previous effects.
![fikou shield
activate](https://github.com/tgstation/tgstation/assets/75863639/1e1f7869-5832-44ad-8778-885fd60695d5)


## Changelog



:cl:
fix: fixed gibtonite explosions having flames and flash ranges
/:cl:

---------

Co-authored-by: Sealed101 <cool.bullseye@yandex.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Rewrites a bug so it appears in different circumstances
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants