[Exolent[jNr]]

1.

            if(arg[1] == 'T')
            {
                copy(tname,15,"TERRORIST");
                copy(teamname,23,"Terrorist");
            }
            if(arg[1] == 'C' && arg[2] == 'T')
            {
                copy(tname,15,"CT");
                copy(teamname,23,"Counter-Terrorist");
            }
            get_players(players,num,"be",tname);

Should be else if(), and should block command if neither team is specified:

            if(arg[1] == 'T')
            {
                copy(tname,15,"TERRORIST");
                copy(teamname,23,"Terrorist");
            }
            else if(arg[1] == 'C' && arg[2] == 'T')
            {
                copy(tname,15,"CT");
                copy(teamname,23,"Counter-Terrorist");
            }
            else
            {
                console_print(); // warning message
                return PLUGIN_HANDLED;
            }
            get_players(players,num,"be",tname);

-----------------------------
2.

new namet[32]; get_user_name(target,namet,31);

Two statements on one line?
-----------------------------
3.

        if(get_user_team(index) == 1)
        {
            fm_give_item(index,"weapon_knife");
            fm_give_item(index,"weapon_glock18");
            fm_give_item(index,"ammo_9mm");
            fm_give_item(index,"ammo_9mm");
            fm_give_item(index,"ammo_9mm");
            fm_give_item(index,"ammo_9mm");
        } else if(get_user_team(index) == 2) {
            fm_give_item(index,"weapon_knife");
            fm_give_item(index,"weapon_usp");
            fm_give_item(index,"ammo_45acp");
            fm_give_item(index,"ammo_45acp");
            fm_give_item(index,"ammo_45acp");
            fm_give_item(index,"ammo_45acp");
        }

Use a switch:

        switch( get_user_team(index) )
        {
            case 1:
            {
                fm_give_item(index,"weapon_knife");
                fm_give_item(index,"weapon_glock18");
                fm_give_item(index,"ammo_9mm");
                fm_give_item(index,"ammo_9mm");
                fm_give_item(index,"ammo_9mm");
                fm_give_item(index,"ammo_9mm");
            }
            case 2:
            {
                fm_give_item(index,"weapon_knife");
                fm_give_item(index,"weapon_usp");
                fm_give_item(index,"ammo_45acp");
                fm_give_item(index,"ammo_45acp");
                fm_give_item(index,"ammo_45acp");
                fm_give_item(index,"ammo_45acp");
            }
        }

-----------------------------
EDIT:
4.

message_begin(MSG_ONE,get_user_msgid("ScreenFade"),{0,0,0},index);

Use a global messageid for ScreenFade.

new g_msgid_ScreenFade;

public plugin_init()
{
    g_msgid_ScreenFade = get_user_msgid("ScreenFade");
}

// ...

message_begin(MSG_ONE,g_msgid_ScreenFade,{0,0,0},index);

[anakin_cstrike]

Done
@ 4: why?

[Alka]

Instead of geting the messageid x times, you store it once.That's why.

[Exolent[jNr]]

Because, like Alka said, everytime someone is revived, that ScreenFade is shown to the player. Your way, the plugin has to retrieve get_user_msgid("ScreenFade") every time someone is respawned. My (and most other coder's) way will only use get_user_msgid("ScreenFade") once and use it. the get_user_msgid() values do not change.

[anakin_cstrike]

I know what...i was thinking at something else.

[Alucard^]

sry for post this here, but no need new thread to this.. i need only the screen fade when u respawn.. u can give me the code? i tried to take ur code but, i cant..

thx and gj

[anakin_cstrike]

Look for 'Fade(index,...)'

[yoonghoe]

how to make it work?

[anakin_cstrike]

Originally Posted by yoonghoe how to make it work?

What ?! it works.

[QuixxyQuixx]

Sweet Job.

You probably already know this but.
You can revive/spawn players who are in spectator too xD.
Thanks for this too, i can use it to spawn rule breakers with 0 hp 0 armor as a kind of punishment besides slay or drug etc.

Once again, Nice Job and Thanks.