mirror of
https://github.com/Bubberstation/Bubberstation.git
synced 2025-12-11 10:11:09 +00:00
980bd74047
* Fixes not being able to open inactive ticket browser on admin ticket tab. (#57882) About The Pull Request Currently admins can't access the ticket logs for Resolved, Disconnected, Closed and Active tickets due to an early return that will block these from being opened in 100% of circumstances. Clicking on the Active/Disconnected/Closed/Resolved tickets would have behaviour handled by /datum/admin_help_tickets/proc/BrowseTickets() which was reached through Click() calls. image When we call .Click() from topic calls, we provide a null var for the control parameter. This then leads through to /client/Click() where #57084 added a bunch of if(!control) checks supposedly ported from TGMC (or at least Fikou couldn't explain why we changed that). I can't find a single instance of us using this locally, it appears it's always passed through to Byond. Byond docs at http://www.byond.com/docs/ref/#/client/proc/Click describe it as control - the name of the skin control involved I can't see any reason to early return if control is a FALSE-y value. This check is what was blocking access to the admin_help_tickets datum. In testing, removing these checks restored previous functionality. Why It's Good For The Game Admins less triggered. Changelog 🆑 fix: Admins can once again open the stat panels to look at active, disconnected, closed and resolved tickets. /🆑 * Fixes not being able to open inactive ticket browser on admin ticket tab. Co-authored-by: Timberpoes <silent_insomnia_pp@hotmail.co.uk>