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

[Crash] Explosives can damage through fog #16413

Open
yyzsong opened this issue Jul 29, 2024 · 1 comment · May be fixed by #16780
Open

[Crash] Explosives can damage through fog #16413

yyzsong opened this issue Jul 29, 2024 · 1 comment · May be fixed by #16780
Labels
Bug Incorrect functionality.

Comments

@yyzsong
Copy link
Contributor

yyzsong commented Jul 29, 2024

Description

See title. This can injure xenos and xeno buildings to the point of their destruction

Test Merges

Server-Side Code Modifications: https://tgstation13.org/parsed-logs/sybil/server_side_modifications.dm

Round ID: 28231
BYOND Version: 515.1633
Server revision compiled on: 2024-07-29T01:44:28+00:00
Master commit: 9c9bd13cb732f9436c7985ad4c91732da7d4b693
The following pull requests are currently test merged:
#16162: 'Attempts to give marines a trickling economy with stimulus on crash' by Ldip999 at commit d31802296e
#16282: 'Adds rappel to tadpole (again)' by breadhunt at commit e245fe1501
#16406: 'Patrol point fix (for campaign)' by Lumipharon at commit 135a1bde9e
#16400: 'Timeshift bugfix' by Lumipharon at commit 248807333b

Local commit: 9261ac9
Server tools version: 6.6.1

Reproduction Steps

  1. Throw grenade at fog
  2. Our Resin silo 1 at North Western Caves (X: 37, Y: 100) is under attack! It has 930/1000 Health remaining.

Screenshots

No response

@yyzsong yyzsong added the Bug Incorrect functionality. label Jul 29, 2024
@Lumipharon
Copy link
Contributor

fog is nondense now, so is excluded from explosion block calcs.

Fix means it needs an explosion block value, and for explosions to not check for density.

@Lumipharon Lumipharon linked a pull request Dec 4, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Incorrect functionality.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants