AbilityFactory TODO list

[jeffwadsworth]

[quote="Sloth"]I've added AbilityFactory_PreventDamage.java. A wiki entry will follow.

Wow, that adds quite a bit of cards to the todo list. Excellent.

[SoulStorm]

You rock Sloth! How far are we away from Vigor now?

[Sloth]

You rock Sloth! How far are we away from Vigor now?

Vigor is kind of different, since it is a static ability. The abilities are not so exciting (I thing no card with it saw any constructed play ever). Samite Healer and friends at least were not so terrible in limited.

[friarsol]

I almost won a Sealed tournament with Field Surgeon and Festering Wound back in the Urza's Destiny PreRelease

[Sloth]

I almost won a Sealed tournament with Field Surgeon and Festering Wound back in the Urza's Destiny PreRelease

Drafting Urza block must have been quite nice.

Another topic:
How do we represent that the next damage point to a player get prevented?

[friarsol]

Another topic:
How do we represent that the next damage point to a player get prevented?

You mean in the UI right? That is a valid consideration.

I'd say the Ideal way would be if (preventNextDamage > 0) show a shield in the top corner of the LifeTotal with the number of Damage that will be prevented.

Realistically? Hmmm...

[friarsol]

I feel like we should pull out Protection from Pump and have it be it's own AF. This way we can have AF$Protection either gain specific things, or have it be a choice that happens on resolution (like Mother of Runes).

This would primarily be to improve the Card Detail description of these cards, but also since Protection's AI should be more similar to Regeneration, where it's reactionary. Protection could also be used as a form of Pseudo-Evasion, and instead of bloating Pump with that, we could just pull all of that out into it's own AF.

Here's some relevant examples:

For Questing Phelddagrif
AB$Protection | Types$ red,black

For Mother of Runes
AB$Protection | Choice$ white,blue,green,red,black

For Pale Wayfarer
AB$Protection | Choice$ white,blue,green,red,black | Chooser$ Controller

For Apostle's Blessing
AB$Protection | Choice$ artifacts,white,blue,green,red,black

I think there's only like 15 Cards with ProtectionAll and we only have a few of them already scripted.

[friarsol]

I keep thinking I can convert Hermit Druid or similar cards to Dig. It's fairly similar, but might be different enough to warrant it's own cousin AF. Mind Funeral would also fit into this category.

Now that I've thought about it for a few minutes I think a AF_RevealUntil would definitely be better as a separate AF. It seems like that is used for many different circumstances.

[slapshot5]

Something else to consider before going crazy adding new mechanics:

Charms and Commands

A Charm AF would wrap around three sub abilities, checking each AI, then chooses one from the positive returns, albeit randomly....

Hi Rob, how do you see this working? Something like:

Code: Select all

A:SP$Charm | Cost$ 1 U | SA1$ Untap | SA2$ Damage | SA3$ GainLife | SpellDescription$ Choose one - Untap target permanent; CARDNAME deals 2 damage to target player; or you gain 3 life.
SVar:Untap:DB$Untap | ValidTgts$ Permanent
SVar:Damage:DB$DealDamage | NumDmg$ 2 | Tgt$ TgtP
SVar:GainLife:DB$GainLife | LifeAmount$ 3 | Defined$ You


I'm interested in getting this implemented.

-slapshot5

Edit: And a "ChooseNum$ 2" type param for the Commands?

[friarsol]

Something else to consider before going crazy adding new mechanics:

Charms and Commands

A Charm AF would wrap around three sub abilities, checking each AI, then chooses one from the positive returns, albeit randomly....

Hi Rob, how do you see this working? Something like:

Code: Select all

A:SP$Charm | Cost$ 1 U | SA1$ Untap | SA2$ Damage | SA3$ GainLife | SpellDescription$ Choose one - Untap target permanent; CARDNAME deals 2 damage to target player; or you gain 3 life.
SVar:Untap:DB$Untap | ValidTgts$ Permanent
SVar:Damage:DB$DealDamage | NumDmg$ 2 | Tgt$ TgtP
SVar:GainLife:DB$GainLife | LifeAmount$ 3 | Defined$ You


I'm interested in getting this implemented.

-slapshot5

Edit: And a "ChooseNum$ 2" type param for the Commands?

I would think more like this, since this allows for better scaling.

Code: Select all

A:SP$Charm | Cost$ 1 U | Choices$ Untap,Damage,GainLife | ChooseNum$ 1 | SpellDescription$ Choose one - Untap target permanent;


(or whatever delimiter you want to use)

If we ever switch over to a "ShouldPlayAI()" that rates SAs, then we could choose from the list the highest rated one(s).

[Sloth]

Hi Rob, how do you see this working? Something like:

Code: Select all

A:SP$Charm | Cost$ 1 U | SA1$ Untap | SA2$ Damage | SA3$ GainLife | SpellDescription$ Choose one - Untap target permanent; CARDNAME deals 2 damage to target player; or you gain 3 life.
SVar:Untap:DB$Untap | ValidTgts$ Permanent
SVar:Damage:DB$DealDamage | NumDmg$ 2 | Tgt$ TgtP
SVar:GainLife:DB$GainLife | LifeAmount$ 3 | Defined$ You


I'm interested in getting this implemented.

While you are working on it you can take a look at the Clash AF. It's very similar I guess. It has problems with targeted DB's though (Springjack Knight and Pulling Teeth are not working at the moment).

[Rob Cashwalker]

I would go along with Sol's version.

[slapshot5]

Indeed, Sol's version is better.

Does anyone know where the choosing would happen? It happens before targets are selected obviously, but I don't know the targeting system well enough to know what happens between select the card in hand and choosing targets.

Any insight would be helpful.

-slapshot5

[friarsol]

Indeed, Sol's version is better.

Does anyone know where the choosing would happen? It happens before targets are selected obviously, but I don't know the targeting system well enough to know what happens between select the card in hand and choosing targets.

Any insight would be helpful.

Hmmm.. SpellAbility_Requirements is what you are looking for. You can see the creation and call to that in playSpellAbility()

[slapshot5]

Added an AF for RegenerateAll (converted Loxodon Hierarch and Wrap in Vigor ; add Wail of the Nim).

-slapshot5