It is currently 25 Apr 2024, 20:21
   
Text Size

Theros Spoiler Season

Post MTG Forge Related Programming Questions Here

Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins

Re: Theros Spoiler Season

Postby friarsol » 08 Sep 2013, 12:43

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 ;)
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Theros Spoiler Season

Postby Marek14 » 10 Sep 2013, 12:05

It usually takes me a few years from getting interesting in a MTG project to developing :)
Marek14
Tester
 
Posts: 2761
Joined: 07 Jun 2008, 07:54
Has thanked: 0 time
Been thanked: 297 times

Re: Theros Spoiler Season

Postby squee1968 » 10 Sep 2013, 23:59

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:
squee1968
 
Posts: 254
Joined: 18 Nov 2011, 03:28
Has thanked: 110 times
Been thanked: 45 times

Re: Theros Spoiler Season

Postby Marek14 » 11 Sep 2013, 11:23

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.
Marek14
Tester
 
Posts: 2761
Joined: 07 Jun 2008, 07:54
Has thanked: 0 time
Been thanked: 297 times

Re: Theros Spoiler Season

Postby jeffwadsworth » 11 Sep 2013, 15:36

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.
jeffwadsworth
Super Tester Elite
 
Posts: 1171
Joined: 20 Oct 2010, 04:47
Location: USA
Has thanked: 287 times
Been thanked: 69 times

Re: Theros Spoiler Season

Postby swordshine » 14 Sep 2013, 08:55

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")) {
swordshine
 
Posts: 682
Joined: 11 Jul 2010, 02:37
Has thanked: 116 times
Been thanked: 87 times

Re: Theros Spoiler Season

Postby friarsol » 14 Sep 2013, 12:45

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)
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Theros Spoiler Season

Postby Marek14 » 14 Sep 2013, 21:24

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 :)
Marek14
Tester
 
Posts: 2761
Joined: 07 Jun 2008, 07:54
Has thanked: 0 time
Been thanked: 297 times

Re: Theros Spoiler Season

Postby friarsol » 14 Sep 2013, 21:40

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.
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Theros Spoiler Season

Postby swordshine » 16 Sep 2013, 13:38

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")) {
swordshine
 
Posts: 682
Joined: 11 Jul 2010, 02:37
Has thanked: 116 times
Been thanked: 87 times

Re: Theros Spoiler Season

Postby friarsol » 17 Sep 2013, 18:28

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?
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Theros Spoiler Season

Postby Sloth » 17 Sep 2013, 19:39

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.
User avatar
Sloth
Programmer
 
Posts: 3498
Joined: 23 Jun 2009, 19:40
Has thanked: 125 times
Been thanked: 507 times

Re: Theros Spoiler Season

Postby friarsol » 17 Sep 2013, 20:37

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.
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Theros Spoiler Season

Postby ZzzzSleep » 18 Sep 2013, 15:11

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)!
ZzzzSleep
 
Posts: 182
Joined: 29 Oct 2010, 02:19
Has thanked: 18 times
Been thanked: 18 times

Re: Theros Spoiler Season

Postby swordshine » 18 Sep 2013, 15:20

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.
swordshine
 
Posts: 682
Joined: 11 Jul 2010, 02:37
Has thanked: 116 times
Been thanked: 87 times

PreviousNext

Return to Developer's Corner

Who is online

Users browsing this forum: No registered users and 70 guests


Who is online

In total there are 70 users online :: 0 registered, 0 hidden and 70 guests (based on users active over the past 10 minutes)
Most users ever online was 4143 on 23 Jan 2024, 08:21

Users browsing this forum: No registered users and 70 guests

Login Form