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: Restrict door remotes to only being able to manipulate doors relevant to their type #315

Conversation

SimpleStation14
Copy link
Member

Mirror of PR #26371: Restrict door remotes to only being able to manipulate doors relevant to their type from space-wizards space-wizards/space-station-14

93e3aed26e30375058aa44482236410ff31484f0

PR opened by nikthechampiongr at 2024-03-24 00:28:39 UTC


PR changed 1 files with 3 additions and 8 deletions.

The PR had the following labels:


Original Body

About the PR

This makes it so door remotes can no longer manipulate doors just because their user has access to them.

Why / Balance

Being able to bolt doors and put them on emergency access from a distance is extremely powerful. You should not be able to do it just because you have the access in your id card, it should be limited to the capabilities of the door remote.

It also makes no sense.

Technical details

Relevant calls to check for access, and try open/bolt doors had their arguments changed to pass in the door remote instead of the user so now only the door remote's accesses are checked.

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

🆑

  • tweak: Door remotes can now only control doors specific to their type and do not use the access of their user.

@SimpleStation14 SimpleStation14 added the Pull Request Mirror Mirrors a PR from another Repo. Automatically applied by mirror bot label Apr 22, 2024
Copy link
Contributor

@DangerRevolution DangerRevolution left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

interesting change, wonder the implication of this but overall seems good

@SimpleStation14 SimpleStation14 marked this pull request as draft May 4, 2024 21:12
@VMSolidus VMSolidus marked this pull request as ready for review May 11, 2024 01:17
Copy link
Member

@FoxxoTrystan FoxxoTrystan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@DangerRevolution DangerRevolution added Priority: 4-Low Should be resolved at some point Size: 4-Small For small issues/PRs Status: Needs Review Someone please review this Type: Feature Creation of or significant changes to a feature labels May 11, 2024
@DangerRevolution
Copy link
Contributor

This is intended to be merged with #357

@stellar-novas
Copy link
Contributor

Hmmm....
I get the intent behind this, but on roleplay servers, I don't think this is abused?

In my experience, remotes don't get stolen that often, so I'm more worried about how this affects legitimate play.

Heads feel no change since their remotes have the same access.
Atmos techs lose the ability to set another department's doors to emergency access by giving the remote to a member of that department.

This isn't useful to antags, but it is quite useful to engineers doing work on a department.

This feels like a change made to balance antag issues that aren't as prevalent on roleplay servers, and so it isn't needed here.

(This comment is really shittily written and rambly, I know, but I'm not doing great and it's taken me 20min to write this already, so I'm giving up here)

@VMSolidus VMSolidus merged commit b1a67cc into Simple-Station:master May 11, 2024
20 checks passed
Day-OS pushed a commit to Day-OS/Einstein-Engines that referenced this pull request Jun 25, 2024
DEATHB4DEFEAT added a commit that referenced this pull request Aug 1, 2024
…e Doors Relevant to Their Type" (#491)

Reverts #315

As [I brought
up](#315 (comment))
in the original pr, this change was made to counter power gaming, which
isn't as much of an issue on EE downstreams, and adversely affects the
engineering department, without any major benefits.

---------

Signed-off-by: stellar-novas <[email protected]>
Co-authored-by: DEATHB4DEFEAT <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 4-Low Should be resolved at some point Pull Request Mirror Mirrors a PR from another Repo. Automatically applied by mirror bot Size: 4-Small For small issues/PRs Status: Needs Review Someone please review this Type: Feature Creation of or significant changes to a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants