-
Notifications
You must be signed in to change notification settings - Fork 745
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Creatures sacrificed don't trigger their own death triggers #12195
Comments
Industrial Advancement - (Gatherer) (Scryfall) (EDHREC)
Junji, the Midnight Sky - (Gatherer) (Scryfall) (EDHREC)
|
Vivien on the Hunt - (Gatherer) (Scryfall) (EDHREC)
|
Sacrificing with [[Evolutionary Leap]] seems to work fine, though... |
Evolutionary Leap - (Gatherer) (Scryfall) (EDHREC)
|
|
It must be a regression... there's no way this has been broken for very long. Also it's not multiple modes - cards like [[Protean Hulk]] don't work and also cards with undying. I'm planning on writing a unit test for it and doing a |
Protean Hulk - (Gatherer) (Scryfall) (EDHREC)
|
Actually this may not be a regression... I've tested on 229e8d3 and it also had the bug. Let me go back further |
I am also having this issue when sacrificing [[Su-Chi]] with [[Sage of Lat-Nam]]. I do not get the mana from Su-Chi in the pool. |
Su-Chi - (Gatherer) (Scryfall) (EDHREC)
Sage of Lat-Nam - (Gatherer) (Scryfall) (EDHREC)
|
Couldn't reproduce the bug with the test suite (added two tests in 907ac5c, each one not failing on 10k attempts) It is either:
|
@Susucre can be related to zcc too (same tests, but make blinks first); |
Can also try in unit test, add to hand and cast, rather than add to battlefield |
No replication either (4d07605). I really think there is something outside of the pair |
I speculate it is related to isInUsableZone and short living lki |
Then put on stack some spells like bolt and exec same test commands after it (without full stack resolve). |
No luck either. I'm not around to continue investigating until next week at the earliest. |
Added reproducible test by e3ae00f. Real reason was in short LKI and move to battlefield code: Nothing to do with it until move or dies logic rework. Maybe I'll look at it again later. P.S. call |
Great work finding the root cause |
I agree that the best solution is to remove the short living LKI logic and replace with standard ZCC logic. "Short Living LKI" has always sounded like a hack/workaround to me. |
[[Industrial Advancement]]. I've noticed it several times. Here's an example of a log:
I sacrificed my [[Junji, the Midnight Sky]] and its death trigger did not trigger. Here's a pic of the board state:
The text was updated successfully, but these errors were encountered: