Whats Wrong With This? - Printable Version

Whats Wrong With This? - 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: Whats Wrong With This? (/showthread.php?tid=318728)

Whats Wrong With This? - Littlehelper - 16.02.2012

Hello,
There is a setlevel command, made by my friend.
But when i tried to /setlevel myid level it says unknown command...
here's the code.

Код:

CMD:setlevel(playerid, params[])
{
    new string[128], id, level, Name[MAX_PLAYER_NAME], AdminName[MAX_PLAYER_NAME];
    GetPlayerName(playerid, AdminName, sizeof(AdminName));
    GetPlayerName(id, Name, sizeof(Name));
    if((PlayerInfo[playerid][PLAYER_LEVEL] == 5) || (PlayerInfo[playerid][PLAYER_LEVEL] <= 5 && IsPlayerAdmin(playerid))) {
        if(sscanf(params, "ri", id, level)) return SendClientMessage(playerid, COLOR_YELLOW, "Usage: /setlevel <ID/Part Of Name> <Level>");
        if(!PlayerInfo[id][PLAYER_LOGGED]) return SendClientMessage(playerid, COLOR_RED, "That player isn't logged in!");
        if(id == INVALID_PLAYER_ID) return SendClientMessage(playerid, COLOR_RED, "That player isn't online!");
        if(level > 5 || level < 0) return SendClientMessage(playerid, COLOR_RED, "The level must be between 0 and 5.");
        PlayerInfo[id][PLAYER_LEVEL] = level;
        format(string, sizeof(string), "You Have Been Set To %d Administrator By %s!", level, AdminName);
        SendClientMessage(id, COLOR_YELLOW, string);
        format(string, sizeof(string), "%s Has Successsfully Been Set To Level %d Administrator!", Name, level);
        return SendClientMessage(playerid, COLOR_YELLOW, string);
    }
    else return SendClientMessage(playerid, COLOR_RED, "You Are Not An Administrator!");
}

Whats Wrong?

Re: Whats Wrong With This? - ReneG - 16.02.2012

Try this.

pawn Код:

CMD:setlevel(playerid, params[])
{
    new string[128], id, level, Name[MAX_PLAYER_NAME], AdminName[MAX_PLAYER_NAME];
    GetPlayerName(playerid, AdminName, sizeof(AdminName));
    GetPlayerName(id, Name, sizeof(Name));
    if!IsPlayerAdmin(playerid)) return return SendClientMessage(playerid, COLOR_RED, "You Are Not An Administrator!");
    if(sscanf(params, "ui", id, level)) return SendClientMessage(playerid, COLOR_YELLOW, "Usage: /setlevel <ID/Part Of Name> <Level>");
    if(!PlayerInfo[id][PLAYER_LOGGED]) return SendClientMessage(playerid, COLOR_RED, "That player isn't logged in!");
    if(id == INVALID_PLAYER_ID) return SendClientMessage(playerid, COLOR_RED, "That player isn't online!");
    if(level > 5 || level < 0) return SendClientMessage(playerid, COLOR_RED, "The level must be between 0 and 5.");
    else
    {
        PlayerInfo[id][PLAYER_LEVEL] = level;
        format(string, sizeof(string), "You Have Been Set To %d Administrator By %s!", level, AdminName);
        SendClientMessage(id, COLOR_YELLOW, string);
        format(string, sizeof(string), "%s Has Successsfully Been Set To Level %d Administrator!", Name, level);
        SendClientMessage(playerid, COLOR_YELLOW, string);
    }
    return 1;
}

Re: Whats Wrong With This? - Littlehelper - 16.02.2012

Quote:

Originally Posted by VincentDunn

Try this.

pawn Код:

CMD:setlevel(playerid, params[])
{
    new string[128], id, level, Name[MAX_PLAYER_NAME], AdminName[MAX_PLAYER_NAME];
    GetPlayerName(playerid, AdminName, sizeof(AdminName));
    GetPlayerName(id, Name, sizeof(Name));
    if!IsPlayerAdmin(playerid)) return return SendClientMessage(playerid, COLOR_RED, "You Are Not An Administrator!");
    if(sscanf(params, "ui", id, level)) return SendClientMessage(playerid, COLOR_YELLOW, "Usage: /setlevel <ID/Part Of Name> <Level>");
    if(!PlayerInfo[id][PLAYER_LOGGED]) return SendClientMessage(playerid, COLOR_RED, "That player isn't logged in!");
    if(id == INVALID_PLAYER_ID) return SendClientMessage(playerid, COLOR_RED, "That player isn't online!");
    if(level > 5 || level < 0) return SendClientMessage(playerid, COLOR_RED, "The level must be between 0 and 5.");
    else
    {
        PlayerInfo[id][PLAYER_LEVEL] = level;
        format(string, sizeof(string), "You Have Been Set To %d Administrator By %s!", level, AdminName);
        SendClientMessage(id, COLOR_YELLOW, string);
        format(string, sizeof(string), "%s Has Successsfully Been Set To Level %d Administrator!", Name, level);
        SendClientMessage(playerid, COLOR_YELLOW, string);
    }
    return 1;
}

why is there return return sendclientmessage? and + there is no (, On (!IsPlayerAdmin).
Well gonna test it.

Re: Whats Wrong With This? - Mark™ - 16.02.2012

Quote:

Originally Posted by Littlehelper[MDZ]

Hello,
There is a setlevel command, made by my friend.
But when i tried to /setlevel myid level it says unknown command...
here's the code.

Код:

CMD:setlevel(playerid, params[])
{
    new string[128], id, level, Name[MAX_PLAYER_NAME], AdminName[MAX_PLAYER_NAME];
    GetPlayerName(playerid, AdminName, sizeof(AdminName));
    GetPlayerName(id, Name, sizeof(Name));
    if((PlayerInfo[playerid][PLAYER_LEVEL] == 5) || (PlayerInfo[playerid][PLAYER_LEVEL] <= 5 && IsPlayerAdmin(playerid))) {
        if(sscanf(params, "ri", id, level)) return SendClientMessage(playerid, COLOR_YELLOW, "Usage: /setlevel <ID/Part Of Name> <Level>");
        if(!PlayerInfo[id][PLAYER_LOGGED]) return SendClientMessage(playerid, COLOR_RED, "That player isn't logged in!");
        if(id == INVALID_PLAYER_ID) return SendClientMessage(playerid, COLOR_RED, "That player isn't online!");
        if(level > 5 || level < 0) return SendClientMessage(playerid, COLOR_RED, "The level must be between 0 and 5.");
        PlayerInfo[id][PLAYER_LEVEL] = level;
        format(string, sizeof(string), "You Have Been Set To %d Administrator By %s!", level, AdminName);
        SendClientMessage(id, COLOR_YELLOW, string);
        format(string, sizeof(string), "%s Has Successsfully Been Set To Level %d Administrator!", Name, level);
        return SendClientMessage(playerid, COLOR_YELLOW, string);
    }
    else return SendClientMessage(playerid, COLOR_RED, "You Are Not An Administrator!");
}

Whats Wrong?

Did your friend created this cmd with chaoz's scripting machine ? well the 1.1 version contains some code bugs.

Re: Whats Wrong With This? - Konstantinos - 16.02.2012

Quote:

Originally Posted by Littlehelper[MDZ]

why is there return return sendclientmessage? and + there is no (, On (!IsPlayerAdmin).
Well gonna test it.

The "(" was typo and about the return

pawn Код:

SendClientMessage( .. );
return 1;

It's same as

pawn Код:

return SendClientMessage( .. );

Re: Whats Wrong With This? - Littlehelper - 16.02.2012

Quote:

Originally Posted by Xtreme_playa

Did your friend created this cmd with chaoz's scripting machine ? well the 1.1 version contains some code bugs.

The code was created by Dwane, dont know if he created with any scripting machine,
he created this when he gave me administration system.
EDIT: the code doesnt works, it says the player is not logged in, even when i'm already logged in....

Re: Whats Wrong With This? - Mark™ - 16.02.2012

Quote:

Originally Posted by Littlehelper[MDZ]

no i meant the code you posted on the #1 post

Re: Whats Wrong With This? - Konstantinos - 16.02.2012

When I gave you the Administrator System I made for you, I test everything and they worked.
And lol I am not noob to use script machines..
Make sure that if you edit it you didn't delete the

pawn Код:

PlayerInfo[playerid][PLAYER_LOGGED] = 1;

in OnDialogResponse callback on the login case.

Re: Whats Wrong With This? - Littlehelper - 16.02.2012

Quote:

Originally Posted by Xtreme_playa

no i meant the code you posted on the #1 post

Yes, i meant the same code.

Re: Whats Wrong With This? - ReneG - 16.02.2012

pawn Код:

CMD:setlevel(playerid, params[])
{
    new string[128], id, level, Name[MAX_PLAYER_NAME], AdminName[MAX_PLAYER_NAME];
    GetPlayerName(playerid, AdminName, sizeof(AdminName));
    GetPlayerName(id, Name, sizeof(Name));
    if(!IsPlayerAdmin(playerid)) return SendClientMessage(playerid, COLOR_RED, "You Are Not An Administrator!");
    if(sscanf(params, "ui", id, level)) return SendClientMessage(playerid, COLOR_YELLOW, "Usage: /setlevel <ID/Part Of Name> <Level>");
    if(!PlayerInfo[id][PLAYER_LOGGED]) return SendClientMessage(playerid, COLOR_RED, "That player isn't logged in!");
    if(id == INVALID_PLAYER_ID) return SendClientMessage(playerid, COLOR_RED, "That player isn't online!");
    if(level > 5 || level < 0) return SendClientMessage(playerid, COLOR_RED, "The level must be between 0 and 5.");
    else
    {
        PlayerInfo[id][PLAYER_LEVEL] = level;
        format(string, sizeof(string), "You Have Been Set To %d Administrator By %s!", level, AdminName);
        SendClientMessage(id, COLOR_YELLOW, string);
        format(string, sizeof(string), "%s Has Successsfully Been Set To Level %d Administrator!", Name, level);
        SendClientMessage(playerid, COLOR_YELLOW, string);
    }
    return 1;
}

Whoops, I made a little bit of a typo. Fixed.