[Kudoz]

So, I need help makin the [REASON: %s ] work. Btw, this is from NAdmin. Thanks to Notis for this system! Love it!
Heres the ban code:

pawn Код:

COMMAND:ban(playerid, params[])
{
new id;
if(IsPlayerConnected(id))
{
if(PlayerInfo[playerid][AdminLevel] >= 1)
{
if(!sscanf(params, "u", id))
{
new string[900],reason[300];
format(file,sizeof(file),"/Data/users/%s.ini",id);
new name[MAX_PLAYER_NAME], PlayerName[MAX_PLAYER_NAME];
GetPlayerName(playerid, name, sizeof(name));
GetPlayerName(id, PlayerName, sizeof(PlayerName));
format(string, sizeof(string), " [[ < YOU HAVE BEEN BANNED BY ADMIN %s ! [REASON: %s ]", name,reason);
SendClientMessage(id, COLOR_RED, string);
dini_IntSet(file,"Banned",1);
format(string, sizeof(string), "You banned %s for %s", id,reason);
SendClientMessage(playerid,COLOR_WHITE, string);
format(string, sizeof(string), "Administrator %s banned %s for %s",name,id,reason);
SendClientMessageToAll(COLOR_WHITE, string);
BanEx(id,reason);
return 1;

[Ronaldo_raul™]

Well, try this code.I changed few things and made better.

Tell me if any bug.

pawn Код:

COMMAND:ban(playerid, params[])
{
    if(IsPlayerAdmin(playerid))
    {
        new id, reason[128];
        if ( sscanf ( params, "us[128]", id, reason ))  return SendClientMessage ( playerid, -1, "Usage: /ban [ID] [REASON]" ) ;
        if ( !IsPlayerConnected(player)) return SendClientMessage ( playerid, -1, "Invalid ID" ) ;
        else
        {
                new string[128];
               format(file,sizeof(file),"/Data/users/%s.ini",id);
               new name[MAX_PLAYER_NAME], PlayerName[MAX_PLAYER_NAME];
               GetPlayerName(playerid, name, sizeof(name));
               GetPlayerName(id, PlayerName, sizeof(PlayerName));
               format(string, sizeof(string), " [[ < YOU HAVE BEEN BANNED BY ADMIN %s ! [REASON: %s ]", name,reason);
               SendClientMessage(id, COLOR_RED, string);
               dini_IntSet(file,"Banned",1);
               format(string, sizeof(string), "You banned %s for %s", id,reason);
               SendClientMessage(playerid,COLOR_WHITE, string);
               format(string, sizeof(string), "Administrator %s banned %s for %s",name,id,reason);
               SendClientMessageToAll(COLOR_WHITE, string);
               BanEx(id,reason);
               return 1;
         }
    return 0;
}

[Kudoz]

Quote:

Originally Posted by Ronaldo_raul™ Well, try this code.I changed few things and made better. Tell me if any bug. pawn Код: COMMAND:ban(playerid, params[]) {     if(IsPlayerAdmin(playerid))     {         new id, reason[128];         if ( sscanf ( params, "us[128]", id, reason ))  return SendClientMessage ( playerid, -1, "Usage: /ban [ID] [REASON]" ) ;         if ( !IsPlayerConnected(player)) return SendClientMessage ( playerid, -1, "Invalid ID" ) ;         else         {                 new string[128];                format(file,sizeof(file),"/Data/users/%s.ini",id);                new name[MAX_PLAYER_NAME], PlayerName[MAX_PLAYER_NAME];                GetPlayerName(playerid, name, sizeof(name));                GetPlayerName(id, PlayerName, sizeof(PlayerName));                format(string, sizeof(string), " [[ < YOU HAVE BEEN BANNED BY ADMIN %s ! [REASON: %s ]", name,reason);                SendClientMessage(id, COLOR_RED, string);                dini_IntSet(file,"Banned",1);                format(string, sizeof(string), "You banned %s for %s", id,reason);                SendClientMessage(playerid,COLOR_WHITE, string);                format(string, sizeof(string), "Administrator %s banned %s for %s",name,id,reason);                SendClientMessageToAll(COLOR_WHITE, string);                BanEx(id,reason);                return 1;          }     return 0; }

pawn Код:

C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(314) : error 017: undefined symbol "player"
C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(318) : warning 217: loose indentation
C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(332) : warning 225: unreachable code
C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(332) : warning 217: loose indentation
C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(334) : warning 217: loose indentation
C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(336) : error 010: invalid function or declaration
C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(338) : error 010: invalid function or declaration
Pawn compiler 3.2.3664          Copyright (c) 1997-2006, ITB CompuPhase

[Ronaldo_raul™]

Quote:

Originally Posted by Kudoz pawn Код: C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(314) : error 017: undefined symbol "player" C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(318) : warning 217: loose indentation C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(332) : warning 225: unreachable code C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(332) : warning 217: loose indentation C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(334) : warning 217: loose indentation C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(336) : error 010: invalid function or declaration C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(338) : error 010: invalid function or declaration Pawn compiler 3.2.3664          Copyright (c) 1997-2006, ITB CompuPhase

Ok change this -

pawn Код:

if ( !IsPlayerConnected(id)) return SendClientMessage ( playerid, -1, "Invalid ID" ) ;

and show me your 332 , 336 and 338 Line.

[Kudoz]

pawn Код:

LINE 332 :
    return 0;

LINE 336:
else return SendClientMessage(playerid, COLOR_RED, "You are not allowed to use this command.");

LINE 338:

else return SendClientMessage(playerid, COLOR_RED, "Player is not connected.");

[Ronaldo_raul™]

Quote:

Originally Posted by Kudoz pawn Код: LINE 332 :     return 0; LINE 336: else return SendClientMessage(playerid, COLOR_RED, "You are not allowed to use this command."); LINE 338: else return SendClientMessage(playerid, COLOR_RED, "Player is not connected.");

Ok, after taking a look at NAdmin. I made it.

Here replace the whole BAN command with this -

pawn Код:

COMMAND:ban(playerid, params[])
{
if(IsPlayerAdmin(playerid))
{
new id, reason[128];
if ( sscanf ( params, "us[128]", id, reason ))  return SendClientMessage ( playerid, -1, "Usage: /ban [ID] [REASON]" ) ;
if ( !IsPlayerConnected(id)) return SendClientMessage ( playerid, -1, "Invalid ID" ) ;
else
{
new string[128];
format(file,sizeof(file),"/Data/users/%s.ini",id);
new name[MAX_PLAYER_NAME], PlayerName[MAX_PLAYER_NAME];
GetPlayerName(playerid, name, sizeof(name));
GetPlayerName(id, PlayerName, sizeof(PlayerName));
format(string, sizeof(string), " [[ < YOU HAVE BEEN BANNED BY ADMIN %s ! [REASON: %s ]", name,reason);
SendClientMessage(id, COLOR_RED, string);
dini_IntSet(file,"Banned",1);
format(string, sizeof(string), "You banned %s for %s", id,reason);
SendClientMessage(playerid,COLOR_WHITE, string);
format(string, sizeof(string), "Administrator %s banned %s for %s",name,id,reason);
SendClientMessageToAll(COLOR_WHITE, string);
BanEx(id,reason);
}
return 1;
}
return 0;
}

[Kudoz]

Thanks! but i still get errors :

pawn Код:

C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(335) : error 010: invalid function or declaration
C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(337) : error 010: invalid function or declaration
C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(339) : error 010: invalid function or declaration

[Ronaldo_raul™]

Quote:

Originally Posted by Kudoz Thanks! but i still get errors : pawn Код: C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(335) : error 010: invalid function or declaration C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(337) : error 010: invalid function or declaration C:\Users\Siv malene\Desktop\NAdminV2\filterscripts\NAdmin.pwn(339) : error 010: invalid function or declaration

You doing something wrong. As i compiled with the command i provided before is working alright.

You can take my NAdmin(Only Edited the Ban Command) - HERE!


Sorry for late reply, i have some net problems. DUH!

[Kudoz]

Quote:

Originally Posted by Ronaldo_raul™ You doing something wrong. As i compiled with the command i provided before is working alright. You can take my NAdmin(Only Edited the Ban Command) - HERE! Sorry for late reply, i have some net problems. DUH!

Thanks m8! : D No problem! +rep ! U helped me alot! : D :cheers:

[Ronaldo_raul™]

Quote:

Originally Posted by Kudoz Thanks m8! : D No problem! +rep ! U helped me alot! : D :cheers:

No problem anytime.


@Notis123

Great, Thank you for the credits

@Twisted_Insane
Did you tested the code ? returning a SendClientMessage while checking the players level wont exclude him from banning someone. I think.