It is currently 22 Nov 2025, 07:46
   
Text Size

[Fixed] Bug Archive

Moderators: Aswan jaguar, gmzombie, stassy, BAgate, drool66, CCGHQ Admins

Re: [confirmed]Kismet - AI and Ravnica Shocklands

Postby Aswan jaguar » 09 Jun 2013, 06:53

I confirm that AI manipulates the Shocklands when player has Kismet in play,and can pay to untap them.It works fine as said if AI owns Kismet.
---
Trying to squash some bugs and playtesting.
User avatar
Aswan jaguar
Super Tester Elite
 
Posts: 8144
Joined: 13 May 2010, 12:17
Has thanked: 751 times
Been thanked: 486 times

[fixed]Static Orb and Paradox Haze

Postby Korath » 09 Jun 2013, 07:23

Describe the Bug:
Static Orb activates twice during my upkeep, letting me untap two permanents each time, when the AI is enchanted by Paradox Haze.

Which card did behave improperly ?
Static Orb.

Which update are you using?(date,name)Which type(Duel,Gauntlet,Sealed Deck)
Planeswalking to Modern 05182013.rar on top of CirothUngol. Solo duel/none.

What exactly should be the correct behavior/interaction ?
Static Orb should let you untap two permanents during the untap step. The approximation to untap them at the start of upkeep instead is fine, but effects that add or remove upkeeps shouldn't affect the orb. Even if they should, it's checking the wrong target for Paradox Haze.

Are any other cards possibly affected by this bug ?
Eon Hub or Gibbering Descent (if the other player has no cards in hand) will almost certainly prevent any untapping at all.

Code: Select all
diff --git a/src/cards/tempest.c b/src/cards/tempest.c
index 4a6b953..097623d 100644
--- a/src/cards/tempest.c
+++ b/src/cards/tempest.c
@@ -3755,8 +3755,6 @@ int card_static_orb(int player, int card, event_t event){
        }

        if( upkeep_trigger(player, card, event) ){
-               int count = count_upkeeps(1-player);
-               while( count > 0 && can_target(&td) ){
                                int i;
                                for( i = 0; i < 2; i++){
                                        if( can_target(&td) ){
@@ -3766,8 +3764,6 @@ int card_static_orb(int player, int card, event_t event){
                                                }
                                        }
                                }
-                               count--;
-               }
        }

        return 0;
Attachments
static_orb_and_paradox_haze.zip
(64.27 KiB) Downloaded 472 times
Last edited by Aswan jaguar on 09 Jun 2013, 13:58, edited 1 time in total.
Reason: fixed bug
User avatar
Korath
DEVELOPER
 
Posts: 3708
Joined: 02 Jun 2013, 05:57
Has thanked: 496 times
Been thanked: 1108 times

[fixed by Gargaroz]Two Static Orbs

Postby Korath » 09 Jun 2013, 07:30

Describe the Bug:
With two copies of Static Orb in play, I untap four permanents during upkeep.

Which card did behave improperly ?
Static Orb.

Which update are you using?(date,name)Which type(Duel,Gauntlet,Sealed Deck)
Planeswalking to Modern 05182013.rar on top of CirothUngol. Solo duel/none.

What exactly should be the correct behavior/interaction ?
Ruling for Static Orb: If there are two of these on the battlefield, they do not add together. The result is that only two permanents can be untapped.

Are any other cards possibly affected by this bug ?
Nope.
Attachments
two_static_orbs.zip
(2.34 KiB) Downloaded 449 times
Last edited by Aswan jaguar on 25 Jul 2013, 21:09, edited 2 times in total.
Reason: fixed bug
User avatar
Korath
DEVELOPER
 
Posts: 3708
Joined: 02 Jun 2013, 05:57
Has thanked: 496 times
Been thanked: 1108 times

Re: Harvester of Souls

Postby stassy » 09 Jun 2013, 07:53

Wow, this is a very consistant random bug, and to top of that the infamous pop up window appear also randomly with a very simple deck of 20 Harvester of Souls and 20 Swamps :shock:

Regardless of the error pop up:
- Sometimes ability doesn't trigger at all
- Sometimes ability trigger but it will stack trigger if several Harvester of Souls are in the bf and apply it in the calculation of drawn card in the window choice :shock: , the trigger that stack every time is beneficial to the player even if a Harvester of Souls trigger on AI side (meaning if player attack with Harvester of Souls and AI block with Harvester of Souls window, Harvester of Souls blocker ability trigger on AI side than choice pop and ask player to draw 2 cards, while it should not trigger for itself.
- The window error randomly pop up once per round

Image

Image

Image

Image
Attachments
1.zip
(1.9 KiB) Downloaded 387 times
stassy
Moderator
 
Posts: 5274
Joined: 25 Feb 2009, 07:06
Has thanked: 471 times
Been thanked: 337 times

Re: [fixed by Korath]Wall of Shards

Postby Korath » 09 Jun 2013, 08:51

I've now tested the others:
  • Glacial Plating: charges cumulative {S} until you can't pay or refuse to, and sacced only if you refuse the first time; also weirdly asks for target permanents after each payment, and a target creature after refusing to pay or running out of mana
  • Vexing Sphinx: cumulatively discards until you can't or refuse, and sacced only if you refuse the first time
  • AEther Rift: not enabled despite card_aether_rift() in invasion.c, so all is well
  • Dimensional Breach: not enabled despite card_dimensional_breach() and dimensional_breach_legacy() in scourge.c, so again, all is well
Fixes for Etherwrought Page and Vexing Sphinx should be as simple as a --count; within their upkeep loops.

Glacial Plating needs count decremented as well, but I'm not sure what's up with the targeting prompts. At least the target-a-permanent ones, from within charge_snow_mana(), look intentional.
User avatar
Korath
DEVELOPER
 
Posts: 3708
Joined: 02 Jun 2013, 05:57
Has thanked: 496 times
Been thanked: 1108 times

[fixed by Korath]Coffin Queen

Postby doran7 » 09 Jun 2013, 09:12

Describe the Bug:

Dont put the creature in play

Which card did behave improperly ?

Coffin Queen

Which update are you using?(date,name)Which type(Duel,Gauntlet,Sealed Deck)

Planeswalking to Modern

What exactly should be the correct behavior/interaction ?

{2}{B}, {T}: Put target creature card from a graveyard onto the battlefield under your control.

Are any other cards possibly affected by this bug ?
Attachments
coffin.rar
(1.64 KiB) Downloaded 417 times
Last edited by Aswan jaguar on 09 Jun 2013, 12:45, edited 1 time in total.
Reason: fixed bug
doran7
 
Posts: 49
Joined: 19 Oct 2012, 18:50
Has thanked: 1 time
Been thanked: 0 time

[fixed by Gargaroz]Wild Nacatl and non-beta mountains/plains

Postby Korath » 09 Jun 2013, 09:35

Describe the Bug:
Wild Nacatl isn't pumped by Temple Garden, Sacred Foundry, Madblind Mountain, or Mistveil Plains.

Which card did behave improperly ?
Wild Nacatl.

Which update are you using?(date,name)Which type(Duel,Gauntlet,Sealed Deck)
Planeswalking to Modern 05182013.rar on top of CirothUngol. Solo duel/none in this savegame (first observed in challenge gauntlet).

What exactly should be the correct behavior/interaction ?
Wild Nacatl should get +1/+1 if I control any cards with subtype mountain, and +1/+1 if I control any cards with subtype plains.

Are any other cards possibly affected by this bug ?
The other Ravnica shocklands (but not the beta duals, which work properly). I haven't looked for other cards that check for land subtypes the same way Wild Nacatl does.

I don't see anything obviously wrong in card_wild_nacatal()... at least not to cause this. It looks like it'll give +1/+1 for each mountain or plains to a maximum of +2/+2 (even if only one of the two subtypes is present, like Taiga + Taiga + Wild Nacatl); or +3/+3, if it happens to look at a Mountain or Plains first and a Plateau second. I can't reproduce either of these problems in-game, though, which makes me wonder whether it's calling the C implementation at all.
Attachments
wild_nacatl_and_nonbasics.zip
(2.65 KiB) Downloaded 365 times
Last edited by Aswan jaguar on 09 Jun 2013, 13:28, edited 2 times in total.
Reason: fixed bug
User avatar
Korath
DEVELOPER
 
Posts: 3708
Joined: 02 Jun 2013, 05:57
Has thanked: 496 times
Been thanked: 1108 times

Re: Coffin Queen

Postby Korath » 09 Jun 2013, 10:41

This fixes it:
Code: Select all
diff --git a/src/cards/tempest.c b/src/cards/tempest.c
index 097623d..3a6953f 100644
--- a/src/cards/tempest.c
+++ b/src/cards/tempest.c
@@ -581,7 +581,7 @@ int card_coffin_queen(int player, int card, event_t event){
                                        char msg[100] = "Select a creature card.";
                                        test_definition_t this_test;
                                        new_default_test_definition(&this_test, TYPE_CREATURE, msg);
-                                       int selected = new_select_target_from_grave(player, card, instance->targets[0].player, 0, AI_MAX_CMC, &this_test, 0);
+                                       int selected = new_select_target_from_grave(player, card, instance->targets[0].player, 0, AI_MAX_CMC, &this_test, 1);
                                        if( selected != -1 ){
                                                tap_card(player, card);
                                        }
User avatar
Korath
DEVELOPER
 
Posts: 3708
Joined: 02 Jun 2013, 05:57
Has thanked: 496 times
Been thanked: 1108 times

Re: Wild Nacatl and non-beta mountains/plains

Postby Aswan jaguar » 09 Jun 2013, 12:43

Kird Ape works fine.
---
Trying to squash some bugs and playtesting.
User avatar
Aswan jaguar
Super Tester Elite
 
Posts: 8144
Joined: 13 May 2010, 12:17
Has thanked: 751 times
Been thanked: 486 times

Re: [confirmed]Two Static Orbs

Postby Aswan jaguar » 09 Jun 2013, 12:53

The good news are that you can cancel the 2nd Static Orb the bad news are that you can cancel the first Static Orb,too. :lol:
---
Trying to squash some bugs and playtesting.
User avatar
Aswan jaguar
Super Tester Elite
 
Posts: 8144
Joined: 13 May 2010, 12:17
Has thanked: 751 times
Been thanked: 486 times

Re: [confirmed]Wild Nacatl and non-beta mountains/plains

Postby Gargaroz » 09 Jun 2013, 13:13

Well, for some strange reasons, Wild Nacatl has still the old ASM code, which is unable to properly recognize the new subtype system devised by Mok. Now it has a new code, consider it fixed.
----
- Current / medium term task: adjusting the code for making Misdirection and such usable
- Long term task: inserting all the good stuff I left out from the "Golden Years" mod
Gargaroz
Programmer
 
Posts: 7097
Joined: 06 Nov 2009, 11:11
Has thanked: 82 times
Been thanked: 595 times

Re: [fixed by Korath]Noetic Scales

Postby Gargaroz » 09 Jun 2013, 13:15

Fixed incorporated. Bug hunters, please does not mention "fix" in the thread title, or I could overlook it and forgot to incorporate Korath's work.
----
- Current / medium term task: adjusting the code for making Misdirection and such usable
- Long term task: inserting all the good stuff I left out from the "Golden Years" mod
Gargaroz
Programmer
 
Posts: 7097
Joined: 06 Nov 2009, 11:11
Has thanked: 82 times
Been thanked: 595 times

Re: [confirmed]Root Maze and becomes-tapped triggers

Postby Gargaroz » 09 Jun 2013, 13:24

Oh, I see. It's another strange way of doing things of Manalink.
Fixed for all the cards you mentioned.
----
- Current / medium term task: adjusting the code for making Misdirection and such usable
- Long term task: inserting all the good stuff I left out from the "Golden Years" mod
Gargaroz
Programmer
 
Posts: 7097
Joined: 06 Nov 2009, 11:11
Has thanked: 82 times
Been thanked: 595 times

Re: [fixed by Korath]Noetic Scales

Postby Aswan jaguar » 09 Jun 2013, 13:32

Ok,Gargaroz I will wait for your response first.
---
Trying to squash some bugs and playtesting.
User avatar
Aswan jaguar
Super Tester Elite
 
Posts: 8144
Joined: 13 May 2010, 12:17
Has thanked: 751 times
Been thanked: 486 times

[fixed by Gargaroz]Dimir House Guard

Postby Aswan jaguar » 09 Jun 2013, 13:38

Describe the Bug:
Dimir House Guard has no Fear.

Which card did behave improperly ?
Dimir House Guard

Which update are you using?(date,name)Which type(Duel,Gauntlet,Sealed Deck)
ML3-MAY\2013

What exactly should be the correct behavior/interaction ?


Are any other cards possibly affected by this bug ?
Attachments
dimir house guard.jpg
blocked by drift of phantasms
Last edited by Aswan jaguar on 10 Jun 2013, 03:46, edited 2 times in total.
Reason: fixed bug
---
Trying to squash some bugs and playtesting.
User avatar
Aswan jaguar
Super Tester Elite
 
Posts: 8144
Joined: 13 May 2010, 12:17
Has thanked: 751 times
Been thanked: 486 times

PreviousNext

Return to Archived Reports

Who is online

Users browsing this forum: Bing [Bot] and 22 guests

Main Menu

User Menu

Our Partners


Who is online

In total there are 23 users online :: 1 registered, 0 hidden and 22 guests (based on users active over the past 10 minutes)
Most users ever online was 9824 on 10 Nov 2025, 04:33

Users browsing this forum: Bing [Bot] and 22 guests

Login Form