CMD:aduty(playerid, params[])
{
if(PlayerInfo[playerid][pAdmin] < 2) return SendClientMessage(playerid, -1, "** This is an admin only command!");
if(PlayerInfo[playerid][pAdminDuty] == 0)
{
SendClientMessageEx(playerid, COLOR_YELLOW, "{FFFFFF} ** You are now on administrative duty! Remember to check /reports!");
PlayerInfo[playerid][pAdminDuty] = 1;
SetPlayerHealth(playerid, 100000);
SetPlayerArmour(playerid, 100000);
PlayerInfo[playerid][pChar] = GetPlayerSkin(playerid);
SetPlayerSkin(playerid, 217);
PlayerInfo[playerid][pModel] = 217;
if(PlayerInfo[playerid][pAdmin] == 1)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
if(PlayerInfo[playerid][pAdmin] == 2)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
if(PlayerInfo[playerid][pAdmin] == 3)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
if(PlayerInfo[playerid][pAdmin] == 4)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
if(PlayerInfo[playerid][pAdmin] == 1337)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
if(PlayerInfo[playerid][pAdmin] == 1338)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
if(PlayerInfo[playerid][pAdmin] == 99998)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
if(PlayerInfo[playerid][pAdmin] == 99999)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
new string[256];
format(string, sizeof(string), ""OFICIAL_COLOR"*Voltage-Gaming-Roleplay* {FFFFFF}%s {32CD32}Its in admin service {00AFA8}to provide support. (/report for assistance)", GetPlayerNameEx(playerid));
SendClientMessageToAllEx(COLOR_RED, string);
}
else
{
new string[256];
format(string, sizeof(string), ""OFICIAL_COLOR"*Voltage-Gaming-Roleplay* {FFFFFF}%s {FF0000}Its no more in admin service {00AFA8}and now its in Role!", GetPlayerNameEx(playerid));
SendClientMessageToAllEx(COLOR_RED, string);
SetPlayerColor(playerid, TEAM_HIT_COLOR);
SetPlayerSkin(playerid, PlayerInfo[playerid][pChar]);
SetPlayerHealth(playerid, 100);
SetPlayerArmour(playerid, 0);
PlayerInfo[playerid][pAdminDuty] = 0;
}
return 1;
}
CMD:aduty(playerid, params[])
{
if(PlayerInfo[playerid][pAdmin] < 2) return SendClientMessage(playerid, -1, "** This is an admin only command!");
if(PlayerInfo[playerid][pAdminDuty] == 0)
{
SendClientMessageEx(playerid, COLOR_YELLOW, "{FFFFFF} ** You are now on administrative duty! Remember to check /reports!");
PlayerInfo[playerid][pAdminDuty] = 1;
SetPlayerHealth(playerid, 100000);
SetPlayerArmour(playerid, 100000);
PlayerInfo[playerid][pChar] = GetPlayerSkin(playerid);
SetPlayerSkin(playerid, 217);
PlayerInfo[playerid][pModel] = 217;
if (PlayerInfo[playerid][pAdmin] >= 1)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
new string[256];
format(string, sizeof(string), ""OFICIAL_COLOR"*Voltage-Gaming-Roleplay* {FFFFFF}%s {32CD32}Is in admin service {00AFA8}to provide support. (/report for assistance)", GetPlayerNameEx(playerid));
SendClientMessageToAllEx(COLOR_RED, string);
}
else
{
new string[256];
format(string, sizeof(string), ""OFICIAL_COLOR"*Voltage-Gaming-Roleplay* {FFFFFF}%s {FF0000}Its no more in admin service {00AFA8}and now its in Role!", GetPlayerNameEx(playerid));
SendClientMessageToAllEx(COLOR_RED, string);
SetPlayerColor(playerid, TEAM_HIT_COLOR);
SetPlayerSkin(playerid, PlayerInfo[playerid][pChar]);
SetPlayerHealth(playerid, 100);
SetPlayerArmour(playerid, 0);
PlayerInfo[playerid][pAdminDuty] = 0;
}
return 1;
}

if (PlayerInfo[playerid][pAdmin] >= 1)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
Instead of if(Playerinfo[playerid][pAdmin] == 1)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
ETC.............................................
|
Just to let you know, you can also remove all these 15-20 lines which are annoying and replace them with:
pawn Код:
![]() What I did is. Код:
if (PlayerInfo[playerid][pAdmin] >= 1)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
Instead of if(Playerinfo[playerid][pAdmin] == 1)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
ETC.............................................
|
...
if(Playerinfo[playerid][pAdmin] == 1)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
if(Playerinfo[playerid][pAdmin] == 2)
{
SetPlayerColor(playerid,COLOR_CYAN);
}
....
//ETC
if(Playerinfo[playerid][pAdmin] >= 1) // DIFFERENCE LIES HERE! INSTEAD OF " Playerinfo == 1 --> Playerinfo == 2" etc, she put: playerinfo >=1
{
SetPlayerColor(playerid,COLOR_CYAN);
}
new i;
for(i=0; i< MAX_PLAYERS; i++)
{
if(IsPlayerConnected(i))
{
if( i != playerid)
{
SetPlayerMarkerForPlayer(playerid, i, YOUR_COLOR);
}
}
}
|
What she did was replace All of your un-neccessary lines, which check if the player is an admin.
You had each line check each level, which is un-needed, since anyone's admin level whom is higher than 1 is considered an admin. So she replaced: if(/*check if player is admin level 1*/) pawn Код:
pawn Код:
|