Skip to content

Fix Rocket/LTB/Caster ground firing and retire AMMO_EXPLOSIVE - #13054

Open
fira wants to merge 1 commit into
cmss13-devs:masterfrom
fira:rocket-target-turfs
Open

Fix Rocket/LTB/Caster ground firing and retire AMMO_EXPLOSIVE#13054
fira wants to merge 1 commit into
cmss13-devs:masterfrom
fira:rocket-target-turfs

Conversation

@fira

@fira fira commented Aug 30, 2026

Copy link
Copy Markdown
Member

About the pull request

For about 4 years, Rocket Launchers and LTB have been unable to fire at the ground. Apparently nobody realized.

I actually fixed that bug for Flare Guns way back in 2022 in #2031 but in my grand ignorance I did not realize that AMMO_EXPLOSIVE implicitely means AMMO_HITS_TARGET_TURF because people were too lazy to type both.

Why does it break ?

As said in #2031 - essentially projectiles subjected to AMMO_HITS_TARGET_TURF must ignore scatter calculation. Scatter re-targets the projectile by raycasting it, so you're now aiming somewhere else than you clicked, far away. It forgets what turf you wanted to hit.

Won't removing scatter from LTB / M5 RPG / Caster be a big difference?

I don't think so, they're spec'ed to zero scatter anyway.
Technically the HPC and Dual Plasma Cannons have a small amount of scatter, but I don't think that's really the biggest worry if HPCs are in play.

Wait, you're removing AMMO_EXPLOSIVE entirely.

Yes because it actually only did two things:

  • Act as AMMO_HITS_TARGET_TURF
  • Cause BE/Suicide to be explosive, which only matters for Mateba HE, which already deals >80 damage to trigger the check, so why even bother?

The first obviously doesn't warrant a flag. The second... I don't know, I guess maybe if you were to somehow try to suicide point blank with a WP rocket it'd make a difference, but i'm pretty sure it's impossible too.

What about HE Mateba?

Well fixing without removing AMMO_EXPLOSIVE would give it ground targeting, which you don't want on small arms, so I didn't give it AMMO_HITS_TARGET_TURF. Happy ending.

HPC/RPG warning

This means HPC and RPG will explode where you click, even if you click 2 tiles from you. So be careful. Point at what you want to blow up, don't shoot in the general direction.

I can't believe you found a holy projectile flag in the land of terrible 24-flags BYOND drought!

Yeah well this goes to show how RELEVANT our flags are and why this is a false problem.

Changelog

🆑
fix: Rocket/LTB/HPC now explode on clicked turf properly again rather than flying by. Make SURE you click on what you want to explode, not the general direction.
del: Removed Explosive Ammo properties. It only mattered for BE/Suicide with HE Mateba, and it does that anyway.
balance: Caster Eradicator (HPC) / Dual Plasma Cannons had scatter disabled to target ground properly. Little difference that makes if that thing is pointed at you.
/:cl:

@github-project-automation github-project-automation Bot moved this to Awaiting Review in Review Backlog Aug 30, 2026
@cmss13-ci cmss13-ci Bot added Removal snap Fix Fix one bug, make ten more size/S Denotes a PR that changes 19-49 lines, ignoring generated files. labels Aug 30, 2026
@fira fira added the Balance You need to be a professional veteran game maintainer to comprehend what is being done here. label Aug 30, 2026
@fira

fira commented Aug 30, 2026

Copy link
Copy Markdown
Member Author

Slapping Balance just for the hell of it because it turns out HPC has low scatter which is removed as a byproduct, that's really the only change i see though

@antlersss

Copy link
Copy Markdown
Contributor

Code LGTM

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

Labels

Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Fix Fix one bug, make ten more Removal snap size/S Denotes a PR that changes 19-49 lines, ignoring generated files.

Projects

Status: Awaiting Review

Development

Successfully merging this pull request may close these issues.

2 participants