Add AfterMoveSecondaryLast event

[andrebastosdias]

Currently applies only to Ice Spinner and Pickpocket.

Fixes:

• Ice Spinner's interaction with Life Orb
• Pickpocket's interaction with Magician

Let me know if there are any other effects that should activate during this event.

Edit: does it make sense to check for EmergencyExit after the new event?

[urkerab]

I don't know how the other per-move AfterMoveSecondarySelf handlers should resolve priority compared to other AfterMoveSecondarySelf effects, but in case it helps, you can combine them in one priority system as follows:

Suggested change

	this.battle.singleEvent('AfterMoveSecondarySelf', move, null, pokemon, target, move);
	this.battle.runEvent('AfterMoveSecondarySelf', pokemon, target, move);
	this.battle.runEvent('AfterMoveSecondarySelf', pokemon, target, move, null, true);

[andrebastosdias]

That code isn't mine, so I'd prefer not to touch it. For AfterMoveSecondaryLast, I used the template from AfterMoveSecondary.

[urkerab]

Oh, I've been confusing AfterMoveSecondary, AfterMoveSecondarySelf and your new AfterMoveSecondaryLast event... well, I can't say I'm surprised...

[andrebastosdias]

I can change it to AfterMoveLast

[pyuk-bot]

I can change it to AfterMoveLast

You’re running Pickpocket in there which is supposed to be suppressed by Sheer Force. It should probably have a name that reflects that (doesn’t this event run before AfterMove as well?)

[andrebastosdias]

You’re running Pickpocket in there which is supposed to be suppressed by Sheer Force. It should probably have a name that reflects that (doesn’t this event run before AfterMove as well?)

Yeah, you're right, and yes, it does!