tgstation / TerraGov-Marine-Corps

TGMC: TerraGov Marine Corps, a SS13 mod
GNU Affero General Public License v3.0
163 stars 821 forks source link

[Crash] Explosives can damage through fog #16413

Open yyzsong opened 3 months ago

yyzsong commented 3 months ago

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: 9261ac9a530c3b369d52cbd0a9c9e961f33783c9 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

Lumipharon commented 2 months ago

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.