Page 67 of 86

Re: SVN Bug Reports

PostPosted: 04 Jun 2011, 15:54
by Sloth
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.
The AI won't use the ability for now. Thanks Agetian.

Re: SVN Bug Reports

PostPosted: 04 Jun 2011, 17:15
by lazylockie
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

Re: SVN Bug Reports

PostPosted: 04 Jun 2011, 17:27
by Sloth
Biribau wrote:.

Goblin Grenade gives an error when it's played by human - as soon as you sacrifice.
I just tried Goblin Grenade and it worked fine. What did you sacrifice Biribau? (and maybe can you post the error report next time)

Re: SVN Bug Reports

PostPosted: 05 Jun 2011, 02:03
by friarsol
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

Re: SVN Bug Reports

PostPosted: 05 Jun 2011, 02:20
by friarsol
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.

Re: SVN Bug Reports

PostPosted: 05 Jun 2011, 02:35
by slapshot5
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

Re: SVN Bug Reports

PostPosted: 05 Jun 2011, 02:46
by Chris H.
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.

Re: SVN Bug Reports

PostPosted: 05 Jun 2011, 08:14
by Sloth
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).

Re: SVN Bug Reports

PostPosted: 05 Jun 2011, 09:21
by slapshot5
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).
What happened? It never used to be this way?

And this works on Siren's Call for example, but on Clone, this forced the getStackDescription() text to be added to the card.

-slapshot5

Re: SVN Bug Reports

PostPosted: 05 Jun 2011, 10:47
by Sloth
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.
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.

Re: SVN Bug Reports

PostPosted: 05 Jun 2011, 11:31
by friarsol
slapshot5 wrote:
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).
What happened? It never used to be this way?
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.

Re: SVN Bug Reports

PostPosted: 05 Jun 2011, 12:33
by Sloth
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.
I tested Condescend and it also worked (both without paying and with paying).

Re: SVN Bug Reports

PostPosted: 05 Jun 2011, 14:16
by friarsol
Sloth wrote:I tested Condescend and it also worked (both without paying and with paying).
Alright good. Must just be my local changes. I didn't think it would affect that, but wasn't sure.

Re: SVN Bug Reports

PostPosted: 05 Jun 2011, 20:47
by goonjamin
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?

error.jpg


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

PostPosted: 06 Jun 2011, 00:06
by friarsol
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.