[CalvinC]

Just started a fresh script, starting with OnPlayerText

pawn Код:

public OnPlayerText(playerid, text[])
{
    new string[128];
    format(string, sizeof(string), "%s(%s): %s", GetPlayerName(playerid), GetPlayerID(playerid), text);
    SendClientMessageToAll(COLOR_WHITE, string);
    return 1;
}

It gives me these:
warning 202: number of arguments does not match definition
warning 202: number of arguments does not match definition
error 035: argument type mismatch (argument 1)

From the format line.

[jackx3rx]

pawn Код:

format(string, sizeof(string), "%s(%s): %s", GetPlayerName(playerid), GetPlayerID(playerid), text);

The players ID is an integer, not a string, try this:

pawn Код:

new playername[MAX_PLAYER_NAME], string[128];
GetPlayerName(playerid,playername,sizeof(playername));
format(string, sizeof(string), "%s(%i): %s", playername, playerid, text);
SendClientMessageToAll(COLOR_WHITE,string);

- Theres no need for GetPlayerID, since 'playerid' is exactly the same.
- GetPlayerName doesn't work like you did, but I fixed it for you.

[Glossy42O]

Eh, not good at scripting but i'll try

new string[128];
format(string, sizeof(string), "%s(%d): %d", name, GetPlayerID(playerid), text);
SendClientMessageToAll(COLOR_WHITE, str);

I'm new so.. ehh


If not working try that..

new name[MAX_PLAYER_NAME], str[128];
format(string, sizeof(string), "%s(%s): %s", GetPlayerName(playerid), GetPlayerID(playerid), text);
SendClientMessageToAll(COLOR_WHITE, string);

or...

new string[128];
format(string, sizeof(string), "%s(%i): %d", GetPlayerName(playerid), GetPlayerID(playerid), text);
SendClientMessageToAll(COLOR_WHITE, string);

[CalvinC]

Thanks Jack, but i still get the warnings.

[jackx3rx]

pawn Код:

public OnPlayerText(playerid, text[])
{
    new string[128], playername[MAX_PLAYER_NAME];
    GetPlayerName(playerid,playername,sizeof(playername));
    format(string, sizeof(string), "%s(%u): %s", playername, playerid, text);
    SendClientMessageToAll(-1, string);
    return 1;
}

Try this, I replaced the %i with %u which is playerid, it SHOULD work. My bad before.

[CalvinC]

Sorry i didn't see what you wrote about "GetPlayerID(playerid)". That was the problem. :P
But how do i make the old text message not show, and make it show the new one instead?
Now it shows both.

[jackx3rx]

pawn Код:

public OnPlayerText(playerid, text[])
{
    new string[128], playername[MAX_PLAYER_NAME];
    GetPlayerName(playerid,playername,sizeof(playername));
    format(string, sizeof(string), "%s(%u): %s", playername, playerid, text);
    SendClientMessageToAll(-1, string);
    return 0;
}

Make it return 0 rather than 1.

[CalvinC]

Alright, thanks!

[JaydenJason]

Quote:

Originally Posted by jackx3rx pawn Код: public OnPlayerText(playerid, text[]) {     new string[128], playername[MAX_PLAYER_NAME];     GetPlayerName(playerid,playername,sizeof(playername));     format(string, sizeof(string), "%s(%u): %s", playername, playerid, text);     SendClientMessageToAll(-1, string);     return 0; } Make it return 0 rather than 1.

Why return 0; return 0 is used for failed commands right?

[Facerafter]

Quote:

Originally Posted by JaydenJason Why return 0; return 0 is used for failed commands right?

They want to replace the original text, therefore the original text has to be cancelled from sending and a new has to be created & sent. The return 0; at OnPlayerText cancels the original message.