What
@Danatoth suggests is another way of doing it - the potential problem I see is that it looks like you intend to have a network of teleport nodes, and relying on a global (not a self) switch the way his events do will either activate all of your nodes at once, or make you need to use lots of switches for them individually or something.
Of course, we can't see from anything you showed how you have a list of places to go or how the player chooses or anything like that, so maybe that's not a problem.
In any case, if that's correct, I do think this is an appropriate place to use a common event, but I can't see any reason to have
three of them. Why isn't it all in one page? (and even if it was a standalone teleporter and it made sense to have it on the map event rather than in a common event, I don't see the benefit to multiple pages)
To address your original question, the ninth line of your Teleporter event asks whether the switch TP? is ON. That is
inside your first conditional branch of is TP? OFF. Since you turn the switch on inside of that branch, that means:
- That conditional will
always be true (which makes it pointless)
- It is probably the wrong place, I expect you meant it to be outside of the first conditional. As it's written, when you call this event and TP? is already ON, that event code will never be seen.
