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

Makes vendors less likely to freeze up when trying to click links #37402

Open
wants to merge 3 commits into
base: Bleeding-Edge
Choose a base branch
from

Conversation

SECBATON-GRIFFON
Copy link
Contributor

@SECBATON-GRIFFON SECBATON-GRIFFON commented Feb 1, 2025

[bugfix][qol]

What this does

moves vend_ready being set to 0 down after a return, adds a feedback message for clicking these links in-game while set to 0

Why it's good

more feedback

How it was tested

vending items, edit mode

Changelog

🆑

  • tweak: Vendors now indicate to users when they aren't ready to vend, if they try to be interacted with in some cases.

@D3athrow-Issues D3athrow-Issues added Bug / Fix This is either a bug or a fix for a bug. ❤️ Quality of Life ❤️ This is a change that removes hassle and improves quality of life. labels Feb 1, 2025
@SECBATON-GRIFFON SECBATON-GRIFFON marked this pull request as ready for review February 1, 2025 03:44
@Inorien
Copy link
Contributor

Inorien commented Feb 1, 2025

a bit of context on this PR:

several hours ago I was minning a round and threw a vendor at the station. the crew found it, brought it before the bridge and several people gathered around it all spamming the buttons as normal.
inevitably the interface broke - this is far from a new issue - but Kanef ahelped it and I went code diving and found that vend_ready was stuck at 0, which caused it to fail checks in Topic() on the vendomat base.
it seems that any vendor is vulnerable to this and that this bug is triggered by too many people hitting the interface at once, resulting in vend_ready getting stuck at 0 and the machine becoming unusable to anyone.
i went back to the round but dear Schlomo had already spaced the malfunctioning vendor. I teleported it to centcomm and set vend_readyback to 1, at which point it seemed to be working fine again.
i relayed to Kanef the info about vend_ready getting stuck and this PR is the result

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug / Fix This is either a bug or a fix for a bug. ❤️ Quality of Life ❤️ This is a change that removes hassle and improves quality of life.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants