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] Floor Viruses Return Unique IDs #747

Merged
merged 1 commit into from
Nov 20, 2023
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25152
Original PR: tgstation/tgstation#79840

About The Pull Request

Changes the floor disease viruses so that they return unique IDs (their typepath) instead of numeric IDs generated like normal Advanced viruses do.

Why It's Good For The Game

Bug reported downstream:
Skyrat-SS13/Skyrat-tg#24724
Skyrat-SS13/Skyrat-tg#24643

When players make a virus with the symptoms of only Fever or Headache in the pandemic (which are common to buff viruses), those will generate with the same ID as gastritium or carpellosis, because the floor viruses generated IDs like they were normal advanced viruses. If the virologist creates a virus sample from that, it will give it the subtype of the floor virus, and then if that sample is used to generate a healing virus the healing virus will get the symptoms and name of the healing virus but keep the subtype (including cure and other affects, like burping tritium) of the floor virus.

This change makes it so that the floor viruses cant be modified in the pandemic (trying to modify it and printing a culture tube will just make the culture tube of the original floor virus), and so that virology players can't accidentally create healing virus with those nasty effects.

The typepath was used as the ID to mimic what it looks like for the other (non-advanced) unique diseases.

If you can think of a better way to fix this, please let me know, I was just getting tired of having to put up with burping green gas to have a standard healing virus if the virologist wasn't aware of this bug.

Changelog

🆑 Thlumyn
fix: healing viruses can no longer have floor virus side effects
/:cl:

* Floor Viruses Return Unique IDs (#79840)

## About The Pull Request

Changes the floor disease viruses so that they return unique IDs (their
typepath) instead of numeric IDs generated like normal Advanced viruses
do.
## Why It's Good For The Game

Bug reported downstream:
Skyrat-SS13/Skyrat-tg#24724
Skyrat-SS13/Skyrat-tg#24643

When players make a virus with the symptoms of only Fever or Headache in
the pandemic (which are common to buff viruses), those will generate
with the same ID as gastritium or carpellosis, because the floor viruses
generated IDs like they were normal advanced viruses. If the virologist
creates a virus sample from that, it will give it the subtype of the
floor virus, and then if that sample is used to generate a healing virus
the healing virus will get the symptoms and name of the healing virus
but keep the subtype (including cure and other affects, like burping
tritium) of the floor virus.

This change makes it so that the floor viruses cant be modified in the
pandemic (trying to modify it and printing a culture tube will just make
the culture tube of the original floor virus), and so that virology
players can't accidentally create healing virus with those nasty
effects.

The typepath was used as the ID to mimic what it looks like for the
other (non-advanced) unique diseases.

If you can think of a better way to fix this, please let me know, I was
just getting tired of having to put up with burping green gas to have a
standard healing virus if the virologist wasn't aware of this bug.
## Changelog
:cl:
fix: healing viruses can no longer have floor virus side effects
/:cl:

* Floor Viruses Return Unique IDs

---------

Co-authored-by: Thlumyn <[email protected]>
@Iajret Iajret merged commit b52e506 into master Nov 20, 2023
24 checks passed
AnywayFarus added a commit that referenced this pull request Nov 20, 2023
@Iajret Iajret deleted the upstream-mirror-25152 branch November 20, 2023 19:57
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