It is currently 07 Jul 2021, 21:28
   
Text Size

[fixed/closed]Qasali Ambusher

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

[fixed/closed]Qasali Ambusher

Postby HarlequinCasts » 11 Nov 2013, 09:36

Describe the Bug:
Qasali Ambusher seems to be uncastable during main phase although it lights up yellow as castable.

Previous bug I reported delt with the Qasali Ambusher showing up as castable via flash even when it was not. This still appears to be a problem for example during the end step of an opponents turn it lights up.

Previous Bug

Which card did behave improperly ?
Qasali Ambusher

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

What exactly should be the correct behavior/interaction ?
Flash alternate cost, otherwise normal casting cost.

Are any other cards possibly affected by this bug ?
NA
Attachments
qasali ambusher2.zip
(3.65 KiB) Downloaded 75 times
Last edited by BAgate on 31 Jan 2014, 03:37, edited 2 times in total.
Reason: closed
User avatar
HarlequinCasts
 
Posts: 917
Joined: 07 May 2013, 14:33
Has thanked: 68 times
Been thanked: 30 times

Re: Qasali Ambusher

Postby Korath » 11 Nov 2013, 10:08

OoAv2 was deployed 28 Sep 2013.
User avatar
Korath
DEVELOPER
 
Posts: 3522
Joined: 02 Jun 2013, 05:57
Has thanked: 491 times
Been thanked: 1037 times

Re: Qasali Ambusher

Postby HarlequinCasts » 11 Nov 2013, 11:06

>.< My bad
User avatar
HarlequinCasts
 
Posts: 917
Joined: 07 May 2013, 14:33
Has thanked: 68 times
Been thanked: 30 times

Re: [already reported & fixed]Qasali Ambusher

Postby Aswan jaguar » 11 Nov 2013, 15:46

Korath was the Cip - flash creatures that target- disenchant issue fixed also?
Korath said:
The bit about disenchanting at cip is clearly out of place, and the ai modifier just delays casting (sometimes, and not very well); the only important bit seems to be the EVENT_CAN_CAST handler. I'll experiment.
---
Turns out the yellow "you can cast this" text and whether you can actually cast the card don't use the same logic for cards that are both (instants/interrupts/sorceries/enchantments) and (artifacts/creatures) - the yellow text doesn't check EVENT_CAN_CAST for these, while actually clicking on them to cast them does, which is why not calling flash() didn't suppress the text color. Fixed d2b2dc6.
---
Trying to squash some bugs and playtesting.
User avatar
Aswan jaguar
Super Tester Elite
 
Posts: 7450
Joined: 13 May 2010, 12:17
Has thanked: 639 times
Been thanked: 351 times

Re: [already reported & fixed]Qasali Ambusher

Postby Korath » 11 Nov 2013, 20:28

I copied flash() into C, mostly for clarity, and removed that part. The only things calling the asm version now are Ashcoat Bear, Benalish Knight, Havenwood Wurm, Nephalia Seakite, Skyline Predator, Spire Monitor (all directly) and Bogardan Hellkite (indirectly). I can't see how any of those can get a target while being cast - the hellkite's cip trigger is implemented as a trigger, not as part of casting like most cip triggers are in C - so there's nothing disenchanting that shouldn't anymore. (A good test case for the bug is Indomitable Will and either artifact creatures or Enchanted Evening.)

There's some small chance that a card in C actually relied on this behavior, in which case I broke it. I'll try to remember to look.
User avatar
Korath
DEVELOPER
 
Posts: 3522
Joined: 02 Jun 2013, 05:57
Has thanked: 491 times
Been thanked: 1037 times


Return to Archived Reports

Who is online

Users browsing this forum: No registered users and 5 guests


Who is online

In total there are 5 users online :: 0 registered, 0 hidden and 5 guests (based on users active over the past 10 minutes)
Most users ever online was 1922 on 07 Jun 2021, 06:01

Users browsing this forum: No registered users and 5 guests

Login Form