Skip to content

Add server-side GM visibility control script#5

Merged
Nyeriah merged 1 commit intomasterfrom
fix/gm-visibility-for-non-gm-accounts
Sep 16, 2025
Merged

Add server-side GM visibility control script#5
Nyeriah merged 1 commit intomasterfrom
fix/gm-visibility-for-non-gm-accounts

Conversation

@Yehonal
Copy link
Collaborator

@Yehonal Yehonal commented Sep 16, 2025

TLDR: .Introduce a player script to manage server-side visibility for game masters, ensuring players with gm visible command can properly use it

This pull request introduces a new player script to enhance server-side visibility handling for Game Masters (GMs). The main change is the addition of logic that automatically upgrades a player's account type to SEC_GAMEMASTER when certain conditions are met, ensuring proper GM visibility permissions.

Enhancements to GM visibility logic

  • Added a new class mod_gm_commands_playerscript in src/GmCommands.cpp that overrides OnPlayerSetServerSideVisibility to set a player's account type to SEC_GAMEMASTER when they should be visible as a GM and have the appropriate permissions.
  • Registered the new player script in the AddGmCommandScripts function to ensure it is loaded with other GM command scripts.
  • Included the PlayerScript.h header in src/GmCommands.cpp to support the new player script functionality

@Nyeriah Nyeriah merged commit 7514e2f into master Sep 16, 2025
2 of 3 checks passed
@Helias Helias deleted the fix/gm-visibility-for-non-gm-accounts branch September 28, 2025 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants