[Sanjay Singh]

I am trying to print a specific message to all admins only.
so is this possible to use like without using full colorchat stock becuz 1.9.0 already includes colorchat
client_print_color

PHP Code:

print_admin(0, 0, "Hi %s You're Admin", name)

stock print_admin(id, const msg[])
{
    new iPlayers[32], iPnum, iPlayer
    get_players(iPlayers, iPnum)

    for(new i = 0; i < iPnum; i++)
    {
        iPlayer = iPlayers[i]
        
        if(get_user_flags(iPlayer) & ADMIN_CHAT)
            client_print_color(iPlayer, 0, msg)
    } 
} 


[OciXCrom]

1. You defined the function with 2 arguments, but you're using 4.

2. Why do you need an "id" parameter in the function if it's going to send the message to a group of players?

3. You're missing "vformat".

4. You probably don't need the "stock" tag.

[Sanjay Singh]

Originally Posted by OciXCrom 1. You defined the function with 2 arguments, but you're using 4. 2. Why do you need an "id" parameter in the function if it's going to send the message to a group of players? 3. You're missing "vformat". 4. You probably don't need the "stock" tag.

like this?

PHP Code:

print_admin(const id, const input[], any:...)
{
    new count = 1, players[32];
    static msg[191];
    vformat(msg, 190, input, 3);
    
    if(id)
        players[0] = id;
    else
        get_players(players, count, "ch");
    
    for (new i = 0; i < count; i++)
    {
        x = players[i]
        if(is_user_admin(x))
        client_print_color(x, 0, msg)
    }
} 


[Shadows Adi]

You can try this:

PHP Code:

    stock print_admin(const id, const input[], any:...)
    {
        new count = 1, players[32]
        static msg[191]
        vformat(msg, 190, input, 3)
        
        replace_all(msg, 190, "!g", "^4") // Green Color
        replace_all(msg, 190, "!y", "^1") // Default Color
        replace_all(msg, 190, "!team", "^3") // Team Color
        replace_all(msg, 190, "!team2", "^0") // Team2 Color
        
        if (id) players[0] = id; else get_players(players, count, "ch")
        {
            for (new i = 0; i < count; i++)
            {
                if (is_user_connected(players[i]))
                {
                    message_begin(MSG_ONE_UNRELIABLE, get_user_msgid("SayText"), _, players[i])
                    write_byte(players[i]);
                    write_string(msg);
                    message_end();
                }
            }
        }
    } 


[OciXCrom]

@Shadows Adi - read the first post again.

PHP Code:

// You don't need an "id" parameter - you're sending the message to a GROUP of players.
print_admin(const id, const input[], any:...)
{
    // Why is count = 1?
    new count = 1, players[32];
    static msg[191];

    // Use charsmax(msg) instead of 190.
    vformat(msg, 190, input, 3);
    
    // Again not needed - only get_players() should stay.
    if(id)
        players[0] = id;
    else
        get_players(players, count, "ch");
    
    for (new i = 0; i < count; i++)
    {
        x = players[i]

        // Indentation or { brackets } after the check would be nice.
        if(is_user_admin(x))
        
        // Don't use "magic numbers". The second argument should be "print_team_default", not 0.
        client_print_color(x, 0, msg)
    }
} 


Try to think what you're doing. If you went line by line through the code, you probably could have figured it out.
You simply need to send a message to all admins, so there's no "id" involved at all - the player sending the message (if existant) is not relevant.

[Natsheh]

Well he actually needs the parameter id? Don't he?
Let's call it instead sender.

PHP Code:

print_to_admins(const id, const input[], any:...)
{
    // Why is count = 1?
    new count, players[32], szName[32];
    static msg[192];

    // Use charsmax(msg) instead of 190.
    vformat(msg, 191, input, 3);
    
    // Again not needed - only get_players() should stay.
    get_players(players, count, "ch");
    get_user_name(id, szName, 31);
    
    for (new i = 0; i < count; i++)
    {
        x = players[i]

        // Indentation or { brackets } after the check would be nice.
        if(is_user_admin(x))
        
        // Don't use "magic numbers". The second argument should be "print_team_default", not 0.
        client_print_color(x, 0, "(ADMINS) %s: %s", szName, msg)
    }
} 


Assuming thats why he needs the parameter id.

Also follow the guidance to get fully code optimization.

[OciXCrom]

He never mentioned that he wants to use the sender's name anywhere, so I'm working with the information he provided.

[Sanjay Singh]

I dont want to show sender, it should work like normal msg to all who have accsss flags.

[Natsheh]

Well your example here is wrong

"Hi %s You're Admin"

It clarify that you only need 1 receiver

[Sanjay Singh]

Originally Posted by Natsheh Well your example here is wrong "Hi %s You're Admin" It clarify that you only need 1 receiver

Actually I want it to be work like both types if possible.

like if am using like this

PHP Code:

print_color(0, "ADMIN type /help for commands")
this will print to all admins 


But what if i want to add their names to in that

PHP Code:

print_color(0, "ADMIN %s type /help for commands", adminname) 


I don't want to show the sender but I want the receiver name in the chat.