[Donny_k]

Quote:

public OnPlayerRequestSpawn(playerid)

If you are requesting to be spawned then you have no position, move the code to "OnPlayerSpawn".

Also I'd just like to point out that you are going to be reading a file each time you request a spawn, this is very stupid (the method not you dude) and if you are going to use files then use buffers like I do in THIS script. I have a buffer in there which holds all the players data which is collected once from the file and then the buffer data is used instead of opening/reading/closing the file each time I want some data.