mage icon indicating copy to clipboard operation
mage copied to clipboard

[Delayed Blast Fireball] Fix #9180

Open the-red-lily opened this issue 2 years ago • 8 comments

  • Add tests for Delayed Blast Fireball (incl. copies)

the-red-lily avatar Jun 29 '22 20:06 the-red-lily

This fix only affects the card logic. Do we also need to look at the logic of changing zones for spell copies? https://github.com/magefree/mage/issues/9180#issuecomment-1170455227

the-red-lily avatar Jun 29 '22 22:06 the-red-lily

The bug was fixed in deb29c0659336b7df407a7696fd88a7a1360d531.

But the test can still be added, can you merge in from main to address the conflict from deb29c0659336b7df407a7696fd88a7a1360d531.

Alex-Vasile avatar Jun 30 '22 23:06 Alex-Vasile

I think I discovered another bug. If I use [[Demilich]] to cast a copy of [[Delayed Blast Fireball]], I'm pretty sure it should deal 5 damage. Currently it is only dealing 2 damage. I think this has something to do with the way the game engine handles copying logic and zone changes. (Possibly related to this: https://github.com/magefree/mage/pull/9172#discussion_r911603495)

the-red-lily avatar Jul 01 '22 04:07 the-red-lily

Demilich - (Gatherer) (Scryfall) (EDHREC)

{U}{U}{U}{U} Creature — Skeleton Wizard 4/3 This spell costs {U} less to cast for each instant and sorcery spell you've cast this turn. Whenever Demilich attacks, exile up to one target instant or sorcery card from your graveyard. Copy it. You may cast the copy. You may cast Demilich from your graveyard by exiling four instant and/or sorcery cards from your graveyard in addition to paying its other costs.

Delayed Blast Fireball - (Gatherer) (Scryfall) (EDHREC)

{1}{R}{R} Instant Delayed Blast Fireball deals 2 damage to each opponent and each creature they control. If this spell was cast from exile, it deals 5 damage to each opponent and each creature they control instead. Foretell {4}{R}{R} (During your turn, you may pay {2} and exile this card from your hand face down. Cast it on a later turn for its foretell cost.)

github-actions[bot] avatar Jul 01 '22 04:07 github-actions[bot]

// 706.12 An effect that instructs a player to cast a copy of an object (and not just copy a spell) follows the
// rules for casting spells, except that the copy is created in the same zone the object is in and then cast
// while another spell or ability is resolving. Casting a copy of an object follows steps 601.2a-h of rule 601,
// "Casting Spells," and then the copy becomes cast. Once cast, the copy is a spell on the stack,
// and just like any other spell it can resolve or be countered.

the-red-lily avatar Jul 01 '22 04:07 the-red-lily

@theelk801 Any thoughts on how to fix the interaction with demilich?

the-red-lily avatar Jul 04 '22 14:07 the-red-lily

You are correct. It should deal 5 damage.

Is it just the coping that doesn't work, or is it other non-foretell exile casts? (e.g. [[Prosper, Tome-Bound]])

Alex-Vasile avatar Jul 04 '22 14:07 Alex-Vasile

Prosper, Tome-Bound - (Gatherer) (Scryfall) (EDHREC)

{2}{B}{R} Legendary Creature — Tiefling Warlock 1/4 Deathtouch Mystic Arcanum — At the beginning of your end step, exile the top card of your library. Until the end of your next turn, you may play that card. Pact Boon — Whenever you play a card from exile, create a Treasure token.

github-actions[bot] avatar Jul 04 '22 14:07 github-actions[bot]