Now that I take a closer look at it...
Should my.x, my.y and my.z beplayer.x, player.y and player.z?
I tried this and I come up with an empty pointer for each one over and over again.
I'm using the a6 walkthrough template for my player action, because I cannot write my own yet.
However, the script you are looking at is in a separate file (I included it)