if(strcmp(cmd, "/mask", true) == 0)
{
if(PlayerInfo[playerid][pMember] != 8 || PlayerInfo[playerid][pMember] != 11)
{
SendClientMessage(playerid, COLOR_GREY, " You are not a member of the Hitman Agency / Bratva !");
return 1;
}
if(PlayerInfo[playerid][pMask] == 0)
{
PlayerInfo[playerid][pMask] = 1;
foreach (Player, i)
{
if(IsPlayerConnected(i))
{
ShowPlayerNameTagForPlayer(i, playerid, 0);
}
}
strmid(sendername, PlayerRPName(playerid), 0, MAX_PLAYER_NAME);
format(string, sizeof(string), "* %s has put a mask on.", sendername);
ProxDetector(15.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
}
else if(PlayerInfo[playerid][pMask] == 1)
{
PlayerInfo[playerid][pMask] = 0;
foreach (Player, i)
{
if(IsPlayerConnected(i))
{
ShowPlayerNameTagForPlayer(i, playerid, 1);
}
}
strmid(sendername, PlayerRPName(playerid), 0, MAX_PLAYER_NAME);
format(string, sizeof(string), "* %s has put their mask away.", sendername);
ProxDetector(15.0, playerid, string, COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE,COLOR_PURPLE);
}
}
You are checking if the player not 8 OR not 11, so if for example the player is in faction 8, he is not in faction 11, so he will still get the error-message.
The way to go is to check if the player is not from 8 AND not from 11, so use && instead of || |