09.02.2011, 11:55
Quote:
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; 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; } Oh, and - to make it worse, "/kick" (and NULL) is not 10 characters long. However, nice attempt. |