Some of your code is very repeticious..
ex:
in cc_messagetype, you use this like 3 times when you can simply put it above all three if() statements..
PHP Code:
new players = get_playersnum()
for (i = 0;i <= players;i++)
{
if (is_user_admin(i))
{
//Your client_prints
}
}
I would also suggest changing the if() statements in there to a switch() statement like so:
PHP Code:
new players = get_playersnum()
for (i = 0;i <= players;i++)
switch(type)
{
case 0:
{
return PLUGIN_HANDLED;
}
case 1:
{
if(is_user_admin(i))
{
client_print(i, print_chat, "%d %s: %s %s", id, name, cvar, value)
}
}
case 2:
{
client_print(id, print_chat, "%d %s: %s %s", id, name ,cvar, value)
if (is_user_admin(i))
{
client_print(i, print_chat, "%d %s: %s %s", id, name, cvar, value)
}
}
case 3:
{
client_print(i, print_chat, "%d %s: %s %s", id, name, cvar, value)
}
}