[fixed]Ulamog's Nullifier no counter
Moderators: BAgate, drool66, Aswan jaguar, gmzombie, stassy, CCGHQ Admins
[fixed]Ulamog's Nullifier no counter
by gnomefry » 31 Mar 2022, 15:45
Describe the Bug:
Ulamog's Nullifier isn't coming into play with the option to transfer exiles to the GY and counter target spell.
The attached save has multiple exiled cards from sources like Spell Queller, Transgress the Mind, Anguished Unmaking, and two Oblivion Ring (which have their exiled card names visible on the board).
Fixed once 2016: viewtopic.php?f=110&t=18334&hilit=ulamog%27s+nullifier
Which update are you using? (date, name)Which type? (duel, gauntlet, sealed deck)
Holidays 2021 ad1a6f8 - gauntlet
What exactly should be the correct behavior/interaction?
Wasteland Strangler has a move-exile-to-graveyard dialog box that works well. Maybe that can be used for template? Thanks as always for your work.
Are any other cards possibly affected by this bug?
Ulamog's Nullifier isn't coming into play with the option to transfer exiles to the GY and counter target spell.
The attached save has multiple exiled cards from sources like Spell Queller, Transgress the Mind, Anguished Unmaking, and two Oblivion Ring (which have their exiled card names visible on the board).
Fixed once 2016: viewtopic.php?f=110&t=18334&hilit=ulamog%27s+nullifier
Which update are you using? (date, name)Which type? (duel, gauntlet, sealed deck)
Holidays 2021 ad1a6f8 - gauntlet
What exactly should be the correct behavior/interaction?
Wasteland Strangler has a move-exile-to-graveyard dialog box that works well. Maybe that can be used for template? Thanks as always for your work.
Are any other cards possibly affected by this bug?
- Attachments
-
- ulamogsnullifier.rar
- (4.08 KiB) Downloaded 60 times
Last edited by drool66 on 03 Apr 2022, 07:09, edited 2 times in total.
Reason: fixed
Reason: fixed
Re: Ulamog's Nullifier no counter
by Aswan jaguar » 02 Apr 2022, 11:30
I confirm this bug.
@drool66 I can only fix this by reverting it back to using comes_into_play() function, which of course will make it not consider Panharmonicon and the other cards that are checked in etb_ability(). Could you take a look?
*Not that we have migrated all cards/functions to use etb_ability(), I still count 75 using comes_into_play() maybe some of those should not be changed.
@drool66 I can only fix this by reverting it back to using comes_into_play() function, which of course will make it not consider Panharmonicon and the other cards that are checked in etb_ability(). Could you take a look?
*Not that we have migrated all cards/functions to use etb_ability(), I still count 75 using comes_into_play() maybe some of those should not be changed.
---
Trying to squash some bugs and playtesting.
Trying to squash some bugs and playtesting.
-
Aswan jaguar - Super Tester Elite
- Posts: 8092
- Joined: 13 May 2010, 12:17
- Has thanked: 732 times
- Been thanked: 465 times
Re: [confirmed]Ulamog's Nullifier no counter
by drool66 » 02 Apr 2022, 18:18
The only thing keeping etb_ability() from working are the event gates at the top of the card; so
Also, cards can't generally counter more than one spell, so I don't know if we want Ulamog's Nullifier to respond to Panharmonicon et al., except maybe to process more than once?
It's frustrating that we have two functions that did the exact same thing. I can go through and update the ~75 cards.
[EDIT] all done in c48221c
- Code: Select all
if (!IS_CASTING(player, card, event) // sic
&& trigger_condition != TRIGGER_COMES_INTO_PLAY)
return 0;
- Code: Select all
if (!IS_CASTING(player, card, event) // sic
&& !(event == EVENT_ETB_ABILITY)
&& trigger_condition != TRIGGER_COMES_INTO_PLAY)
return 0;
Also, cards can't generally counter more than one spell, so I don't know if we want Ulamog's Nullifier to respond to Panharmonicon et al., except maybe to process more than once?
It's frustrating that we have two functions that did the exact same thing. I can go through and update the ~75 cards.
[EDIT] all done in c48221c
Last edited by drool66 on 03 Apr 2022, 07:08, edited 1 time in total.
Reason: hash info
Reason: hash info
The latest images for Manalink will be here.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
-
drool66 - Programmer
- Posts: 1167
- Joined: 25 Nov 2010, 22:38
- Has thanked: 186 times
- Been thanked: 269 times
3 posts
• Page 1 of 1
Who is online
Users browsing this forum: No registered users and 6 guests