message multiple - Printable Version

message multiple - 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: message multiple (/showthread.php?tid=645275)

message multiple - rakinz - 24.11.2017

Код:

CMD:warn(playerid, params[])
{
	if(PlayerInfo[playerid][pAdmin] == 0)
		return SendClientMessage(playerid, COLOR_RED, "ERROR: Insufficient Permissions!");
		
	new PID, str[128], reason[64], name[MAX_PLAYER_NAME], Adminname[MAX_PLAYER_NAME];
	if(sscanf(params, "us[64]", PID,reason))
		return SendClientMessage(playerid, COLOR_LIGHTGREEN, "Usage: /Warn [playerid] [reason]");

	if(!IsPlayerConnected(PID))
		return SendClientMessage(playerid, COLOR_RED, "Player is not connected!");

	playerWarn[PID] ++;
	GetPlayerName(playerid, Adminname, sizeof(Adminname));
	GetPlayerName(PID, name, sizeof(name));
	format(str, sizeof str, "Admin %s has warned %s, Reason: %s. [Warning %i/3]", Adminname, PID, reason, playerWarn[PID]);
	SendClientMessageToAll(COLOR_RED, str);
	format(str, sizeof str, "Admin %s has warned %s, Reason: %s. [Warning %i/3]", Adminname, PID, reason, playerWarn[PID]);
	SendClientMessageToAll(COLOR_RED, str);
	if(playerWarn[PID] > 2)
	{
		format(str, sizeof str, "%s has been kicked by server 3 warnings, Reason [%s].", name, reason);
			SendClientMessageToAll(COLOR_RED, str);
			SendClientMessage(PID, COLOR_RED, "You were warned three times and got kicked!");
		Kick(PID);
	}
	return 1;
}

Hey, the command works just fine and compiling as well but I'm getting a multiple message like :

Admin %s has warned %s, Reason: %s. [Warning %i/3]
Admin %s has warned %s, Reason: %s. [Warning %i/3]

Or:

%s has been kicked by server 3 warnings, Reason [%s]
%s has been kicked by server 3 warnings, Reason [%s]

I need to put the 'ClientMessage' out of the loop, anyone knows how?

Re: message multiple - Kane - 24.11.2017

Not sure what you're asking if you could rephrase it but:

PID isn't a string. You're doing:

PHP код:



GetPlayerName(PID, name, sizeof(name));

but you aren't actually using name but PID instead.

Код:

format(str, sizeof str, "Admin %s has warned %s, Reason: %s. [Warning %i/3]", Adminname, PID, reason, playerWarn[PID]);

Replace it with "name".

Re: message multiple - rakinz - 24.11.2017

Quote:

Originally Posted by Arthur Kane

Not sure what you're asking if you could rephrase it but:

PID isn't a string. You're doing:

PHP код:



GetPlayerName(PID, name, sizeof(name));

but you aren't actually using name but PID instead.

Код:

format(str, sizeof str, "Admin %s has warned %s, Reason: %s. [Warning %i/3]", Adminname, PID, reason, playerWarn[PID]);

Replace it with "name".

That isn't the problem here,

Let me show you that with an image:

https://imgur.com/a/C3SDX

Re: message multiple - Kraeror - 24.11.2017

Here you are, +1 REP if I helped

PHP код:



CMD:warn(playerid, params[])

{

    if(PlayerInfo[playerid][pAdmin] == 0)

        return SendClientMessage(playerid, COLOR_RED, "ERROR: Insufficient Permissions!");


    new PID, str[128], reason[64], name[MAX_PLAYER_NAME], Adminname[MAX_PLAYER_NAME];

    if(sscanf(params, "us[64]", PID,reason))

        return SendClientMessage(playerid, COLOR_LIGHTGREEN, "Usage: /Warn [playerid] [reason]");


    if(!IsPlayerConnected(PID))

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


    playerWarn[PID] ++;

    GetPlayerName(playerid, Adminname, sizeof(Adminname));

    GetPlayerName(PID, name, sizeof(name));

    format(str, sizeof str, "Admin %s has warned %s, Reason: %s. [Warning %i/3]", Adminname, PID, reason, playerWarn[PID]);

    SendClientMessageToAll(COLOR_RED, str);

    if(playerWarn[PID] > 2)

    {

        format(str, sizeof str, "%s has been kicked by server 3 warnings, Reason [%s].", name, reason);

            SendClientMessageToAll(COLOR_RED, str);

            SendClientMessage(PID, COLOR_RED, "You were warned three times and got kicked!");

        Kick(PID);

    }

    return 1;

}

Re: message multiple - TaiRinsuru - 24.11.2017

Actually look at your code, you have 2 copies of it pretty sure it will duplicate.

Re: message multiple - Lucases - 24.11.2017

It's normal, you are sending the message to all once and then to the warned player again.

Add this at top of your script:

pawn Код:

stock SendClientMessageToAllEx(exception, color, const message[])
{
  for(new i; i<MAX_PLAYERS; i++)
  {
    if(IsPlayerConnected(i))
    {
      if(i != exception)
      {
        SendClientMessage(i, color, message);
      }
    }
  }
}

Then remove the line which sends the message to PID.
And change SendClientMessageToAll with SendMessageToAllEx with PID as first parameter