Skip to content
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

Zulaport Cutthroat doesn't trigger when it dies. #1426

Closed
Filar opened this issue Dec 18, 2015 · 6 comments
Closed

Zulaport Cutthroat doesn't trigger when it dies. #1426

Filar opened this issue Dec 18, 2015 · 6 comments
Assignees
Labels

Comments

@Filar
Copy link

@Filar Filar commented Dec 18, 2015

Zulaport's ability doesn't trigger when it dies. I'm not sure if that's always the case, but I've encountered that bug at least several times today.

@LevelX2 LevelX2 added the bug label Dec 19, 2015
@LevelX2 LevelX2 self-assigned this Dec 19, 2015
@LevelX2
Copy link
Contributor

@LevelX2 LevelX2 commented Dec 19, 2015

I can't reproduce that problem.

Can you describe more in detail what happens before the bug happens?
How did the Zulaport Cutthroat die exactly?

The game log would be very helpful.

@Filar
Copy link
Author

@Filar Filar commented Dec 20, 2015

20151220_011920_Two Player Duel_1.txt this an example gamelog. I've got two Cutthroats on the battlefield, sac one of them, only one of them triggers (should be two).

@LevelX2
Copy link
Contributor

@LevelX2 LevelX2 commented Jan 10, 2016

It's reproducable but hard to fix. It's on my list.

@diogovk
Copy link

@diogovk diogovk commented Jan 24, 2016

It happened to me as well when sacrificing it.

@diogovk
Copy link

@diogovk diogovk commented Jan 26, 2016

I implemented the faling test at
diogovk@041bd84

I'll try to discovery why the ability isn't being triggered in this case.

@LevelX2
Copy link
Contributor

@LevelX2 LevelX2 commented Jan 26, 2016

Tests are already included in the https://github.com/magefree/mage/blob/121f40c019dcf1b390845b9998f89bb7232a74e8/Mage.Tests/src/test/java/org/mage/test/cards/triggers/dies/BloodArtistTest.java class and set to @ignore.
I already analysed the problem and that is no simple fix, because the handling of zone-change triggers (603.6) is not implemented in all parts correctly. Probably we need an additional way to save some information about LKI of objects to handle this correctly.

To say it in anither way - it's easy to fix for exactly this kind of ability, but other abilities do after that fix no longer work correctly.

@LevelX2 LevelX2 closed this in ba6b609 Mar 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants