[Exolent[jNr]]

Then maybe you should put a note in the first post saying it is for communities with trustworthy admins only.

[Trafalete]

i still say it isn't a good ideea

[Exolent[jNr]]

1. Don't create variables inside of loops.
Create them before the loop and then assign them values inside of the loop.

2. In print_all_ip(), don't mix return values.
Return 0 if fail and 1 if success is the standard.

3. Cache access(id, ADMIN_KICK) outside of the loop so you don't call it every time.

4. Don't use get_user_ip() and get_user_authid() in the loop unless player is not an admin.
Example:

if(access(iPlayer, ADMIN_KICK))
{
    // format fake ip and steamid
}
else
{
    // use get_user_ip and get_user_authid
}

5. Don't use #define for strings that are used more than once in the plugin.
You should use new const instead.

6. (OPTIONAL) If you want to make it even better, you could use the new Orpheu2 and hook the "status" command, block the results and print your own.
That way player's can't check if an admin's SteamID is fake.

Approval pending on changes.

[crazyeffect]

jayprakash Junior Member Last Activity: 30-12-09 166

[Hawk552]

This is redundant and cannot be approved. There are other plugins which do the job equally or better.

This plugin has been given time to be fixed but no changes have been made.

If you would like any information regarding possible adjustments you could make or things you could do to make this better, please feel free to post here or PM me.

Unapproved.