@joaquimandrade
I don't see the benefit of switching to an array of cvar pointers for a plugin with only a handful of cvars; it only reduces readability in my opinion.
As for get_players, it functions consistently fine in every plugin I've used it in. I do see that bailopan does not recommend using it with flags but being it functions fine consistently, I choose to use it. If\when it is done away with in future releases of AMX-X, I will then do away with using it.
This portion I agree with and I had already corrected.
PHP Code:
public client_disconnect(id)
{
//If an admin is disconnecting we clear out his powers so they will not be recognized in the
//SetAdmins function.
if ( g_AdminFlags[id] )
{
g_AdminFlags[id] = 0;
g_AdminLevel[id] = 0;
if(g_CurrentLevel[id])
{
g_CurrentLevel[id] = 0;
SetAdmins(0);
}
}
}