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

[MIRROR] Fixes body collision causing a stun, despite a successful block. #739

Merged
merged 1 commit into from
Nov 21, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25146
Original PR: tgstation/tgstation#79824

About The Pull Request

Puts a block check into the throw_impact() of carbon mobs.

Why It's Good For The Game

I'm touching on a lot of 'get around shields' stuns, and this has been a big one for the better part of a few years and potentially not even intentional. I would say it gained its largest popularity when it became weaponized with fireman carrying.

Despite seemingly rolling to block, blocking a body hitting you doesn't actually do anything at all. This reminds me a bit of energy bolas. So I fixed it? I think, there might be a better fix, I'm just replicating code present in xenomorph tackles. This shit sucks, please recommend a better fix if you know it.

Changelog

🆑 necromanceranne
fix: When you successfully block a body collision, it does something rather than nothing at all.
/:cl:

…ock. [MDB IGNORE] (#25146)

* Fixes body collision causing a stun, despite a successful block. (#79824)

## About The Pull Request

Puts a block check into the ``throw_impact()`` of carbon mobs.

## Why It's Good For The Game

I'm touching on a lot of 'get around shields' stuns, and this has been a
big one for the better part of a few years and potentially not even
intentional. I would say it gained its largest popularity when it became
weaponized with fireman carrying.

Despite seemingly rolling to block, blocking a body hitting you doesn't
actually do anything at all. This reminds me a bit of energy bolas. So I
fixed it? I think, there might be a better fix, I'm just replicating
code present in xenomorph tackles. This shit sucks, please recommend a
better fix if you know it.

## Changelog
:cl:
fix: When you successfully block a body collision, it does something
rather than nothing at all.
/:cl:

* Fixes body collision causing a stun, despite a successful block.

---------

Co-authored-by: necromanceranne <[email protected]>
@Iajret Iajret merged commit 00d1559 into master Nov 21, 2023
24 checks passed
@Iajret Iajret deleted the upstream-mirror-25146 branch November 21, 2023 20:29
AnywayFarus added a commit that referenced this pull request Nov 21, 2023
Iajret pushed a commit that referenced this pull request Feb 13, 2024
* Minor STuff 1

* More stuff 2: tarkonian boogaloo

* THE solar array.

* Yeah this is annoying to deal with.

* Because Common Sense Doesn't Exist.

* That might actually track better.

* Lets just make these more bearable.

* they dont need cable.

* cant be pocket.

* Fix The Polarizers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants