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

Add new trigger to ion storm to scramble ID cards #11366

Closed
wants to merge 3 commits into from

Conversation

Glamurio
Copy link
Contributor

[STATION SYSTEMS] [FEATURE]

About the PR

Adds a new ion_category which queries all player ID cards and randomly choose 5 of them to scramble their access. The modifier is currently removing a 5th of all accesses available, with a minimum of one. Afterwards, it will add 1 or 2 random accesses back to the card, excluding any removed ones.

The excluded accesses are access_fuck_all (to avoid breaking stuff), access_maxsec (for armory) and access_change_ids (to allow an easy reversion), as well as maintaining all hidden accesses such as syndicate which would be otherwise unobtainable.

Finally, it will use the radioGarbleText() method to scramble the name on the ID card.

Why's this needed?

This change is part of a long series of changes that I'm planning to make HoP gameplay more varied and interesting. A small subset of players on the station will suddenly find themselves unable to access a location, or dislike their new garbled name, and seek out the HoP to have them fix it. Additionally, it simply adds an additional layer of variety to the game, and fits well into the thematic of an ion storm messing with all electronic devices.

Changelog

(u)Glamurio (Ryou)
(*)Adds a category to the ion storm event which will scramble player ID cards.

@keywordlabeler keywordlabeler bot added A-Station-Systems Deals with station systems, e.g. networks, power, shields, belt hell C-Feature A new feature or enhancements to existing features labels Oct 14, 2022
@github-actions github-actions bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 14, 2022
@Wisemonster Wisemonster added the E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. label Oct 15, 2022
@github-actions
Copy link
Contributor

Created discussion thread: https://forum.ss13.co/showthread.php?tid=19949

@github-actions github-actions bot added the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Oct 17, 2022
@flrsh
Copy link
Contributor

flrsh commented Oct 18, 2022

This would just get people inconvenienced and mad at the HoP and doesn't actually make their job funner or give them further incentives.

@flrsh flrsh closed this Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Station-Systems Deals with station systems, e.g. networks, power, shields, belt hell C-Feature A new feature or enhancements to existing features E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants