Page 1 of 1

AbililtyFactory_Mana

PostPosted: 15 Jan 2011, 00:20
by friarsol
Alright now that it's the beginning of a new cycle, I'm going to submit my changes for AF_Mana. As before, if you see anything major or minor please let me know in this thread. I've been playing around with this for a fair amount and it's pretty stable. So I mostly just need more eyes to see if anything terrible has broken, or if some strange interaction isn't handled.

Since we can now use Cost with Mana things like Elvish Spirit Guide can be right behind. I've added Squandered Resources and Star Compass as types of the Reflected Mana portion, along with all the other Reflected Mana that's been converted.

The revision is r5155. For those with slower computers/connections, you may be better purging your cardsfolder and resyncing everything than dealing with the 400+ changes that happened in that folder.

Re: AbililtyFactory_Mana

PostPosted: 15 Jan 2011, 00:50
by slapshot5
Excellent. I've been looking forward to this since you said you were working on it. Updating now. I'll take a look at various things.

Did you go through the CardFactories and convert, or are there things in there that need to be converted yet?

-slapshot5

Re: AbililtyFactory_Mana

PostPosted: 15 Jan 2011, 00:59
by friarsol
slapshot5 wrote:Excellent. I've been looking forward to this since you said you were working on it. Updating now. I'll take a look at various things.

Did you go through the CardFactories and convert, or are there things in there that need to be converted yet?

-slapshot5
I think I got everything in the CardFactories, since it either was using Ability_Mana or ManaPool and those were easy enough to search for.

Re: AbililtyFactory_Mana

PostPosted: 15 Jan 2011, 01:00
by Chris H.
I was able to sync without purging my cardsfolder, the sync went fairly quickly. I guess that there are several old style mana keywords to retire.

Re: AbililtyFactory_Mana

PostPosted: 15 Jan 2011, 02:43
by jeffwadsworth
Same here. This and the trigger code should make for some sweet card creation. =D>

Re: AbililtyFactory_Mana

PostPosted: 16 Jan 2011, 03:54
by slapshot5
It looks like drawback for Mana isn't implemented. (Thinking cards like Deconstruct ).

Is this just because it isn't done yet, or is there a technical reason?

-slapshot5

Re: AbililtyFactory_Mana

PostPosted: 16 Jan 2011, 04:10
by friarsol
slapshot5 wrote:It looks like drawback for Mana isn't implemented. (Thinking cards like Deconstruct ).

Is this just because it isn't done yet, or is there a technical reason?

-slapshot5
Nope I didn't implement it. Mostly because I couldn't think of anything that used it. I even have the comment to back up my thinking "// Mana never really appears as a Drawback"

I figured mana abilities where mana abilities first and something else as a Sub Ability. I'll take a crack at it in a bit. Shouldn't be too hard to get done.

Edit: Drawback and Deconstruct are in. I also fixed a bug in ExileFromHand<1/CARDNAME> to get the Spirit Guides to work.