08.02.2011, 18:41
Sorry, but this is a big fail. Check your kick command for example.
You type /kick. It has no validation rules. You are then kicked (the player that typed the command) - then it checks to see if you're admin and if your not it tells you otherwise. First off; Thats not logical, secondly, it wont allow you to kick anyone but yourself. This goes for most of the admin commands i have seen, i havent had time to check them all.
Example;
Oh, and - to make it worse, "/kick" (and NULL) is not 10 characters long.
However, nice attempt.
You type /kick. It has no validation rules. You are then kicked (the player that typed the command) - then it checks to see if you're admin and if your not it tells you otherwise. First off; Thats not logical, secondly, it wont allow you to kick anyone but yourself. This goes for most of the admin commands i have seen, i havent had time to check them all.
Example;
pawn Code:
if (strcmp("/kick", cmdtext, true, 10) == 0)
{
Kick(playerid);
SendClientMessage(playerid, 0x0000FFAA,"/kick <id> <reason>");
if(!IsPlayerAdmin(playerid))
SendClientMessage(playerid, 0xAA3333AA,"You Are Not Admin.");
return 1;
}
However, nice attempt.