First off, you're defining the farx,fary,farz vars on ALL /obj's, which is pointless since you only need them on all /obj/teleporter's, so define them on that instead.

Other than that, seems your only problem is in Entered(), not checking if a teleporter in the turf actually exists (check if T is valid before using it, as Lummox's example).

EDIT: I see your miss. Lummox has a slight typo/confusion in his sample code. [link] The line that checks if the teleporter is valid is actually wrong; there is no 'O' variable there, he meant

Kaioken wrote:

Yes. That approach is basically the same as having a Trigger() proc, but the latter is better especially if you happen to have lots of objects requiring that functionality.

Agreed. Although I actually took a different approach in SotS II, which requires a little more complexity because the order in which things happen matters quite a bit there.

Lummox JR