Guardian of the Ages trigger needs a rework

magefree / mage

Magic Another Game Engine

http://xmage.today

MIT License

1.91k stars 779 forks source link

Guardian of the Ages trigger needs a rework #12145

Open Susucre opened 7 months ago

Susucre commented 7 months ago

From reading its code, haven't tested it live, [[Guardian of the Ages]] seems bugged on multiple aspect:

Should be an "intervening if" trigger, but is not
Doesn't check what or who is being attacked

Should probably made new, and from common effects like AttacksAllTriggeredAbility or ConditionalInterveningIfTriggeredAbility

github-actions[bot] commented 7 months ago

Guardian of the Ages - (Gatherer) (Scryfall) (EDHREC)

{7} Artifact Creature — Golem 7/7 Defender (This creature can't attack.) When a creature attacks you or a planeswalker you control, if Guardian of the Ages has defender, it loses defender and gains trample.