Page 1 of 1

[fixed/closed]Lion's Eye Diamond activates as mana source

PostPosted: 12 Oct 2015, 17:32
by Korath
Describe the Bug:
A. Bad: You can sac Lion's Eye Diamond for mana while paying for a spell on the stack.
B. Good: That spell doesn't actually get cast, at least.
C. Bad: You don't get the mana from the LED, though.

Which card did behave improperly ?
LED.

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

What exactly should be the correct behavior/interaction ?
It shouldn't light up as activateable during charge_mana(), nor do anything if you click on it anyway; this fixes A, and completely bypasses B and C.

Are any other cards possibly affected by this bug ?
Charmed Pendant, Grinning Ignus, Mana Bloom, Wall of Roots, and Manaforge Cinder all have restrictions on when you can activate them for mana, listed in order of decreasing similarity to Lion's Eye Diamond's. I haven't tested any of them.

(And here I was hoping I wouldn't have to figure out how to implement this myself, let alone teach the AI to sac it in response to card draw, especially draw-7s.)

Re: [confirmed]Lion's Eye Diamond activates as mana source

PostPosted: 12 Oct 2015, 21:51
by BAgate
Charmed Pendant has similar behavior.
Grinning Ignus is fine.
The others have limits on how often they can be used, not when, so should be fine.

Re: [confirmed]Lion's Eye Diamond activates as mana source

PostPosted: 11 Apr 2016, 17:29
by Gargaroz
I didn't see they changed the LED ruling, I'll update the values for him Charmed Pendant and in "ct_all.csv". The others should be fine.

Re: [confirmed]Lion's Eye Diamond activates as mana source

PostPosted: 11 Apr 2016, 17:37
by Korath
Please test it and make sure (or remind me to do so in a couple days); I experimented with flag changes before reporting this, and couldn't find a combination that worked right.

Since that time, though, I added LCBP_CHARGING_MANA to fix a different bug (the "no fast effect during combat" one about lands with non-mana abilities), and checking for that in LED's and Charmed Pendant's EVENT_CAN_ACTIVATE handlers just might do the right thing. At least for the non-AI part.

Re: [New info-Test?]Lion's Eye Diamond activates as mana sou

PostPosted: 11 Apr 2016, 19:21
by Aswan jaguar
I tested Lion's Eye Diamond and different ct_all values and I don't think you can make it not highlight while paying for mana (besides if you notice all instants in your hand are still highlighted when you pay mana for something else but can't be selected/cast) if you remove however the mana source value from Lion's Eye Diamond you can't select it to use it while paying for mana and it seems to work fine for human at least.