Whats wrong here
#1

pawn Код:
CMD:ban(playerid, params[])
{
    if(Player[playerid] [pAdmin] >=1)
    {
        new targetid;
        new sendername[MAX_PLAYERS];
        new recievername[MAX_PLAYERS];
        new reason[50];
        new string[120];
        GetPlayerName(playerid,sendername,sizeof(sendername));
        GetPlayerName(targetid,recievername,sizeof(recievername));
        if(sscanf(params,"uz",targetid,reason)) return SendClientMessage(playerid,COLOR_RED,"[ERROR]/ban [PlayerID] [REASON]");
        format(string,sizeof(string),"Administrator %s has banned %s reason: %s.",sendername,recievername,reason);
        SendClientMessageToAll(COLOR_RED,string);
        SavePlayer(targetid);
        Ban(targetid);
    }
    else
    {
        SendClientMessage(playerid,COLOR_RED,"[SERVER]:You are not allowed to use this command");
    }
    return 1;
}
It says Administrator Gogeta Has Banned Gogeta reason: blablalbla
Reply
#2

try to change

pawn Код:
"uz"
to

pawn Код:
"us"
Reply
#3

Did not work.
Reply
#4

its wrong
pawn Код:
new sendername[MAX_PLAYERS];
 new recievername[MAX_PLAYERS];

its correct
pawn Код:
new sendername[MAX_PLAYER_NAME];
 new recievername[MAX_PLAYER_NAME];
Reply
#5

You can't possibly know the name of the 'receiver' before you have established who 'receiver' actually is in the first place. It's simple logic, really.
Reply
#6

Change:
pawn Код:
"uz"
to
pawn Код:
"us[50]"
EDIT: Try that
pawn Код:
CMD:ban(playerid, params[])
{
    if(Player[playerid] [pAdmin] >=1)
    {
        new targetid, sendername[MAX_PLAYER_NAME], recievername[MAX_PLAYER_NAME], reason[50], string[144];
        if(sscanf(params,"us[50]", targetid, reason)) return SendClientMessage(playerid,COLOR_RED,"[ERROR]/ban [PlayerID] [REASON]");
       
        GetPlayerName(playerid, sendername, sizeof(sendername));
        GetPlayerName(targetid, recievername, sizeof(recievername));
        format(string,sizeof(string),"Administrator %s has banned %s reason: %s.",sendername,recievername,reason);
        SendClientMessageToAll(COLOR_RED,string);
        SavePlayer(targetid);
        Ban(targetid);
    }
    else
    {
        SendClientMessage(playerid,COLOR_RED,"[SERVER]:You are not allowed to use this command");
    }
    return 1;
}
Reply
#7

Try this code

pawn Код:
CMD:ban(playerid, params[])
{
    if(Player[playerid] [pAdmin] >=1)
    {
        new targetid;
        new sendername[MAX_PLAYERS];
        new recievername[MAX_PLAYERS];
        new reason[50];
        new string[120];
        if(sscanf(params,"us[50]",targetid,reason)) return SendClientMessage(playerid,COLOR_RED,"[ERROR]/ban [PlayerID] [REASON]");
        GetPlayerName(playerid,sendername,sizeof(sendername)); // As Vince said, it should be done after you find out who the target player is.
        GetPlayerName(targetid,recievername,sizeof(recievername));
        format(string,sizeof(string),"Administrator %s has banned %s reason: %s.",sendername,recievername,reason);
        SendClientMessageToAll(COLOR_RED,string);
        SavePlayer(targetid);
        Ban(targetid);
    }
    else
    {
        SendClientMessage(playerid,COLOR_RED,"[SERVER]:You are not allowed to use this command");
    }
    return 1;
}
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)