Page 3 of 5

Re: Theros Spoiler Season

PostPosted: 08 Sep 2013, 12:43
by friarsol
Marek14 wrote:MTGSalvation shows second card with "you can use mana as mana of any color" after Daxos of Meletis. Perhaps my table idea I wrote about some time ago could help with this sort of thing? :)
Was that Marek offering to help develop in Forge? No? Must be wishful thinking ;)

Re: Theros Spoiler Season

PostPosted: 10 Sep 2013, 12:05
by Marek14
It usually takes me a few years from getting interesting in a MTG project to developing :)

Re: Theros Spoiler Season

PostPosted: 10 Sep 2013, 23:59
by squee1968
Marek14 wrote:It usually takes me a few years from getting interesting in a MTG project to developing :)
Seeing as you joined (CCGH) 07 JUN 2008, a little over 5 years ago, I think that qualifies as a few years, so get crackin'! :D Just kidding around (or am I)? :twisted:

Re: Theros Spoiler Season

PostPosted: 11 Sep 2013, 11:23
by Marek14
squee1968 wrote:
Marek14 wrote:It usually takes me a few years from getting interesting in a MTG project to developing :)
Seeing as you joined (CCGH) 07 JUN 2008, a little over 5 years ago, I think that qualifies as a few years, so get crackin'! :D Just kidding around (or am I)? :twisted:
Yes, but I only started extensively test Forge a few months ago. I'm mainly BotArena developer.

Re: Theros Spoiler Season

PostPosted: 11 Sep 2013, 15:36
by jeffwadsworth
Marek14 wrote:
squee1968 wrote:
Marek14 wrote:It usually takes me a few years from getting interesting in a MTG project to developing :)
Seeing as you joined (CCGH) 07 JUN 2008, a little over 5 years ago, I think that qualifies as a few years, so get crackin'! :D Just kidding around (or am I)? :twisted:
Yes, but I only started extensively test Forge a few months ago. I'm mainly BotArena developer.
I knew that name seemed familiar.

Re: Theros Spoiler Season

PostPosted: 14 Sep 2013, 08:55
by swordshine
Nemesis of Mortals
4gg
Creature - Snake Uncommon
Nemesis of Mortals costs 1 less to cast for each creature card in your graveyard.
7{G}{G}: Monstrosity 5. This ability costs 1 less to activate for each creature card in your graveyard.
How to apply cost change effect to a specific ability?
The following patch would affect all monstrosity abilities on this card.
Code: Select all
### Eclipse Workspace Patch 1.0
#P trunk
Index: src/main/java/forge/card/staticability/StaticAbilityCostChange.java
===================================================================
--- src/main/java/forge/card/staticability/StaticAbilityCostChange.java   (revision 23206)
+++ src/main/java/forge/card/staticability/StaticAbilityCostChange.java   (working copy)
@@ -230,6 +230,11 @@
                 if (!sa.isSpell() || !((Spell) sa).isCastFaceDown()) {
                     return;
                 }
+            } else if (params.get("Type").equals("SelfMonstrosity")) {
+                if (!(sa instanceof AbilityActivated) || !sa.hasParam("Monstrosity")) {
+                    // include granted Monstrosity ability by other cards currently
+                    return;
+                }
             }
         }
         if (params.containsKey("ValidTarget")) {

Re: Theros Spoiler Season

PostPosted: 14 Sep 2013, 12:45
by friarsol
swordshine wrote:Nemesis of Mortals
4gg
Creature - Snake Uncommon
Nemesis of Mortals costs 1 less to cast for each creature card in your graveyard.
7{G}{G}: Monstrosity 5. This ability costs 1 less to activate for each creature card in your graveyard.
How to apply cost change effect to a specific ability?
The following patch would affect all monstrosity abilities on this card.
Code: Select all
### Eclipse Workspace Patch 1.0
#P trunk
Index: src/main/java/forge/card/staticability/StaticAbilityCostChange.java
===================================================================
--- src/main/java/forge/card/staticability/StaticAbilityCostChange.java   (revision 23206)
+++ src/main/java/forge/card/staticability/StaticAbilityCostChange.java   (working copy)
@@ -230,6 +230,11 @@
                 if (!sa.isSpell() || !((Spell) sa).isCastFaceDown()) {
                     return;
                 }
+            } else if (params.get("Type").equals("SelfMonstrosity")) {
+                if (!(sa instanceof AbilityActivated) || !sa.hasParam("Monstrosity")) {
+                    // include granted Monstrosity ability by other cards currently
+                    return;
+                }
             }
         }
         if (params.containsKey("ValidTarget")) {
Maybe just make the Cost X G G. Where X = 7 - Creatures in your graveyard (with a minimum of 0)

Re: Theros Spoiler Season

PostPosted: 14 Sep 2013, 21:24
by Marek14
friarsol wrote:
swordshine wrote:Nemesis of Mortals
4gg
Creature - Snake Uncommon
Nemesis of Mortals costs 1 less to cast for each creature card in your graveyard.
7{G}{G}: Monstrosity 5. This ability costs 1 less to activate for each creature card in your graveyard.
How to apply cost change effect to a specific ability?
The following patch would affect all monstrosity abilities on this card.
Code: Select all
### Eclipse Workspace Patch 1.0
#P trunk
Index: src/main/java/forge/card/staticability/StaticAbilityCostChange.java
===================================================================
--- src/main/java/forge/card/staticability/StaticAbilityCostChange.java   (revision 23206)
+++ src/main/java/forge/card/staticability/StaticAbilityCostChange.java   (working copy)
@@ -230,6 +230,11 @@
                 if (!sa.isSpell() || !((Spell) sa).isCastFaceDown()) {
                     return;
                 }
+            } else if (params.get("Type").equals("SelfMonstrosity")) {
+                if (!(sa instanceof AbilityActivated) || !sa.hasParam("Monstrosity")) {
+                    // include granted Monstrosity ability by other cards currently
+                    return;
+                }
             }
         }
         if (params.containsKey("ValidTarget")) {
Maybe just make the Cost X G G. Where X = 7 - Creatures in your graveyard (with a minimum of 0)
That wouldn't work correctly in some cases, like if someone used Painter's Servant and Enchanted Evening to make the card susceptible to Gloom :)

Re: Theros Spoiler Season

PostPosted: 14 Sep 2013, 21:40
by friarsol
Marek14 wrote:That wouldn't work correctly in some cases, like if someone used Painter's Servant and Enchanted Evening to make the card susceptible to Gloom :)
Yea, you probably could have just said Suppression Field. I tried looking to see if anything had a way to make activated abilities cost more, but didn't see any. Guess my search just wasn't using the right keywords.

Re: Theros Spoiler Season

PostPosted: 16 Sep 2013, 13:38
by swordshine
Abilities granted by static abilities in forge are defined as temporary abilities. If sa.isTemporary() returns true, then don't apply the cost change.
Code: Select all
### Eclipse Workspace Patch 1.0
#P trunk
Index: src/main/java/forge/card/staticability/StaticAbilityCostChange.java
===================================================================
--- src/main/java/forge/card/staticability/StaticAbilityCostChange.java   (revision 23231)
+++ src/main/java/forge/card/staticability/StaticAbilityCostChange.java   (working copy)
@@ -230,6 +230,11 @@
                 if (!sa.isSpell() || !((Spell) sa).isCastFaceDown()) {
                     return;
                 }
+            } else if (params.get("Type").equals("SelfMonstrosity")) {
+                if (!(sa instanceof AbilityActivated) || !sa.hasParam("Monstrosity") || sa.isTemporary()) {
+                    // Nemesis of Mortals
+                    return;
+                }
             }
         }
         if (params.containsKey("ValidTarget")) {

Re: Theros Spoiler Season

PostPosted: 17 Sep 2013, 18:28
by friarsol
Man, Sloth and Swordshine are absolutely tearing it up this spoiler season. When do you guys think we'll be ready for integrating the branch into the trunk?

Re: Theros Spoiler Season

PostPosted: 17 Sep 2013, 19:39
by Sloth
friarsol wrote:Man, Sloth and Swordshine are absolutely tearing it up this spoiler season.

I didn't do very much. I think Swordshine is responsible for more than 90% of the work on Theros.

friarsol wrote:When do you guys think we'll be ready for integrating the branch into the trunk?
Reintegrating the branch would help find the little scripting errors (larger problems shouldn't arise), but could delay the next beta since we would at least wait for oracle texts.

Re: Theros Spoiler Season

PostPosted: 17 Sep 2013, 20:37
by friarsol
Sloth wrote:Reintegrating the branch would help find the little scripting errors (larger problems shouldn't arise), but could delay the next beta since we would at least wait for oracle texts.
I'll see if I can dig up my original Oracle Script that scraped from magiccards.info once their database is up to date. Shouldn't take too long to whip together a new script to do it if I can't find it.

Re: Theros Spoiler Season

PostPosted: 18 Sep 2013, 15:11
by ZzzzSleep
Sloth wrote:
friarsol wrote:Man, Sloth and Swordshine are absolutely tearing it up this spoiler season.

I didn't do very much. I think Swordshine is responsible for more than 90% of the work on Theros.

friarsol wrote:When do you guys think we'll be ready for integrating the branch into the trunk?
Reintegrating the branch would help find the little scripting errors (larger problems shouldn't arise), but could delay the next beta since we would at least wait for oracle texts.
I had a look at the wiki to see if any help was needed with the coding, only to find that the entire set (bar two cards) had been done.
Nice work folks (especially Swordshine)!

Re: Theros Spoiler Season

PostPosted: 18 Sep 2013, 15:20
by swordshine
ZzzzSleep wrote:I had a look at the wiki to see if any help was needed with the coding, only to find that the entire set (bar two cards) had been done.
Nice work folks (especially Swordshine)!
Thanks, I just wanna finish the new set before the prerelease day.