[einsfuhrer]

Hay, I'm struggling with a error on my plugin. I'm trying to set the chattag and chatcolor when you type a command and I'm making a public Action but I get the error:

"(36) : error 017: undefined symbol 'Command_hidetag'"

And my code is:

PHP Code:

public void OnPluginStart()
{
    /* Similar chunk from Kyles clientprefs tutorial. */
    g_hCookie = RegClientCookie("OurTestCookie", "A Test Cookie for use in our Tutorial", CookieAccess_Private);
    SetCookiePrefabMenu(g_hCookie, CookieMenu_OnOff_Int, "");
    
    g_cSbTime = CreateConVar("Chat-Timestamps_Scoreboard", "1", "Sets whether the scoreboard time is enabled or disabled");
    
    //RegAdminCmd("sm_cts", Cmd_ChatTimeStamps, ADMFLAG_ROOT);
    RegAdminCmd("sm_hiderole", Command_hidetag, ADMFLAG_KICK, "Desc", "Group");
    
    for (int i = 1; i <= MaxClients; i++)
    {
        if (!AreClientCookiesCached(i))
            continue;
        
        OnClientCookiesCached(i);
    }
    
    AutoExecConfig();
} 


PHP Code:

public Action Command_hiderole(int client, int args)
{
        if (IsClientInGame(iClient))
            {
                if(IsPlayerAlive(iClient)) 
                {   
                    if (CheckCommandAccess(iClient, "chattag", ADMFLAG_ROOT))
                    {
                        if (GetClientTeam(iClient) == 3)
                            PrintToChatAll(" \x0AHIDEROLE \x01| \x0B%N \x08» \x01%s", iClient, sArgs);
                                
                        if (GetClientTeam(iClient) == 2)
                            PrintToChatAll(" \x0AHIDEROLE \x01| \x09%N \x08» \x01%s", iClient, sArgs);

                        else if (GetClientTeam(iClient) == 1)
                            PrintToChatAll(" \x0AHIDEROLE \x01| \x03%N \x08» \x01%s", iClient, sArgs);
                    }
                }
                else
                {
                    if (CheckCommandAccess(iClient, "chattag", ADMFLAG_ROOT))
                {
                    if (GetClientTeam(iClient) == 3)
                        PrintToChatAll(" \x0FDEAD \x01| \x0AOWNER \x01| \x0B%N \x08» \x01%s", iClient, sArgs);
                            
                    if (GetClientTeam(iClient) == 2)
                        PrintToChatAll(" \x0FDEAD \x01|  \x0AOWNER \x01| \x09%N \x08» \x01%s", iClient, sArgs);

                    else if (GetClientTeam(iClient) == 1)
                        PrintToChatAll(" \x0FDEAD \x01|  \x0AOWNER \x01| \x03%N \x08» \x01%s", iClient, sArgs);
                }
                else
                {
                    if (GetClientTeam(iClient) == 3)
                        PrintToChatAll(" \x0FDEAD \x01|  \x08\x01\x0B%N \x08» \x01%s", iClient, sArgs);
                            
                    if (GetClientTeam(iClient) == 2)
                        PrintToChatAll(" \x0FDEAD \x01|  \x08\x01\x09%N \x08» \x01%s", iClient, sArgs);

                    else if (GetClientTeam(iClient) == 1)
                        PrintToChatAll(" \x0FDEAD \x01|  \x08\x01\x03%N \x08» \x01%s", iClient, sArgs);
                }
                }
            }
 
    return Plugin_Handled;
} 


What am I doing wrong?

[CrazyHackGUT]

You named your callback wrong.

[einsfuhrer]

Originally Posted by CrazyHackGUT You named your callback wrong.

So I changed the callback to the correct one but it still gives me the same error. Is there another problem in the code?

[Cruze]

You have given 2 else condition On IsPlayerAlive

[I am inevitable]

What you're saying right now is: if someone times sm_hiderole, then print a message in the chat, using an argument string that doesn't exist.

@Cruze
Yeah, that's not the issue.

You could make it even better. Btw, check out https://developer.mozilla.org/en-US/...ional_Operator

statement ? true : false

PrintToChatAll("%s", GetClientTeam(iClient) == 3 ? "hey" : "ho"); // in case this is false it'll just return 2

PHP Code:

public void OnClientSayCommand(int iClient, const char[] sCommand, const char[] sArgs)
{
    // Validate client.
    if (IsValidClient(iClient))
    {
        int iTeam = GetClientTeam(iClient);
        
        // If the client isn't in spec.
        if (iTeam > 1)
        {
            if (CheckCommandAccess(iClient, "", ADMFLAG_ROOT))
            {
                PrintToChatAll(" %s %s%N \x01: %s", !IsPlayerAlive(iClient) ? "\x0FDEAD" : NULL_STRING, iTeam == 3 ? "\x0B" : "\x09", sArgs);
                
                // Here's for printing messages to team only:
                for (int i = 1; i <= MaxClients; i++)
                {
                    if (IsValidClient(i) && iTeam == GetClientTeam(i))
                    {
                        PrintToChat(i, do this yourself);
                    }
                }
            }
        }
    }
}

bool IsValidClient(int iClient)
{
    return (0 < iClient <= MaxClients && IsClientInGame(iClient) && !IsFakeClient(iClient))
} 


[Govert]

Originally Posted by Cruze You have given 2 else condition On IsPlayerAlive

This can be the case. Did you fix it?

[I am inevitable]

Originally Posted by Govert This can be the case. Did you fix it?

No that's not the issue. Count the brackets, the else statement is inside the other.