I think it would be nice to add the usage of RequestFrame instead of CreateTimer with 0.1 seconds to execution as a good practice.
For example:
PHP Code:
public Action EventOnPlayerConnectedFull(Event event, char[] name, bool dontBroadcast){
int client = GetClientOfUserId(GetEventInt(event, "userid"));
RequestFrame(view_as<RequestFrameCallback>(MoveToSpectator), client);
}
public Action MoveToSpectator(any data){
ChangeClientTeam(data, CS_TEAM_NONE);
}