Bug Reports (snapshot builds)

[Agetian]

r31279: Something is now broken with cards such as Banisher Priest and Chained to the Rocks. These cards no longer work correctly: when they leave the battlefield, the card that was exiled with them is no longer returned from the exile to the battlefield. This used to work relatively recently. Maybe something happened to the imprint mechanism that is utilized by these cards internally?
EDIT: This is broken by r31277.

- Agetian

many thanks for pointing that out, i didn't think that this specific revision should have broke it.

the check it had was "zoneType == ZoneType.Battlefield || !c.isToken()" that should have been true for that but i investigate check it out.

Sorry for breaking it i will try to fix it soon.

No problem! The thing is, effects created with DB Effect or whatever (e.g. "Banisher's Priest Effect") are internally considered to be "tokens" (c.isToken() will return True for them) - I remember there's a reason for that, too, something that has to do with the fact that the effect has to be cleaned up once it leaves play. Not sure how to fix this *and* keep your fix intact for that other card you were trying to fix... Hopefully you can come up with a solution though.

- Agetian

[Hanmac]

@Agetian: yeah i did, i fixed it in r31280.
i just add Command zone to have tokens too.
(let's hope that creature tokens are never moved to Command zone)

PS: we might need more testcases so we i will be alert when i broke something like that again.

[Marek14]

@Agetian: yeah i did, i fixed it in r31280.
i just add Command zone to have tokens too.
(let's hope that creature tokens are never moved to Command zone)

PS: we might need more testcases so we i will be alert when i broke something like that again.

Well, you could have tokens disappear on moving, but allow them to be created wherever

[Marek14]

Using this deck:

Code: Select all

[metadata]
Name=Deck #2 "Kick then Sin" by e42
[general]
Constructed
[main]
4 Tombstone Stairwell
3 Elvish Vanguard
3 Urborg Elf
3 Wirewood Channeler
4 Artificial Evolution
3 Skulltap
2 Merchant Scroll
3 Golgari Guildmage
1 Golgari Rotwurm
3 Wellwisher
4 Wirewood Herald
2 Wirewood Savage
3 Llanowar Elves
8 Swamp
2 Island
12 Forest
[sideboard]


It seems that Artificial Evolution doesn't work properly with Tombstone Stairwell. I changed "Zombie" to "Elf" so my Elvish Vanguard would get boosted by the Tombspawns, but the tokens are still Zombies.

[Hanmac]

i tried to apply the changedText to the TokenEffect,
i found applyAbilityTextChangeEffects which does look like what i want.

but i currently fail that the SpellAbility is not Intrinsic. (what ever that means in that context)
or was my plan wrong to use it like that?

Found it, i needed the RootAbility.

Edit:
@Marek14, fixed it in r31283

[Hanmac]

i add cards for "The Fate of the Flammable", "May Civilization Collapse", "Feed the Machine" and "Surrender Your Thoughts" with OppNonParentTarget.
i tested them with me as Arch-enemy and they did work.
PS: we need a "SetInMotion" Button in the Developer Tab because waiting to find the right Scheme is annoying.

Cards use same AI as the Sieges yet. i will try later to add more AI into it.
(currently he AI is a bad team player and always choose others)
later i will add Code that if AI got hit by "Discard 4 or others Discard 2" then AI will choose self if it has no cards.


also i need to find the place where i define the code that AI does use to target the opponent if AI would be the Arch-enemy. Like in the example for "Discard 4 or others Discard 2" AI does try to target the Player with the Most cards in hand.

[PopeSquad]

Just noticed that Vengeful Pharaoh doesn't seem to be working properly. While playing, my opponent's Mishra's Factory hit my Liliana of the Veil while VP was in the grave and he did not trigger.

[FiiSKiiS]

Noticed this bug today:

Kytheon, Hero of Akros triggering on opponents upkeep (his ability is "if Kytheon, Hero of Akros and at least tow other creatures attacked this combat, exile Kytheon, Hero of Akros). Kytheon can't attack on opponent's combat phase, and thus can only trigger on his controller's turn.

Since I'm fairly new, it won't let me share the screenshot I saved for it.

[Marek14]

Noticed this bug today:

Kytheon, Hero of Akros triggering on opponents upkeep (his ability is "if Kytheon, Hero of Akros and at least tow other creatures attacked this combat, exile Kytheon, Hero of Akros). Kytheon can't attack on opponent's combat phase, and thus can only trigger on his controller's turn.

Since I'm fairly new, it won't let me share the screenshot I saved for it.

Well, the ability is not actually limited by turn, but it should trigger at end of combat -- and since it has intervening if, it should only trigger if the condition is satisfied.

[Hanmac]

CombatDamageDoneOnce somehow does not work for Planeswalkers (maybe I did broke something) i currently try to write some work around with DamageDone.

Vengeful Pharaoh and others like Firemane Angel are need to be written with IsPresent check, i am currently doing that.


@Marek14: i thought that i fixed the Token and CardChanger problem, but it seems i did it to good.
for example Riftmarked Knight and cardtext changer like Glamerdye should not be able to change the color and text of the token while the ability is already on the stack. (i managed to get a red token from that)
I need to find a way to have the Ability on the Stack not that much connected to the Original Card any more

[gamerfreak12345]

Hi,

Thunderbreak Regent's ability triggers even when the opponent targets a dragon card in the graveyard. Per rule 109.4, this should only trigger on dragons that are on the battlefield.

[Marek14]

Yavimaya Granger found via Oath of Druids didn't require echo payment on my next turn.

Krosan Verge should only shuffle the library once.

I controlled opponent via Mindslaver, then I tapped his lands. I wanted to activate his Isochron Scepter, but Forge doesn't allow me to click on mana in his mana pool. Autopayment works.

AI copied my Emrakul, the Aeons Torn with Evil Twin, but sacrificed it to annihilator of my real Emrakul. It shuffled its graveyard into library when Evil Twin died. However, Emrakul's ability triggers on going to graveyard from anywhere, which means that it checks stats after the card is already in the graveyard.
(Basically, Clones and similar that copy creatures with "to graveyard from anywhere" triggers should NOT trigger them when they die.)

[Bog Wraith]

With a Xantid Swarm attacking, AI cast Second Thoughts when they should not be able to cast any spells on my turn.

[Hanmac]

i tried to fix Firemane Angel with
"IsPresent$ Card.Self | PresentZone$ Battlefield" and
"IsPresent$ Card.Self | PresentZone$ Graveyard"
but its not 100% correct.

before that fix:
if the trigger goes on the stack, and you remove it from the graveyard, you still gain life.

after the fix:
if the trigger goes on the stack, and you remove it from the graveyard, you don't gain life.
BUT if you return it again, you still gain life.

what should be:
if you return it from the graveyard, you should not gain life even if you put it back.

because of "if CARDNAME is in your graveyard"
that does Tangent other cards like Vengeful Pharaoh too.

on of the deeper coders has ideas to fix that?

[friarsol]

With a Xantid Swarm attacking, AI cast Second Thoughts when they should not be able to cast any spells on my turn.

Was it in response to the trigger?