Collectible Card Game Headquarters

I was testing the deck editor and listed in the Aspect of Mongoose type is "Enchanted Creature has shroud".

This should be in the card's text in instead of the card type.

apthaven wrote:I was testing the deck editor and listed in the Aspect of Mongoose type is "Enchanted Creature has shroud".

This should be in the card's text in instead of the card type.

`
Currently, the Aspect of Mongoose appears in the cards.txt file as:

Code: Select all

Aspect of Mongoose
1 G
Enchant creature. Enchanted creature has shroud.
When this card is put into a graveyard from the battlefield, return this card to its owner's hand

`
I suspect that we may need to change it to look like this:

Code: Select all

Aspect of Mongoose
1 G
Enchantment Aura
Enchant creature. Enchanted creature has shroud.
When this card is put into a graveyard from the battlefield, return this card to its owner's hand

`
Note that the card missing the line "Enchantment Aura".

Three token pictures are missing (Thrull, Camarid and Citizen produced by Sarpadian Empires, Vol. VII)
Also, Think Twice doesn't have flashback.

I noticed a minor bug in the deck editor with the new 10-02-2009 version.

Clicking and dragging the mouse pointer over the text displayed inside of the text area for the Card Detail panel in the Deck Editors will result in the following error:

(Clicking and not dragging appears not to trigger the error.)

Code: Select all

An error has occured. You can copy/paste this message or save it to a file.
Please report this, plus what you tried to do, to:
   http://www.slightlymagic.net/forum/viewforum.php?f=26
If you don't want to register an account, you can mail it directly to
   mtgrares@yahoo.com

11

Detailed error trace:
java.lang.ArrayIndexOutOfBoundsException: 11
   at sun.font.FontDesignMetrics.charsWidth(FontDesignMetrics.java:492)
   at javax.swing.text.Utilities.getTabbedTextOffset(Utilities.java:381)
   at javax.swing.text.Utilities.getTabbedTextOffset(Utilities.java:302)
   at javax.swing.text.Utilities.getTabbedTextOffset(Utilities.java:286)
   at javax.swing.text.WrappedPlainView$WrappedLine.viewToModel(WrappedPlainView.java:697)
   at javax.swing.text.CompositeView.viewToModel(CompositeView.java:408)
   at javax.swing.text.BoxView.viewToModel(BoxView.java:486)
   at javax.swing.plaf.basic.BasicTextUI$RootView.viewToModel(BasicTextUI.java:1541)
   at javax.swing.plaf.basic.BasicTextUI.viewToModel(BasicTextUI.java:1090)
   at javax.swing.text.DefaultCaret.positionCaret(DefaultCaret.java:292)
   at javax.swing.text.DefaultCaret.adjustCaret(DefaultCaret.java:497)
   at javax.swing.text.DefaultCaret.adjustCaretAndFocus(DefaultCaret.java:485)
   at javax.swing.text.DefaultCaret.mousePressed(DefaultCaret.java:475)
   at com.apple.laf.AquaCaret.mousePressed(AquaCaret.java:124)
   at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:263)
   at java.awt.Component.processMouseEvent(Component.java:6345)
   at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
   at java.awt.Component.processEvent(Component.java:6113)
   at java.awt.Container.processEvent(Container.java:2085)
   at java.awt.Component.dispatchEventImpl(Component.java:4714)
   at java.awt.Container.dispatchEventImpl(Container.java:2143)
   at java.awt.Component.dispatchEvent(Component.java:4544)
   at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4618)
   at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4279)
   at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4212)
   at java.awt.Container.dispatchEventImpl(Container.java:2129)
   at java.awt.Window.dispatchEventImpl(Window.java:2475)
   at java.awt.Component.dispatchEvent(Component.java:4544)
   at java.awt.EventQueue.dispatchEvent(EventQueue.java:635)
   at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:296)
   at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:211)
   at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:201)
   at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:196)
   at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:188)
   at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)


Oops, as always, the bugs slip through

I will probably release something in a week or so that should fix most of the bugs that will undoubtedly follow.
I'm not sure what's going on with the mouse dragging thing in the deck editor though, I can't seem to reproduce it either.

DennisBergkamp wrote:Oops, as always, the bugs slip through

Oh my, you must have foreseen my next bug report.




I have a list of fairly minor bugs with the new 10-02-2009 … sorry about that. I hope that they all will be easy to fix. These are not mission critical. They can be part of the next update.



(1) The card Volcanic Fallout needs to be changed to an Instant. The card should look like this:

Volcanic Fallout
1 R R
Instant
Volcanic Fallout deals 2 damage to each creature and each player.
This card can't be countered.



(2) The card Cao Ren, Wei Commander is missing a type, it should be: "Legendary Creature Human Soldier Warrior". The card should look like this:

Cao Ren, Wei Commander
2 B B
Legendary Creature Human Soldier Warrior
When Cao Ren, Wei Commander enters the battlefield, you lose 3 life.
3/3
Horsemanship



(3) We need to add the "Enchantment Aura" line to the card Aspect of Mongoose. It should look like this:

Aspect of Mongoose
1 G
Enchantment Aura
Enchant creature. Enchanted creature has shroud.
When this card is put into a graveyard from the battlefield, return this card to its owner's hand



(4) The card Dosan's Oldest Chant for some reason is displaying "Cycling 1 W (1 W, Discard this card: Draw a card.)" in the text area of the card detail pane. This card is supposed to have Cantrip and not have cycling. The problem is not in the cards.txt file.

The problem is in the CardFactory.java file. Might try placing the last line "card.addSpellAbility(CardFactoryUtil.ability_cycle(card, "1 W"));" in an If statement. We give Renewed Faith cycling and not to both spells.



(5) The card Guilty Conscience is missing the text "Enchant creature. " The card should look like this:

Guilty Conscience
W
Enchantment Aura
Enchant creature. Whenever enchanted creature deals damage, Guilty Conscience deals that much damage to that creature.



(6) The card Obsidian Fireheart is missing the text string which displays in the text area of the card detail panel. The card should look like this:

Obsidian Fireheart
1 R R R
Creature Elemental
1 R R: Put a blaze counter on target land without a blaze counter on it. For as long as that land has a blaze counter on it, it has "At the beginning of your upkeep, this land deals 1 damage to you." (The land continues to burn after Obsidian Fireheart has left the battlefield.)
4/4



(7) The card Trusty Machete has the wrong Equip cost listed in the text area of the card detail panel. The true cost is "Equip: 2".



(8) The "This card can't be countered." keyword will appear on creature cards but not spells, I am making an assumption. The string appears on the creature card Terra Stomper but not on the spells Wreak Havoc, Obliterate or on Volcanic Fallout.

This problem may be similar to the cycling code problem that we had. We moved the cycling code to the bottom of CardFactory and the cycling keyword now works with spells. I have not yet tested this.



(9) The card Wreak Havoc has the wrong casting cost. The card should look like this:

Wreak Havoc
2 R G
Sorcery
Destroy target artifact or land.
spDestroyTgt:Artifact,Land
This card can't be countered.



(10) The card Sphinx of Jwar Isle is missing it's descriptive text string "You may look at the top card of your library. (You may do this at any time.)". If we change the cards.txt entry to this then the problem may be fixed:

Sphinx of Jwar Isle
4 U U
Creature Sphinx
You may look at the top card of your library. (You may do this at any time.)
5/5
Flying
Shroud



(11) The card Sarpadian Empires, Vol. VII is missing the string "{3}, : Put a 1/1 creature token of the chosen color and type onto the battlefield.". If we change the cards.txt entry to this then the problem may be fixed:

Sarpadian Empires, Vol. VII
3
Artifact
As Sarpadian Empires, Vol. VII enters the battlefield, choose white Citizen, blue Camarid, black Thrull, red Goblin, or green Saproling. 3, T: Put a 1/1 creature token of the chosen color and type onto the battlefield.



(12) The card Wall of Diffusion is missing the string "Wall of Diffusion can block creatures with shadow as though they didn't have shadow.". If we change the cards.txt entry to this then the problem may be fixed:

Wall of Diffusion
1 R
Creature Wall
Wall of Diffusion can block creatures with shadow as though they didn't have shadow.
0/5
Defender

Pfew, no I certainly didn't expect that!
I fixed most of these just now, most of them are indeed cards.txt related. But not all of them, sometimes I forgot to add descriptions for them in CardFactory.java.

Currently, Sarpadian Empires, Vol. VII is tricky: the ability gets added as soon as it comes into play (and is shown on the text then), but not in the deck editor nor when the card is in hand.
This card cannot be countered is tricky too, I just checked Obliterate, and I think I actually forgot to code it, so I'm not sure what's going on with those.
But it looks like the keywords associated with the others are wiping out the keyword, kind of similar to the cycling code, yes.

But it looks like the keywords associated with the others are wiping out the keyword, kind of similar to the cycling code, yes.

I think we definitely should put an if block around all spell keywords that does the clearSpellAbility once and once only.

Secluded Steppe and others should come into play tapped. (I always pretend they do to be fair, but anyway . . . )

Also, the game should really implement stops for various phases and steps or, barring that, at least one for declaring attackers. I cannot count the times I've drafted a card along the lines of Master Decoy or Flash Foliage only to find out the hard way how utterly useless it is (just for now, hopefully (wink wink nudge nudge implementors)). And even with stacked AI land enabled, the computer never playing first is just too great a handicap for it.

EDIT: Sower of Temptation and cards with similar coming-into-play abilities should still be playable without a creature or other target already in play, by the way.

EDIT2: This one's nasty: Impulse with an empty library forces you to restart the game (not the entire program, but still a nuisance). Is the same true for other library-accessing cards, and what if the computer attempts the same action . . . ?

The reason Secluded Steppe (and Forgotten Cave) doesn't come into play tapped is because it misses a dot at the end of the keyword: "Comes into play tapped."
I just fixed Impulse (which, by the way, can't be played by the AI).

Wow, you're fast! I wish I were as good at squashing bugs as you . . .

On another note, however, I received an error from the computer trying to enumerate its counterspells whenever I cast a spell of my own (at least I believe that's what it was trying to do -- I'm unable to reproduce the situation now, and I forgot the name of the subroutine).

By the way, Polluted Mire can't be cycled, and these lands I've been examining lately sometimes have mana costs listed in the deck editor. If the latter is necessary to produce the correct behaviour, however, then by all means I have no problem with it.

EDIT: Soul Warden's and Essence Warden's abilities should not trigger upon activation of Faerie Conclave or other animatable permanents.

Chittering Rats has you put a random card from your hand on top of your library. You should be able to choose.

Also, if Wall of Diffusion works correctly now, was AEtherflame Wall corrected in the same way? Also, this might mean Heartwood Dryad can be added now? Perhaps even AEther Web?

freestorageaccount wrote:By the way, Polluted Mire can't be cycled,

`
I do not think that cycling ever worked with lands.

The code which places lands onto the battlefield's land zone is separate from the code that lets us cast a spell.

Marek14 wrote:Chittering Rats has you put a random card from your hand on top of your library. You should be able to choose.

Also, if Wall of Diffusion works correctly now, was AEtherflame Wall corrected in the same way? Also, this might mean Heartwood Dryad can be added now? Perhaps even AEther Web?

Yes, the Chittering Rats bug has been in there for ages now... I've been meaning to fix it but I haven't gotten around to it (it's a bit of a boring fix, if you know what I mean).

AEtherflame Wall should also work correctly, except it also didn't have the text, I just fixed that. I will add Heartwood Dryad, maybe we should turn this into a keyword too (it's very easy) - that way we could add AEther Web a lot more easily too.