[ANY] Admin Funcommands Limiter [21/07/13] Version 1.3

Dr. McKay · *Last edited by Dr. McKay; 07-20-2013 at 03:59.*

Feedback as per our IRC discussion:

You might want to look into clientprefs or something to overcome rejoining wiping the limit
You defined a VersionCV variable but never used it (your version cvar is commented out for some reason)
You should be using AddCommandListener, not RegConsoleCmd
Your cvars don't follow any standard naming convention. I'd suggest admin_control_cvarname, like kickcv -> admin_control_kick
You may want to add a cvar to suppress the "You have had x out of y warnings" chat message
Don't return Plugin_Handled or there's a chance that you'll block the base SM plugin from doing its job. Plugin_Continue will suffice.
Using GetClientName at all is unnecessary in this plugin. Just use the %N format specifier. See also: http://wiki.alliedmods.net/Format_Cl...Mod_Scripting)
You're repeating if(CheckCommandAccess(client, "sm_addban", ADMFLAG_RCON, true) && GetConVarInt(AdminControlType) == 1) a lot. You might want to put it into a function.