Susucre

Results 89 comments of Susucre

Haven't investigated that one yet, but I hugely suspect #12089 to be the same issue of not processing the last payment properly on activating abilities: ![image](https://github.com/magefree/mage/assets/34709007/1710f82c-645d-4704-96bd-11d130d4f30b)

> I’ll look to it later. @JayDi85 so how it's looking?

That check in Halvar code is flawed: ![image](https://github.com/magefree/mage/assets/34709007/f0e15903-9327-4946-83f7-e101587121e0) Should call `Permanent::cantBeAttachedBy` instead. For instance on creating Role tokens, the check is this: ![image](https://github.com/magefree/mage/assets/34709007/f996b7e3-3af5-497c-8df7-a0e492894351)

I feel like the attachment actions are not setup the best way. There is currently `Permanent::attachTo`, `Card::addAttachment` and `Player::addAttachment`. Those methods should probably handle the invalid attachment (and not attach...

(The custom `ContinuousEffect` should probably be replaced by some `BecomesCreatureAllEffect`)

> Xmage uses scryfall sets structure. So if scryfall set contains that cards then it must be included in xmage too. While I do agree with that, we could have...

I don't think the current trigger-caring cards can cause issue at the moment. Need the Felix text, of "If a creature you control dealing combat damage to a player causes...

There might be, but it needs to be a combination of one of those: https://scryfall.com/search?q=o%3A%22causes+a+triggered+ability%22 And a batch triggers that filters a batch event in an disjoint way. I didn't...

I think this should be an affected case (blinking wizard non-artifact + non-wizard artifact with [[Naban, Dean]] + [[Ingenious artillerist]]): ![image](https://github.com/magefree/mage/assets/34709007/d0c0a8ce-15d1-4cac-bb99-fba95ed06127) But Naban is bugged and not looking at batchs:...

I have not looked at all of them, but the move zones "If [...] causes a triggered ability of [...] to trigger," all seem to not work on batch move...