[CallumIsDaBeast]

Right the error i get:

Код:

C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1248) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1248) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1248) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1248) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1249) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1249) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1253) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1253) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1253) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1253) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1254) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1254) : warning 202: number of arguments does not match definition
C:\Users\Gamer\Desktop\Goodly\gamemodes\Millenium.pwn(1258) : error 030: compound statement not closed at the end of file (started at line 1243)
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


1 Error.

Then the code:

Код:

CMD:makehelper(playerid, params[])
{
	new advlvl, id, str[128], string[128];
    if(PlayerInfo[playerid][pAdmin] < 6 && PlayerInfo[playerid][pPAStaff] == 0 && PlayerInfo[playerid][pHelper] < 4 && PlayerInfo[playerid][pHR] == 0) return SendClientMessage(playerid, COLOR_GREY, "You are not authorized to use that command.");
	if (sscanf(params, "ui", id, advlvl)) return SendClientMessage(playerid, COLOR_GREY, "SYNTAX: /makehelper [id] [Helperlevel]");
	if(PlayerInfo[id][pStaffBanned] == 1) return SendClientMessage(playerid, COLOR_GREY, "That user is staff banned!");
 	if (id == INVALID_PLAYER_ID) return SendClientMessage(playerid, COLOR_GREY, "This player is not connected");
 	if (advlvl == 0) { PlayerInfo[playerid][pNewbies] = 0; }
    if (advlvl > 5 || advlvl < 0) return SendClientMessage(playerid, COLOR_GREY,"Helper levels reach from '0' to '5'");
    {
        if(PlayerInfo[id][pHelper] != advlvl) // 1243
        {
            if(PlayerInfo[id][pAdmin] > 1) return SendClientMessage(playerid, COLOR_GREY, "You cannot make admins Helpers.");
	        if(PlayerInfo[id][pHelper] < advlvl)
	        {
	        	format(str,128,"AdmCmd: %s has promoted %s to helper level %i", GetPlayerName(playerid), GetPlayerName(id), advlvl);
	        	format(string,128,"You have been promoted to helper level %i by administrator %s", advlvl, GetPlayerName(playerid));
			}
			else if(PlayerInfo[id][pHelper] > advlvl)
			{
	        	format(str,128,"AdmCmd: %s has demoted %s to helper level %i", GetPlayerName(playerid), GetPlayerName(id), advlvl);
	        	format(string,128,"You have been demoted to helper level %i by administrator %s", advlvl, GetPlayerName(playerid));// 1254
				}
			return 1;
}

// No 1258



__________________________________________________ ___
Owner of Millenium Roleplay!
CallumDaBest - Got hacked
Update on MRP: Next week - I got hacked so i got delayed!
__________________________________________________ ___

[Matess]

Closing brackets?

pawn Код:

CMD:makehelper(playerid, params[])
{
    new advlvl, id, str[128], string[128];
    if(PlayerInfo[playerid][pAdmin] < 6 && PlayerInfo[playerid][pPAStaff] == 0 && PlayerInfo[playerid][pHelper] < 4 && PlayerInfo[playerid][pHR] == 0) return SendClientMessage(playerid, COLOR_GREY, "You are not authorized to use that command.");
    if (sscanf(params, "ui", id, advlvl)) return SendClientMessage(playerid, COLOR_GREY, "SYNTAX: /makehelper [id] [Helperlevel]");
    if(PlayerInfo[id][pStaffBanned] == 1) return SendClientMessage(playerid, COLOR_GREY, "That user is staff banned!");
    if (id == INVALID_PLAYER_ID) return SendClientMessage(playerid, COLOR_GREY, "This player is not connected");
    if (advlvl == 0) { PlayerInfo[playerid][pNewbies] = 0; }
    if (advlvl > 5 || advlvl < 0) return SendClientMessage(playerid, COLOR_GREY,"Helper levels reach from '0' to '5'");
    {
        if(PlayerInfo[id][pHelper] != advlvl) // 1243
        {
            if(PlayerInfo[id][pAdmin] > 1) return SendClientMessage(playerid, COLOR_GREY, "You cannot make admins Helpers.");
            if(PlayerInfo[id][pHelper] < advlvl)
            {
                format(str,128,"AdmCmd: %s has promoted %s to helper level %i", GetPlayerName(playerid), GetPlayerName(id), advlvl);
                format(string,128,"You have been promoted to helper level %i by administrator %s", advlvl, GetPlayerName(playerid));
            }
            else if(PlayerInfo[id][pHelper] > advlvl)
            {
                format(str,128,"AdmCmd: %s has demoted %s to helper level %i", GetPlayerName(playerid), GetPlayerName(id), advlvl);
                format(string,128,"You have been demoted to helper level %i by administrator %s", advlvl, GetPlayerName(playerid));// 1254
            }
        }
     }
            return 1;
}

[Corekt]

Your usage of GetPlayerName is incorrect. The function doesn't return a string (which means you can't use it directly in format). You need to declare a string to be passed by reference.

pawn Код:

new
    name[MAX_PLAYER_NAME],
    tname[MAX_PLAYER_NAME];

GetPlayerName(playerid, name, sizeof(name));
GetPlayerName(id, tname, sizeof(tname));
format(str,128,"AdmCmd: %s has promoted %s to helper level %i", name, tname, advlvl);

// OR if you don't want to declare a new variable every time,
// you can create a separate name getter function to use directly in format.

stock GetPlayerNameEx(playerid)
{
    new name[MAX_PLAYER_NAME];
    GetPlayerName(playerid, name, sizeof(name));
    return name;
}

CMD:makehelper(playerid, params[])
{
    // [...]
    format(str,128,"AdmCmd: %s has promoted %s to helper level %i", GetPlayerNameEx(playerid), GetPlayerNameEx(id), advlvl);
    // [...]
}