Collectible Card Game Headquarters

PalladiaMors wrote:I was trying to do the card Override and got a crash which I'm pretty sure is related to the card's name conflicting with the @Override code. Is there any way around this?

CRASH REPORT FOR MAGARENA THREAD Thread[AWT-EventQueue-0,6,main]
CREATED ON 2014/09/15 22:43:43
MAGARENA VERSION 1.54, JRE 1.7.0_67, OS Windows 7_6.1 x86
================================
Used Memory: 75M
Free Memory: 172M
Total Memory: 247M
Max Memory: 494M
================================
-Xms256m
-Xmx512m
-Xverify:none


Turn : 1 Phase : Mulligan Step : Begin Player : PalladiaMors Score : 0
0] Player : PalladiaMors Life : 20 Delayed : Total=0
1] Player : Mini Max Life : 20 Delayed : Total=0
Stack : 0
Score = 0

Exception from controller.runGame: java.util.concurrent.ExecutionException: org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
C:\Magarena\Magarena\scripts\Override.groovy: 3: class Override is not an annotation in @Override
@ line 3, column 9.
@Override
^

C:\Magarena\Magarena\scripts\Override.groovy: 12: class Override is not an annotation in @Override
@ line 12, column 9.
@Override
^

2 errors

java.lang.RuntimeException: java.util.concurrent.ExecutionException: org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
C:\Magarena\Magarena\scripts\Override.groovy: 3: class Override is not an annotation in @Override
@ line 3, column 9.
@Override
^

C:\Magarena\Magarena\scripts\Override.groovy: 12: class Override is not an annotation in @Override
@ line 12, column 9.
@Override
^

2 errors

at magic.ui.screen.DuelGameScreen$1.done(DuelGameScreen.java:47)
at javax.swing.SwingWorker$5.run(Unknown Source)
at javax.swing.SwingWorker$DoSubmitAccumulativeRunnable.run(Unknown Source)
at sun.swing.AccumulativeRunnable.run(Unknown Source)
at javax.swing.SwingWorker$DoSubmitAccumulativeRunnable.actionPerformed(Unknown Source)
at javax.swing.Timer.fireActionPerformed(Unknown Source)
at javax.swing.Timer$DoPostEvent.run(Unknown Source)
at java.awt.event.InvocationEvent.dispatch(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$200(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)
Caused by: java.util.concurrent.ExecutionException: org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
C:\Magarena\Magarena\scripts\Override.groovy: 3: class Override is not an annotation in @Override
@ line 3, column 9.
@Override
^

C:\Magarena\Magarena\scripts\Override.groovy: 12: class Override is not an annotation in @Override
@ line 12, column 9.
@Override
^

2 errors

at java.util.concurrent.FutureTask.report(Unknown Source)
at java.util.concurrent.FutureTask.get(Unknown Source)
at javax.swing.SwingWorker.get(Unknown Source)
at magic.ui.screen.DuelGameScreen$1.done(DuelGameScreen.java:41)
... 20 more
Caused by: org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
C:\Magarena\Magarena\scripts\Override.groovy: 3: class Override is not an annotation in @Override
@ line 3, column 9.
@Override
^

C:\Magarena\Magarena\scripts\Override.groovy: 12: class Override is not an annotation in @Override
@ line 12, column 9.
@Override
^

2 errors

at org.codehaus.groovy.control.ErrorCollector.failIfErrors(ErrorCollector.java:309)
at org.codehaus.groovy.control.CompilationUnit.applyToPrimaryClassNodes(CompilationUnit.java:1067)
at org.codehaus.groovy.control.CompilationUnit.doPhaseOperation(CompilationUnit.java:583)
at org.codehaus.groovy.control.CompilationUnit.processPhaseOperations(CompilationUnit.java:561)
at org.codehaus.groovy.control.CompilationUnit.compile(CompilationUnit.java:538)
at groovy.lang.GroovyClassLoader.doParseClass(GroovyClassLoader.java:286)
at groovy.lang.GroovyClassLoader.parseClass(GroovyClassLoader.java:259)
at groovy.lang.GroovyShell.parseClass(GroovyShell.java:674)
at groovy.lang.GroovyShell.parse(GroovyShell.java:686)
at groovy.lang.GroovyShell.evaluate(GroovyShell.java:568)
at groovy.lang.GroovyShell.evaluate(GroovyShell.java:617)
at magic.data.CardDefinitions.addCardSpecificGroovyCode(CardDefinitions.java:140)
at magic.data.CardProperty$30.setProperty(CardProperty.java:185)
at magic.model.MagicCardDefinition.loadAbilities(MagicCardDefinition.java:179)
at magic.model.MagicCard.<init>(MagicCard.java:98)
at magic.model.MagicCard.<init>(MagicCard.java:86)
at magic.model.MagicPlayer.createHandAndLibrary(MagicPlayer.java:389)
at magic.model.MagicDuel.nextGame(MagicDuel.java:181)
at magic.ui.screen.DuelGameScreen$1.doInBackground(DuelGameScreen.java:36)
at magic.ui.screen.DuelGameScreen$1.doInBackground(DuelGameScreen.java:31)
at javax.swing.SwingWorker$1.call(Unknown Source)
at java.util.concurrent.FutureTask.run(Unknown Source)
at javax.swing.SwingWorker.run(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.lang.Thread.run(Unknown Source)

Code: Select all

name=Override
image=http://mtgimage.com/card/override.jpg
value=2.500
rarity=C
type=Instant
cost={2}{U}
timing=counter
requires_groovy_code
oracle=Counter target spell unless its controller pays {1} for each artifact you control.


Code: Select all

[
    new MagicSpellCardEvent() {
        @Override
        public MagicEvent getEvent(final MagicCardOnStack cardOnStack,final MagicPayedCost payedCost) {
            return new MagicEvent(
                cardOnStack,
                MagicTargetChoice.NEG_TARGET_SPELL,
                this,
                "Counter target spell\$ unless its controller pays {1} for each artifact PN controls."
            );
        }
        @Override
        public void executeEvent(final MagicGame game, final MagicEvent event) {
            event.processTargetCardOnStack(game, {
                final int amount = event.getPlayer().getNrOfPermanents(MagicType.Artifact);
                game.addEvent(new MagicCounterUnlessEvent(event.getSource(),it,MagicManaCost.create("{"+amount+"}")));
            });
        }
    }
]


Code: Select all

name=Novijen Sages
image=http://mtgimage.com/card/novijen%20sages.jpg
value=2.500
rarity=R
type=Creature
subtype=Human,Advisor,Mutant
cost={4}{U}{U}
pt=0/0
ability=Graft 4;\
        {1}, Remove a +1/+1 counter from a creature you control, remove a +1/+1 counter from a creature you control: Draw a card.
timing=main
oracle=Graft 4. {1}, Remove two +1/+1 counters from among creatures you control: Draw a card.


Code: Select all

name=Phantom Whelp
image=http://mtgimage.com/card/phantom%20whelp.jpg
value=2.500
rarity=C
type=Creature
subtype=Illusion,Hound
cost={1}{U}
pt=2/2
timing=main
requires_groovy_code=Windscouter
oracle=When Phantom Whelp attacks or blocks, return it to its owner's hand at end of combat.


Code: Select all

name=Mire Blight
image=http://mtgimage.com/card/mire%20blight.jpg
value=2.500
rarity=C
type=Enchantment
subtype=Aura
cost={B}
timing=aura
enchant=can't block,neg creature
requires_groovy_code=Mortal Wound
oracle=Enchant creature. When enchanted creature is dealt damage, destroy it.


Code: Select all

name=Frozen Solid
image=http://mtgimage.com/card/frozen%20solid.jpg
value=2.500
rarity=C
type=Enchantment
subtype=Aura
cost={1}{U}{U}
ability=Enchanted creature doesn't untap during its controller's untap step.
timing=aura
enchant=can't attack or block,neg creature
requires_groovy_code=Mortal Wound
oracle=Enchant creature. Enchanted creature doesn't untap during its controller's untap step. When enchanted creature is dealt damage, destroy it.


PalladiaMors wrote:This one is just a simple matter of separating the removal of multiple counters in the cost.

Code: Select all

name=Insectile Aberration
image=http://mtgimage.com/card/insectile%20aberration.jpg
value=2.500
rarity=C
type=Creature
subtype=Human,Insect
color=u
pt=3/2
ability=Flying
transform=Delver of Secrets
timing=main
hidden
oracle=Flying

PalladiaMors wrote:Insectile Aberration for Delver of Secrets. Firemind is running a test on the card which I've no idea if it's going to pass since it's not a stand-alone card.

Code: Select all

name=Insectile Aberration
image=http://mtgimage.com/card/insectile%20aberration.jpg
value=2.500
rarity=C
type=Creature
subtype=Human,Insect
color=u
pt=3/2
ability=Flying
transform=Delver of Secrets
timing=main
hidden
oracle=Flying

Code: Select all

[
    new MagicAtYourUpkeepTrigger() {
        @Override
        public MagicEvent executeTrigger(final MagicGame game,final MagicPermanent permanent,final MagicPlayer upkeepPlayer) {
            return new MagicEvent(
                permanent,
                new MagicMayChoice("Reveal the top card of your library?"),
                this,
                "Look at the top card of PN's library. You may\$ reveal that card. If an instant or sorcery card is revealed this way, transform SN."
            );
        }
        @Override
        public void executeEvent(final MagicGame game, final MagicEvent event) {
            for (final MagicCard card : event.getPlayer().getLibrary().getCardsFromTop(1)) {
                game.doAction(new MagicRemoveCardAction(
                    card,
                    MagicLocationType.OwnersLibrary
                ));
                game.doAction(new MagicMoveCardAction(
                    card,
                    MagicLocationType.OwnersLibrary,
                      MagicLocationType.TopOfOwnersLibrary
                ));
            if (event.isYes() && card.hasType(MagicType.Instant) || card.hasType(MagicType.Sorcery)) {
            game.doAction(new MagicTransformAction(event.getPermanent()));
            }
            }
        }
    }
]


PalladiaMors wrote:I've submitted Goblin Caves and Goblin Shrine but now I've noticed that the Oracle text checks if the enchanted land is a basic mountain, rather than checking if it has the mountain subtype. I don't know if that's fixable, I think it could check the card's name instead of subtype but I don't know if that would work with cards that get transformed into mountains. If it's not easily fixable, please ignore the submissions.

Code: Select all

hasType(MagicType.Basic) && hasSubType(MagicSubType.Mountain)

PalladiaMors wrote:Ah, cool, I actually didn't know that - maybe it opens the path to some other cards, I'll check later. Also, I'm a bit unsure about the way I did Leviathan. Now that I'm thinking a bit more I believe its attack ability should be possible to activate at any time during a turn, it shouldn't just trigger at the beginning of combat (and it should be possible to activate it multiple times, also). Yeah, taking a second look that one is just plain wrong... Probably needs to get trashed. I did think my way was clunky but somehow I got optimistic while doing it and thought it was pretty much equivalent, lol.

It would be useful to have a way to edit scripts after submitting them, I've lost count of the amount of times I've noticed some kind of mistake after submitting a card. Melvin later sorts them out - and the ones I don't notice as well - but that's more unnecessary work for him.

Edit: Hmm, actually, I hadn't seen the very last part of Leviathan's oracle. I think that might mean the script's OK. Only issue is that I think it should trigger during the declare attackers step instead of beginning of combat.