Collectible Card Game Headquarters

Sloth wrote:

FabioFLX wrote:Ohhh... Maybe it's I don't speak English as my main language, too.

No prob anyway, I'm here to contribute to Forge whatever the project may need

Well, so, which undercovered and/or understaffed areas need one to work in the most?

 
A function that is long overdue for an update is getSpellCostChange. This function gets a spell/ability that should be played and returns its modified mana cost (by cards like Helm of Awakening).
Example of current script:

Code: Select all

K:CostChange:All:Less:1:Spell:All:All:NoSpecial:Spells cost 1 less to cast.

 
The plan is to convert this to be a static ability and use the modern scripting system.

First Draft:

Code: Select all

S:Mode$ CostChange | Affected$ Card | Type$ Spell | Reduction$ 1


 
If you agree to take a shot, I can set up a new class called StaticAbilityCostChange.java and hook it up. Both systems will work parallel. You can then convert the cards from the old system to the new system at your own pace.

Some new cards that can be made possible: Not of This World, Spellwild Ouphe, Elderwood Scion, Dream Chisel, Locket of Yesterdays, Semblance Anvil, Exiled Doomsayer, Kaervek's Torch.

FabioFLX wrote:Hi, writing the code I choosed not to use a new StaticAbilityCostChange class but adding a new ReduceCostBy$ parameter to the actual S:Mode$ Continuous class.
This is allowing me coding the functionality easly because it is working like setSemiPermanent... power and thoughness methods.
So a question; should I absolutely move to the StaticAbilityCostChange class or can I continue my way?

Code: Select all

S:Mode$ ReduceCost | ValidCard$ Card | Type$ Spell | Amount$ 1

Code: Select all

S:Mode$ RaiseCost | ValidCard$ Artifact,Enchantment | Type$ Spell | Activator$ Opponent | Amount$ 2

Sloth wrote:I thought about this and want to suggest a little change:

Instead of having one hook for CostChange it's better to have two static abilities RaiseCost and ReduceCost. This way they will be ordered in the right way.

I hope this won't mess with what you did already (you might have to cut'n paste some stuff).

Example script:
Helm of Awakening

Code: Select all

S:Mode$ ReduceCost | ValidCard$ Card | Type$ Spell | Amount$ 1

Aura of Silence

Code: Select all

S:Mode$ RaiseCost | ValidCard$ Artifact,Enchantment | Type$ Spell | Activator$ Opponent | Amount$ 2

FabioFLX wrote:I was using past method, the one with one shared ability name and two different parameters.
This is allowing me to work as a mana transofmation tool because it semplifies cards that changes mana type in the cost.

FabioFLX wrote:Altough it is not mandatory, can I follow my previous path?