foreach(Player, i)
{
if(PlayerInfo[i][pFaction] == PlayerInfo[playerid][pFaction] && PlayerInfo[i][pFaction] != 0)
{
format(string, sizeof(string), "*** %s from your faction has loggedin", GetPlayerNameEx(playerid));
SendClientMessage(i, COLOR_JUNIORADMIN, string);
}
if(PlayerInfo[i][pGang] == PlayerInfo[playerid][pGang] && PlayerInfo[i][pGang] != 255)
{
format(string, sizeof(string), "*** %s from your gang has loggedin", GetPlayerNameEx(playerid));
SendClientMessage(i, COLOR_JUNIORADMIN, string);
}
}
for(new i = 0; i < GetPlayerPoolSize(); i++)
{
if(PlayerInfo[i][pFaction] == PlayerInfo[playerid][pFaction] && PlayerInfo[i][pFaction] != 0)
{
new Name[MAX_PLAYER_NAME], string[100];
GetPlayerName(playerid, Name, sizeof(Name));
format(string, sizeof(string), "*** %s from your faction has logged in", Name);
SendClientMessage(i, COLOR_JUNIORADMIN, string);
}
if(PlayerInfo[i][pGang] == PlayerInfo[playerid][pGang] && PlayerInfo[i][pGang] != 0)
{
new Name[MAX_PLAYER_NAME], string[100];
GetPlayerName(playerid, Name, sizeof(Name));
format(string, sizeof(string), "*** %s from your gang has loggedin", Name);
SendClientMessage(i, COLOR_JUNIORADMIN, string);
}
}
Your code isn't an improvement, Arbico.
OP: Debug it. print() what needs to be called and check the server_logs after to see if it works. |
GetPlayerName(playerid, PlayerInfo[playerid][PlayerName], 24);
foreach(Player, i) { if(PlayerInfo[i][pFaction] == PlayerInfo[playerid][pFaction] && PlayerInfo[i][pFaction] != 0) { format(string, sizeof(string), "*** %s from your faction has loggedin", PlayerInfo[playerid][PlayerName]); SendClientMessage(i, COLOR_JUNIORADMIN, string); } if(PlayerInfo[i][pGang] == PlayerInfo[playerid][pGang] && PlayerInfo[i][pGang] != 255) { format(string, sizeof(string), "*** %s from your gang has loggedin", PlayerInfo[playerid][PlayerName]); SendClientMessage(i, COLOR_JUNIORADMIN, string); } }