Some suggestions.
Add a shorter command option like this:
function serverCmdTele(%client)
{
serverCmdTeleport(%client,%p1,%p2)
}
Make the wording clearer like this:
else if(!isObject(%victim.player))
messageclient(%client,'',"<color:FF0000>Teleporting player not found.");
else if(!isObject(%target.player))
messageclient(%client,'',"<color:FF0000>Destination player not found.");
and
else if(%p2 $= "")
messageclient(%client,'',"<color:FFFFFF>Usage: <color:FF0000>/teleport or /tele [Teleporting Player] [Desination Player]");