[Jing_Chan]

pawn Код:

if(PlayerData[playerid][pFactionRank] > FactionData[PlayerData[playerid][pFaction]][factionEditrank] || PlayerData[playerid][pAdmin] < 5)
        return SendClientMessage(playerid, COLOR_ADM, "ACCESS DENIED:{FFFFFF} Your rank doesn't have permission to do this.");

For some reason, even as above level 5 admin it doesn't let me do the command? I'm obviously failing to see the obvious, can someone help please.

[GTLS]

Try this.

PHP код:

if((PlayerData[playerid][pFactionRank] > FactionData[PlayerData[playerid][pFaction]][factionEditrank]) && PlayerData[playerid][pAdmin] < 5)
        return SendClientMessage(playerid, COLOR_ADM, "ACCESS DENIED:{FFFFFF} Your rank doesn't have permission to do this."); 


[Jing_Chan]

Quote:

Originally Posted by GTLS Try this. PHP код: if((PlayerData[playerid][pFactionRank] > FactionData[PlayerData[playerid][pFaction]][factionEditrank]) && PlayerData[playerid][pAdmin] < 5)         return SendClientMessage(playerid, COLOR_ADM, "ACCESS DENIED:{FFFFFF} Your rank doesn't have permission to do this.");

Wouldn't that require both structures to be correct in order to work?

Код:

&&	and	if (Left && Right)

[Exhibit]

If this is a command show us the full code.

[Mike861]

Try PlayerData[playerid] [pAdmin] <= 5

[Jing_Chan]

Quote:

Originally Posted by Exhibit If this is a command show us the full code.

pawn Код:

CMD:configfaction(playerid, params[])
{
    if(!PlayerData[playerid][pFaction])
        return SendClientMessage(playerid, COLOR_ADM, "ERROR: You aren't in a faction.");

    if(PlayerData[playerid][pFactionRank] > FactionData[PlayerData[playerid][pFaction]][factionEditrank] || PlayerData[playerid][pAdmin] < 5)
        return SendClientMessage(playerid, COLOR_ADM, "ACCESS DENIED:{FFFFFF} Your rank doesn't have permission to do this.");

    ShowYourFactionMenu(playerid);
    return 1;
}

[Exhibit]

PHP код:

CMD:configfaction(playerid, params[])
{
    if(!PlayerData[playerid][pFaction])
        return SendClientMessage(playerid, COLOR_ADM, "ERROR: You aren't in a faction.");
    if(PlayerData[playerid][pFactionRank] > FactionData[PlayerData[playerid][pFaction]][factionEditrank])
        return SendClientMessage(playerid, COLOR_ADM, "ACCESS DENIED:{FFFFFF} Your rank doesn't have permission to do this.");
        if(PlayerData[playerid][pAdmin] < 5) return SendClientMessage(playerid, COLOR_ADM, "ACCESS DENIED:{FFFFFF} Your rank doesn't have permission to do this. ADMIN");
    ShowYourFactionMenu(playerid);
    return 1;
} 


try this.

[jasperschellekens]

Are you sure that this:
PlayerData[playerid][pAdmin]
Holds the correct value?
Try printing it to be sure, your code looks ok. It should work

Quote:

Originally Posted by Exhibit try this.

Im pretty sure the topic starter knows this method. He clearly doesn't want to do it this way hence he is using || (means or in an if-statement).
The way you showed him is repeating code and his method using ||/or should work.

[Jing_Chan]

Quote:

Originally Posted by jasperschellekens Are you sure that this: PlayerData[playerid][pAdmin] Holds the correct value? Try printing it to be sure, your code looks ok. It should work Im pretty sure the topic starter knows this method. He clearly doesn't want to do it this way hence he is using || (means or in an if-statement). The way you showed him is repeating code and his method using ||/or should work.

It does hold the correct value. I'm not sure why it isn't working - trying to figure ways around it.

[GTLS]

Quote:

Originally Posted by Jing_Chan Wouldn't that require both structures to be correct in order to work? Код: && and if (Left && Right)

Yes and thats what you should achieve here. If Player is admin less than 5 and he is also less than edit rank, error shows up.