SVN Bug Reports
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins
Re: SVN Bug Reports
by Sloth » 04 Jun 2011, 15:54
The AI won't use the ability for now. Thanks Agetian.Agetian wrote:SVN r9251: the AI casts Shambling Shell and sacrifices it to try and put a counter on the sacrificed Shambling Shell instead of on other creatures that it has.
-
Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: SVN Bug Reports
by lazylockie » 04 Jun 2011, 17:15
r9243
Sower of Temptation dies and I still have control of AI's creature.
Dreamspoiler Witches - after I cast a creature with Flash (Scion of Oona), the creature doesn't appear on battlefield (and doesn't go to graveyard either). Also, might be related, the same creature with flash couldn't be cast at all (doesn't prompt mana cost payment) on the subsequent games.
Allies (Kazandu Blademaster and others) are triggering when themselves enter the battlefield twice
Sower of Temptation dies and I still have control of AI's creature.
Dreamspoiler Witches - after I cast a creature with Flash (Scion of Oona), the creature doesn't appear on battlefield (and doesn't go to graveyard either). Also, might be related, the same creature with flash couldn't be cast at all (doesn't prompt mana cost payment) on the subsequent games.
Allies (Kazandu Blademaster and others) are triggering when themselves enter the battlefield twice
Last edited by lazylockie on 04 Jun 2011, 19:54, edited 1 time in total.
- lazylockie
- Posts: 508
- Joined: 13 Jul 2010, 22:44
- Has thanked: 74 times
- Been thanked: 15 times
Re: SVN Bug Reports
by Sloth » 04 Jun 2011, 17:27
I just tried Goblin Grenade and it worked fine. What did you sacrifice Biribau? (and maybe can you post the error report next time)
-
Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: SVN Bug Reports
by friarsol » 05 Jun 2011, 02:03
It looks like enPump isn't displaying in the card detail Panel, here's a few I noticed.
Cursed Flesh
Sea's Claim (duplicates)
Viscerid Armor
Shackles
Cursed Flesh
Sea's Claim (duplicates)
Viscerid Armor
Shackles
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: SVN Bug Reports
by friarsol » 05 Jun 2011, 02:20
Has anyone noticed the AI properly paying mana for CounterMagic with UnlessCosts? I just Condescend -ed his spell for 3, it resolved (allowing me to Scry) but then the Spell still resolved. The AI only had 2 mana available, and neither land is now tapped.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: SVN Bug Reports
by slapshot5 » 05 Jun 2011, 02:35
r9279
Many cards have duplicate text. See:
Clone
Siren's Call (good example; it shows '\r\n' not being translated, which may give a clue)
Vesuvan Doppelganger
-slapshot5
Many cards have duplicate text. See:
Clone
Siren's Call (good example; it shows '\r\n' not being translated, which may give a clue)
Vesuvan Doppelganger
-slapshot5
- slapshot5
- Programmer
- Posts: 1391
- Joined: 03 Jan 2010, 17:47
- Location: Mac OS X
- Has thanked: 25 times
- Been thanked: 68 times
Re: SVN Bug Reports
by Chris H. » 05 Jun 2011, 02:46
`slapshot5 wrote:r9279
Many cards have duplicate text. See:
Clone
Siren's Call (good example; it shows '\r\n' not being translated, which may give a clue)
Vesuvan Doppelganger
-slapshot5
I have spent a lot of time trying to figure this one out. It appears that somewhere in the code some cards are getting a:
getText() + getSpellText()
It seems to only show up in hard coded cards but I could be mistaken.
There are a number of cards showing the "\r\n" that Slapshot mentions. If the second copy was being added somewhere in the getText() method then these symbols should be replaced.
-
Chris H. - Forge Moderator
- Posts: 6320
- Joined: 04 Nov 2008, 12:11
- Location: Mac OS X Yosemite
- Has thanked: 644 times
- Been thanked: 643 times
Re: SVN Bug Reports
by Sloth » 05 Jun 2011, 08:14
The problem with duplicate spell texts comes from cards with:
A. "SpellAbility XXX = new Spell(card)" and
B. "card.addSpellAbility(XXX);"
just put "XXX.setDescription("");" before B and the duplicate is gone (But maybe the stack description too).
A. "SpellAbility XXX = new Spell(card)" and
B. "card.addSpellAbility(XXX);"
just put "XXX.setDescription("");" before B and the duplicate is gone (But maybe the stack description too).
-
Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: SVN Bug Reports
by slapshot5 » 05 Jun 2011, 09:21
What happened? It never used to be this way?Sloth wrote:The problem with duplicate spell texts comes from cards with:
A. "SpellAbility XXX = new Spell(card)" and
B. "card.addSpellAbility(XXX);"
just put "XXX.setDescription("");" before B and the duplicate is gone (But maybe the stack description too).
And this works on Siren's Call for example, but on Clone, this forced the getStackDescription() text to be added to the card.
-slapshot5
- slapshot5
- Programmer
- Posts: 1391
- Joined: 03 Jan 2010, 17:47
- Location: Mac OS X
- Has thanked: 25 times
- Been thanked: 68 times
Re: SVN Bug Reports
by Sloth » 05 Jun 2011, 10:47
I just did a test with Mana Leak and it worked (both without paying and with paying). It must be a problem with the X cost.friarsol wrote:Has anyone noticed the AI properly paying mana for CounterMagic with UnlessCosts? I just Condescend -ed his spell for 3, it resolved (allowing me to Scry) but then the Spell still resolved. The AI only had 2 mana available, and neither land is now tapped.
-
Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: SVN Bug Reports
by friarsol » 05 Jun 2011, 11:31
I can't say for those, but I wonder if some of the other ones have to do with my converting some Auras to be Spell_Permanents.slapshot5 wrote:What happened? It never used to be this way?Sloth wrote:The problem with duplicate spell texts comes from cards with:
A. "SpellAbility XXX = new Spell(card)" and
B. "card.addSpellAbility(XXX);"
just put "XXX.setDescription("");" before B and the duplicate is gone (But maybe the stack description too).
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: SVN Bug Reports
by Sloth » 05 Jun 2011, 12:33
I tested Condescend and it also worked (both without paying and with paying).friarsol wrote:Has anyone noticed the AI properly paying mana for CounterMagic with UnlessCosts? I just Condescend -ed his spell for 3, it resolved (allowing me to Scry) but then the Spell still resolved. The AI only had 2 mana available, and neither land is now tapped.
-
Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
Re: SVN Bug Reports
by friarsol » 05 Jun 2011, 14:16
Alright good. Must just be my local changes. I didn't think it would affect that, but wasn't sure.Sloth wrote:I tested Condescend and it also worked (both without paying and with paying).
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Re: SVN Bug Reports
by goonjamin » 05 Jun 2011, 20:47
r9307
Hi, I receive the error in the attached jpg when trying to launch this SVN revision with the forge.exe that comes with build scripts? All my previous builds up to yesterday worked fine? Any idea how I can fix this?
Here is a little more info on the error when I try to run via the command line.
Hi, I receive the error in the attached jpg when trying to launch this SVN revision with the forge.exe that comes with build scripts? All my previous builds up to yesterday worked fine? Any idea how I can fix this?
Here is a little more info on the error when I try to run via the command line.
- Code: Select all
C:\Users\Mike\Downloads\forge>java -Xmx1024m -jar run-forge.jar
Exception in thread "main" java.lang.NoClassDefFoundError: forge/BoosterDraft
Caused by: java.lang.ClassNotFoundException: forge.BoosterDraft
at java.net.URLClassLoader$1.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(Unknown Source)
at java.lang.ClassLoader.loadClass(Unknown Source)
at sun.misc.Launcher$AppClassLoader.loadClass(Unknown Source)
at java.lang.ClassLoader.loadClass(Unknown Source)
Could not find the main class: forge.Gui_NewGame. Program will exit.
Re: SVN Bug Reports
by friarsol » 06 Jun 2011, 00:06
r9314
Pupeteer Clique doesn't seem to give Raised Creature Haste or Exile keywords. Probably has to do with the fact the Target happens before a ChangeZone
Also, my 3/3 Mindless Automaton took 2 Wither damage, and now it has 3 +1/+1 counters and 2 -1/-1 counters. It should only have 1 +1/+1 Counter.
Pupeteer Clique doesn't seem to give Raised Creature Haste or Exile keywords. Probably has to do with the fact the Target happens before a ChangeZone
Also, my 3/3 Mindless Automaton took 2 Wither damage, and now it has 3 +1/+1 counters and 2 -1/-1 counters. It should only have 1 +1/+1 Counter.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
Who is online
Users browsing this forum: No registered users and 18 guests