[Jay_Dixon]

Ok, so basically, i need this to show ALL admins that are online, not just one of em. I'm completely lost as to why it won't.

Code:

dcmd_admins(playerid,params[])
{
	#pragma unused params
	//new count = 0;
	new string[128];
	for(new i = 0; i <MAX_PLAYERS; i++)
	{
	    if(IsPlayerConnected(i))
	    {
	        if(AccInfo[i][Level] >= 1 && AccInfo[i][Hide] == 0)
	        {
	            if(AccInfo[i][Level] > 5)
	            {
	                AdmRank = "Server Owner";
				}
				if(IsPlayerAdmin(i))
				{
				    AdmRank = "RCON Administrator";
				}
				else
				{
				    switch(AccInfo[i][Level])
				    {
				        case 1: {
				        AdmRank = "Trial Moderator";
						}
						case 2: {
						AdmRank = "Moderator";
						}
						case 3: {
						AdmRank = "Administrator";
						}
						case 4: {
						AdmRank = "Senior Administrator";
						}
						case 5: {
						AdmRank = "Community Leader";
						}
					}
				}
				format(string, sizeof(string), "%s (Id:%i) | %s",PlayerName2(i),i,AdmRank);
				ShowPlayerDialog(playerid, 668, DIALOG_STYLE_MSGBOX, "Online Administrators",string,"Submit","Cancel");
				return 1;
			}
		}
	}
	return 1;
}

[iggy1]

Remove return 1, from the inside of the loop.

EDIT: Use [pawn] tags not [code], there may be more wrong with the code but i CBA looking through code with poo indentation.

[Income]

PHP Code:

dcmd_admins(playerid,params[])
{
    #pragma unused params
    //new count = 0;
    new string[128];
    new PlayerName2[MAX_PLAYER_NAME];
    for(new i = 0; i <MAX_PLAYERS; i++)
    {
        if(IsPlayerConnected(i))
        {
            if(AccInfo[i][Level] >= 1 && AccInfo[i][Hide] == 0)
            {
                if(AccInfo[i][Level] > 5)
                {
                    AdmRank = "Server Owner";
                }
                if(IsPlayerAdmin(i))
                {
                    AdmRank = "RCON Administrator";
                }
                else
                {
                    switch(AccInfo[i][Level])
                    {
                        case 1: AdmRank = "Trial Moderator";
                        case 2: AdmRank = "Moderator";
                        case 3: AdmRank = "Administrator";
                        case 4: AdmRank = "Senior Administrator";
                        case 5: AdmRank = "Community Leader";
                    }
                }
                GetPlayerName(i, PlayerName2, MAX_PLAYER_NAME);
                format(string, sizeof(string), "%s (Id:%i) | %s", PlayerName2, i, AdmRank);
                ShowPlayerDialog(playerid, 668, DIALOG_STYLE_MSGBOX, "Online Administrators",string,"Submit","Cancel");
            }
        }
    }
    return 1;
} 


There was a problem with the "PlayerName2(i)" lines, since you've never defined that variable perhaps? Unless that's a function of course.

[Jay_Dixon]

Quote:

Originally Posted by Income PHP Code: dcmd_admins(playerid,params[]) {     #pragma unused params     //new count = 0;     new string[128];     new PlayerName2[MAX_PLAYER_NAME];     for(new i = 0; i <MAX_PLAYERS; i++)     {         if(IsPlayerConnected(i))         {             if(AccInfo[i][Level] >= 1 && AccInfo[i][Hide] == 0)             {                 if(AccInfo[i][Level] > 5)                 {                     AdmRank = "Server Owner";                 }                 if(IsPlayerAdmin(i))                 {                     AdmRank = "RCON Administrator";                 }                 else                 {                     switch(AccInfo[i][Level])                     {                         case 1: AdmRank = "Trial Moderator";                         case 2: AdmRank = "Moderator";                         case 3: AdmRank = "Administrator";                         case 4: AdmRank = "Senior Administrator";                         case 5: AdmRank = "Community Leader";                     }                 }                 GetPlayerName(i, PlayerName2, MAX_PLAYER_NAME);                 format(string, sizeof(string), "%s (Id:%i) | %s", PlayerName2, i, AdmRank);                 ShowPlayerDialog(playerid, 668, DIALOG_STYLE_MSGBOX, "Online Administrators",string,"Submit","Cancel");             }         }     }     return 1; }  There was a problem with the "PlayerName2(i)" lines, since you've never defined that variable perhaps? Unless that's a function of course.

Nah, PlayerName2 seems to be a function on this thing, i'm using the LuxAdmin script at the moment for this. But either way i still can't get /admins to work, and don't know what i'm doing wrong here