mage icon indicating copy to clipboard operation
mage copied to clipboard

[don't merge] implement [UNF] Xenosquirrels

Open tiera3 opened this issue 1 year ago • 5 comments

tiera3 avatar Aug 27 '24 04:08 tiera3

So first of all, your replacement effect isn't implementing checksEventType().

But as far as I can tell there doesn't exist an appropriate event to replace. ROLL_DIE is fired too early for an effect that reads "after you roll a die" (side note: it's only a replacement effect because that makes the most sense, this is not properly supported by the rules). You need to see the result before making the choice of whether to apply the effect or not. The die rolling code would need careful study and adjustment, and it's entangled with planar die stuff too which is a whole mess.

xenohedron avatar Aug 27 '24 04:08 xenohedron

Ahh - found the appropriate section in SnickeringSquirrel.java to copy. I am hoping the RollDieType.NUMERICAL will stop it messing with a planar die.

When I found SnickeringSquirrel was already implemented, I thought this was going to be pretty simple.

tiera3 avatar Aug 27 '24 04:08 tiera3

I don't think it works, because the die hasn't been rolled at the time of the event. Test it in game and see if either card gives you the initial result in time to make the choice.

xenohedron avatar Aug 27 '24 05:08 xenohedron

If you roll the planar die in a Planechase game, you can remove a +1/+1 counter from Xenosquirrels, but you can’t alter the result of the roll. (2022-10-07)

xenohedron avatar Aug 27 '24 05:08 xenohedron

You are correct. Snickering Squirrel doesn't work.

image

tiera3 avatar Aug 27 '24 05:08 tiera3