SA-MP Forums Archive
/am problem - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+--- Thread: /am problem (/showthread.php?tid=560897)


/am problem - Ghost110 - 31.01.2015

Hello, i have a little problem here. I have this cmd, but everyone can see /am messages, and the admins should be the only ones able to see it!


pawn Код:
CMD:am(playerid, params[])
{
    if(PlayerInfo[playerid][pAdmin] >= 1)
    {
        new toid, message[80];
        if(sscanf(params, "us[80]", toid, message)) return SendClientMessageEx(playerid, COLOR_WHITE,"USAGE: /[a]dmin[m]essage [playerid] [message]");
        if(strlen(message) > 80) return SendClientMessageEx(playerid, COLOR_RED,"Your message cannot be longer than 80 characters.");
        if(!IsPlayerConnected(toid)) return SendClientMessageEx(playerid, COLOR_RED,"That player is not connected!");
        new string[128];
        format(string, sizeof(string), "Admin message from %s: %s", GetPlayerNameEx(playerid), message);
        SendClientMessageEx(AMReplyId[playerid], COLOR_GREEN, string);
        for(new i=0; i<MAX_PLAYERS; i++)
        {
            if(PlayerInfo[i][pAdmin] >= 1)
            {
                format(string, sizeof(string), "Admin %s [%d] > %s [%d]: %s", GetPlayerNameEx(playerid), playerid, GetPlayerNameEx(toid), toid, message);
                SendClientMessageEx(i,COLOR_GREEN,string);
            }
        }
    }
    else
    {
        new message[80];
        if(sscanf(params, "s[80]", message)) return SendClientMessageEx(playerid, COLOR_WHITE,"USAGE: /am [message]");
        if(strlen(message) > 80) return SendClientMessageEx(playerid, COLOR_RED,"Your message cannot be longer than 80 characters.");
        new string[128];
        for(new i=0; i<MAX_PLAYERS; i++)
        {
            if(PlayerInfo[i][pAdmin] >= 1)
            {
                format(string, sizeof(string), "Admin message from %s [%d]: %s", GetPlayerNameEx(playerid), playerid, message);
                SendClientMessageEx(i,COLOR_GREEN,string);
                AMReplyId[i] = playerid;
            }
        }
        format(string, sizeof(string), "Your message, %s, has been sent.", message);
        SendClientMessageEx(playerid, COLOR_GREEN, string);
        format(string, sizeof(string), "Player %s [%d] > Admins: %s", GetPlayerNameEx(playerid), playerid, message);
    }
    return 1;
}


Re: /am problem - Rabea - 31.01.2015

change this:
pawn Код:
SendClientMessageEx(i,COLOR_GREEN,string);
to this
pawn Код:
SendAdminMessage(i,COLOR_GREEN,string);
and if you don't have a stock for sendadminmessage
pawn Код:
stock SendAdminMessage(color, admlvl, string[])
{
    foreach(Player, i)
    {
        if(IsPlayerLoggedIn(i) && PlayerInfo[i][pAdmin] >= admlvl)
        {
            SendClientMessage(i, color, string);
        }
    }
    return 1;
}


Re: /am problem - ATGOggy - 31.01.2015

Try this:

Add this on top if you don't have isnull stock:
PHP код:
#if !defined isnull
    #define isnull(%1) \
                ((!(%1[0])) || (((%1[0]) == '\1') && (!(%1[1]))))
#endif 
Command:
PHP код:
CMD:am(playerid, params[])
{
    new toid, message[80];
    if(isnull(params)) return SendClientMessage(playerid, COLOR_WHITE, "USAGE: /am [id/name] [message] or /am [message]");
    if(sscanf(params, "us[80]", toid, message) 
    {
        if(sscanf(params, "s[80]", message)) return SendClientMessageEx(playerid, COLOR_WHITE,"USAGE: /am [message]");
        if(strlen(message) > 80) return SendClientMessageEx(playerid, COLOR_RED,"Your message cannot be longer than 80 characters.");
        new string[128];
        for(new i=0; i<MAX_PLAYERS; i++)
        {
            if(PlayerInfo[i][pAdmin] >= 1)
            {
                format(string, sizeof(string), "Admin message from %s [%d]: %s", GetPlayerNameEx(playerid), playerid, message);
                SendClientMessageEx(i,COLOR_GREEN,string);
                AMReplyId[i] = playerid;
            }
        }
        format(string, sizeof(string), "Your message, %s, has been sent.", message);
        SendClientMessageEx(playerid, COLOR_GREEN, string);
        format(string, sizeof(string), "Player %s [%d] > Admins: %s", GetPlayerNameEx(playerid), playerid, message);
    }
    else
    {
        if(strlen(message) > 80) return SendClientMessageEx(playerid, COLOR_RED,"Your message cannot be longer than 80 characters.");
        if(!IsPlayerConnected(toid)) return SendClientMessageEx(playerid, COLOR_RED,"That player is not connected!");
        new string[128];
        format(string, sizeof(string), "Admin message from %s: %s", GetPlayerNameEx(playerid), message);
        SendClientMessageEx(AMReplyId[playerid], COLOR_GREEN, string);
        for(new i=0; i<MAX_PLAYERS; i++)
        {
            if(PlayerInfo[i][pAdmin] >= 1)
            {
                format(string, sizeof(string), "Admin %s [%d] > %s [%d]: %s", GetPlayerNameEx(playerid), playerid, GetPlayerNameEx(toid), toid, message);
                SendClientMessageEx(i,COLOR_GREEN,string);
            }
        }
    }
    return 1;
} 


Re: /am problem - Ghost110 - 01.02.2015

@ATGOggy -
Код:
C:\Users\GHT\Desktop\OARPsv\OARP\gamemodes\OARP.pwn(40693) : error 001: expected token: ")", but found "{"
Line:
Код:
new toid, message[80];
    if(isnull(params)) return SendClientMessage(playerid, COLOR_WHITE, "USAGE: /am [id/name] [message] or /am [message]");
    if(sscanf(params, "us[80]", toid, message)
 [[[HERE]]]] ->   {
        if(sscanf(params, "s[80]", message)) return SendClientMessageEx(playerid, COLOR_WHITE,"USAGE: /am [message]");
        if(strlen(message) > 80) return SendClientMessageEx(playerid, COLOR_RED,"Your message cannot be longer than 80 characters.");
        new string[128];
        for(new i=0; i<MAX_PLAYERS; i++)


Re: /am problem - Primard - 01.02.2015

Change this
Код:
 [[[HERE]]]] ->   {
to
Код:
 [[[HERE]]]] ->   )


Re: /am problem - lwilson - 01.02.2015

Command Copyright from P:LA
You should copy functions and others stock need for that COMMAND.

Re: /am problem - ATGOggy - 01.02.2015

Change this:
pawn Код:
if(sscanf(params, "us[80]", toid, message)
to this:[pawn if(sscanf(params, "us[80]", toid, message)) [/psawn]

Re: /am problem - CalvinC - 01.02.2015

Quote:

expected token: ")", but found "{"
You should read the error message, it expected to find a " ) ", but instead the next thing it found, was a " { ".
Which would indicate you forgot to put a " ) " in that place.