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] Deletes area var on clients #897

Merged
merged 1 commit into from
Dec 1, 2023
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25358
Original PR: tgstation/tgstation#80037

About The Pull Request

Literally nothing wrote to or accessed this variable, in pretty much every case we just use get_area(mob) instead of ever looking at the client's area (again, ???)

Why It's Good For The Game

this variable was added 12 years ago and I don't think anyone has ever novelly used this.

Changelog

Doesn't matter to players.

* Deletes `area` var on clients (#80037)

## About The Pull Request

Literally nothing wrote to or accessed this variable, in pretty much
_every_ case we just use `get_area(mob)` instead of ever looking at the
client's area (again, ???)
## Why It's Good For The Game

this variable was added 12 years ago and I don't think anyone has ever
novelly used this.
## Changelog
Doesn't matter to players.

* Deletes `area` var on clients

---------

Co-authored-by: san7890 <[email protected]>
@Iajret Iajret merged commit 1404504 into master Dec 1, 2023
24 checks passed
@Iajret Iajret deleted the upstream-mirror-25358 branch December 1, 2023 18:47
Iajret pushed a commit that referenced this pull request Feb 11, 2024
…e it is eaten (#897)

* Fixes the energy cake slice's on-eat effect triggering before it is eaten (#81382)

## About The Pull Request

This makes the energy cake deal its damage/effects AFTER being fed,
instead of before.

It also adds combat logging to force-fed energy cakes, just in case.

Also, this adds a balloon alert to the pacifist check. Cool!
## Why It's Good For The Game

Fixes a wacky combat bug, and adds some minor adjustments made along the
way.

Closes #81351.
## Changelog
:cl: Rhials
fix: The Energy Cake slice now does its on-eat effect AFTER being eaten,
instead of before.
/:cl:

* Fixes the energy cake slice's on-eat effect triggering before it is eaten

---------

Co-authored-by: Rhials <[email protected]>
ReezeBL pushed a commit that referenced this pull request Feb 11, 2024
…e it is eaten (#897) (#1928)

* Fixes the energy cake slice's on-eat effect triggering before it is eaten (#81382)

## About The Pull Request

This makes the energy cake deal its damage/effects AFTER being fed,
instead of before.

It also adds combat logging to force-fed energy cakes, just in case.

Also, this adds a balloon alert to the pacifist check. Cool!
## Why It's Good For The Game

Fixes a wacky combat bug, and adds some minor adjustments made along the
way.

Closes #81351.
## Changelog
:cl: Rhials
fix: The Energy Cake slice now does its on-eat effect AFTER being eaten,
instead of before.
/:cl:

* Fixes the energy cake slice's on-eat effect triggering before it is eaten

---------

Co-authored-by: NovaBot <[email protected]>
Co-authored-by: Rhials <[email protected]>
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