magefree / mage

Magic Another Game Engine
http://xmage.today
MIT License
1.83k stars 751 forks source link

Cost reduction does not work properly #79

Closed LevelX2 closed 11 years ago

LevelX2 commented 11 years ago

Cost reduction like e.g. Ruby Medallion 1) does not work with AlternativeCostImpl like used in Lavaball Trap. 2) does not work with Retrace ability. 3) does not work with Flashback ability.

magenoxx commented 11 years ago

the implementation of flashback and retrace is akward but I had to take it into account while update the cost modification

now seems work at least I tested various cases with Medalion, Thalia and spells at hand and in graveyard (flashback, trace) as well as traps (alt. cost). All scenarios work.

battlefield:Human:Mountain:4 battlefield:Human:Plains:2 battlefield:Human:Llanowar Elves:1 battlefield:Human:Thalia, Guardian of Thraben:1 hand:Human:Llanowar Elves:1 graveyard:Human:Unburial Rites:2 graveyard:Human:Flame Jab:1 hand:Human:Lightning Bolt:2

hand:Computer:Umezawa's Jitte:3 battlefield:Computer:Swamp:5

battlefield:Human:Jet Medallion:2 battlefield:Human:Ruby Medallion:2

hand:Human:Lavaball Trap:1 hand:Human:Baloth Cage Trap:1

Fixed. Closing.