SA-MP Forums Archive
/setadmin - 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: /setadmin (/showthread.php?tid=439915)


/setadmin - NathNathii - 27.05.2013

// Fixed.

AW: /setadmin - Blackazur - 27.05.2013

Код:
CMD:setadmin(playerid, params[])
{
    new OtherPlayer, Level, Msg[128], Name[24], AdminName[24], OldLevel;
    SendAdminText(playerid, "/setadmin", params);
    if (APlayerData[playerid][LoggedIn] == true)
    {
        if (APlayerData[playerid][PlayerLevel] >= 5)
        {
            if (sscanf(params, "ui", OtherPlayer, Level)) SendClientMessage(playerid, 0xFF0000AA, "Usage: \"/setlevel [id] [level]\"");
            else
            {
                if (IsPlayerConnected(OtherPlayer))
                {
                    OldLevel = APlayerData[OtherPlayer][PlayerLevel];
                    GetPlayerName(playerid, AdminName, sizeof(AdminName));
                    GetPlayerName(OtherPlayer, Name, sizeof(Name));
                    APlayerData[OtherPlayer][PlayerLevel] = Level;
                    }
                else return SendClientMessage(playerid, 0xFF0000FF, "{FF0000}Error: {FFFFFF}That player is not online");
            }
        }
  return 1;
}
try that.

Re: /setadmin - NathNathii - 27.05.2013

**REMOVED

Re: /setadmin - Faisal_khan - 27.05.2013

Missed a brace:
pawn Код:
CMD:setadmin(playerid, params[])
{
    new OtherPlayer, Level, Msg[128], Name[24], AdminName[24], OldLevel;
    SendAdminText(playerid, "/setadmin", params);
    if (APlayerData[playerid][LoggedIn] == true)
    {
        if (APlayerData[playerid][PlayerLevel] >= 5)
        {
            if (sscanf(params, "ui", OtherPlayer, Level)) SendClientMessage(playerid, 0xFF0000AA, "Usage: \"/setlevel [id] [level]\"");
            else
            {
                if (IsPlayerConnected(OtherPlayer))
                {
                    OldLevel = APlayerData[OtherPlayer][PlayerLevel];
                    GetPlayerName(playerid, AdminName, sizeof(AdminName));
                    GetPlayerName(OtherPlayer, Name, sizeof(Name));
                    APlayerData[OtherPlayer][PlayerLevel] = Level;
                }
                else return SendClientMessage(playerid, 0xFF0000FF, "{FF0000}Error: {FFFFFF}That player is not online");
            }
        }
    }
    return 1;
}
Show us your APlayerData array.

Re: /setadmin - NathNathii - 27.05.2013

// FIxed.

Re: /setadmin - Faisal_khan - 27.05.2013

There doesn't seem anything wrong with the indentation in this code please post the exact line.
For the second and third warning, use it somewhere now or just ignore it till you find a place to use.

Re: /setadmin - NathNathii - 27.05.2013

Fixed it, thanks.

Re: /setadmin - Pottus - 27.05.2013

Just a suggestion for you to make commands a bit easier when you have this....

[PAWN] if (APlayerData[playerid][LoggedIn] == true)
{
if (APlayerData[playerid][PlayerLevel] >= 5)
{[/PAWN

Try doing this for readability.

pawn Код:
#define CMDLEVEL(%0) if(APlayerData[playerid][PlayerLevel] < %0) return SendClientMessage(playerid, 0xFFFF0000, "You are not high enough level to use this command")

and 

#define LOGGEDIN()  if(!APlayerData[playerid][LoggedIn]) return SendClientMessage(playerid, 0xFFFF0000, "You are not logged in!")

now all you need to do in your commands

cmd:setlevel(playerid, params[])
{
    LOGGEDIN();
    CMDLEVEL(5);
}