[Avo]

[ANY] No Admin Config
Version 1.0

Description:

Change configuration with or without admins. Useful for example to enable/disable Voting systems, Rock The Vote, Auto kick, ...

Cvars:

noadminconfig_version - plugin version
noadminconfig_enabled - Enable No Admin Config (default 1)
noadminconfig_adminflag - Admin Flags to watch (could be several) (default "k")
noadminconfig_cfg_with - Config file to execute with admins (empty to disable) (default "sourcemod/noadminconfig_with.cfg")
noadminconfig_cfg_without - Config file to execute without admins (empty to disable) (default "sourcemod/noadminconfig_without.cfg")
noadminconfig_inform - Inform non-admins of config change (default 0)

Installation:

Create your with/without admins config files into cfg subdirectories. For example, default config file to execute with admins ("sourcemod/noadminconfig_with.cfg") would be actually in xxx/cfg/sourcemod/noadminconfig_with.cfg

Versions History:

• V1.0.0 (2012-12-14)
  • Initial Release

[Sheepdude]

Do not use convars for admin flags, as this bypasses the admin overrides system. Use CheckCommandAccess instead.

[Avo]

I would for a simple admin command to check of course. But there isn't any command to check the access in this plugin, and it could be configure with several flags to check. So would really CheckCommandAccess be the fittest choice ?

For the information, I used the same mechanism here than Anti-TK Manager, AFK Manager, and CS:S Team Balance, three well-known approved plugins, using it for the same reasons.

[Sheepdude]

You make your own command name, such as "noadminconfig". It doesn't even have to be an actual command, just a text string that's unique to your plugin. This allows users to use the admin overrides system for your plugin.

https://forums.alliedmods.net/showpo...39&postcount=3

So yes, CheckCommandAccess is the fittest choice, even though both ways "work". And it's quite simple to implement.

if(CheckCommandAccess(client, "noadminconfig", ADMFLAG_VOTE, true))
// Do stuff

[Avo]

Ok thanks for the advice, I'll change it soon

[Dr. McKay]

I think in this case a cvar is the best option. The overrides system doesn't allow you to specify multiple flags. If multiple flags are specified in the overrides system, the admin must have all flags in order for it to return true.