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

Broken Excel Implants still give you excel antag status and allow you to use the #841

Open
5 tasks done
ScalesyScales opened this issue Feb 28, 2022 · 2 comments
Open
5 tasks done
Labels
Good First Issue We all started somewhere. (Theoretically) simple issues requiring simple fixes. 👥 Impact: I-1 All (How many users is it expected to affect?) 10+ Lore Issues regarding missing or incorrect lore. ⚠️ Severity: S-4 Minor Small issue that may cause minor issues to gameplay.

Comments

@ScalesyScales
Copy link

Description of issue

So if you find a broken compliance implant, either from a stash or a trash pile. If you use it, either on yourself or someone else, that person will get the excel antag status, and the ability to use excel equipment.

Difference between expected and actual behavior

Excel antags aren't supposed to exist.

Steps to reproduce

Find a broken compliance chip, inject it, now you are excel.

Specific information for locating

Length of time in which bug has been known to occur

Since we pulled excel, probably.

Client version, Server revision & Game ID

Server revision: master - 2022-02-24
70aa5e6bd9df92c0cb1af4b4419d6a49da83c12b

Issue bingo

Please check whatever applies. More checkboxes checked increase your chances of the issue being looked at sooner.

  • Issue could be reproduced at least once
  • Issue could be reproduced by different players
  • Issue could be reproduced in multiple rounds
  • Issue happened in a recent (less than 7 days ago) round
  • Couldn't find an existing issue about this
@SigDoesCode
Copy link
Collaborator

While I thoroughly enjoyed the RP that came out of the event that probably spawned this PR, we probably should just either remove the implanters from the stash/spawning pool or make it so that you can't inject them if you find one. Probably both, if we're going to be safest.

@SigDoesCode
Copy link
Collaborator

Apologies, I hit the wrong button. This issue isn't resolved. (Also, why is there not a "are you sure you want to close this PR" tickbox or something?)

@ShadzFen ShadzFen added Lore Issues regarding missing or incorrect lore. ⚠️ Severity: S-4 Minor Small issue that may cause minor issues to gameplay. 👥 Impact: I-1 All (How many users is it expected to affect?) 10+ Good First Issue We all started somewhere. (Theoretically) simple issues requiring simple fixes. labels Apr 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue We all started somewhere. (Theoretically) simple issues requiring simple fixes. 👥 Impact: I-1 All (How many users is it expected to affect?) 10+ Lore Issues regarding missing or incorrect lore. ⚠️ Severity: S-4 Minor Small issue that may cause minor issues to gameplay.
Projects
None yet
Development

No branches or pull requests

3 participants