[Goldkiller]

You better do if(GetPlayerName(...)) { } because it has an inbuild IsPlayerConnected() check. You never check if the player is connected,which will make the code useless at all, because it will always return 0.
Besides I would recommend using INVALID_PLAYER_ID instead of -1.