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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds radio blacklist for non radio tokens in use #16224

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Ldip999
Copy link
Contributor

@Ldip999 Ldip999 commented Jul 1, 2024

About The Pull Request

Adds a blacklist of tokens that are already in use elsewhere other than the radio list

Why It's Good For The Game

Today I noticed that "Naptelek" has been assigned the :n token, which is already in use by binary, making command staff unable to use it. This adds various tokens to the blacklist that are already in use by other things.

Changelog

馃啈
fix: Fixes custom squads being assigned tokens that were already in use by non radio prefixes
/:cl:

@tgstation-server tgstation-server added the Fix Fixes an issue with the game. label Jul 1, 2024
@@ -417,6 +417,7 @@
GLOBAL_LIST_EMPTY_TYPED(custom_squad_radio_freqs, /datum/squad)
///initializes a new custom squad. all args mandatory
/proc/create_squad(squad_name, squad_color, mob/living/carbon/human/creator)
var/radio_whitelist = list(MODE_KEY_BINARY = MODE_KEY_BINARY, MODE_KEY_R_HAND = MODE_KEY_R_HAND, MODE_KEY_L_HAND = MODE_KEY_L_HAND, MODE_KEY_INTERCOM = MODE_KEY_INTERCOM, MODE_KEY_DEPARTMENT , MODE_KEY_DEPARTMENT)
Copy link
Member

Choose a reason for hiding this comment

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

You mean blacklist?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes I did, sorry

@Ldip999 Ldip999 changed the title Adds radio whitelist for non radio tokens in use Adds radio blacklist for non radio tokens in use Jul 2, 2024
code/datums/jobs/squads.dm Outdated Show resolved Hide resolved
code/datums/jobs/squads.dm Outdated Show resolved Hide resolved
code/datums/jobs/squads.dm Outdated Show resolved Hide resolved
code/datums/jobs/squads.dm Outdated Show resolved Hide resolved
code/datums/jobs/squads.dm Outdated Show resolved Hide resolved
@Ldip999
Copy link
Contributor Author

Ldip999 commented Jul 4, 2024

That was a much better way to do it, thank you, I learned lots!

@ivanmixo
Copy link
Contributor

ivanmixo commented Jul 4, 2024

You should put the statement in brackets, byond is being annoying

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fixes an issue with the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants