Collectible Card Game Headquarters

Marek14 wrote:I played Mana Clash, got heads for opponent and tails for myself, took 1 damage, and then the effect ended. Mana Clash should continue until both players get heads.

Fiery Gambit doesn't work for me (no coin flips appear). I had Krark's Thumb and cast Fiery Gambit vie Eye of the Storm.

EDIT: Next attempt to cast Fiery Gambit worked, except that when I guessed tails and both Krark's Thumb flips were heads, I won the flip anyway.

EDIT2: I just can't get Fiery Gambit to deal any damage, no matter how I win or lose the flips. Also, the effect doesn't stop after I lose the flip, I still can flip more coins, which is wrong.

So far I narrowed this down to the fact that WinSubAbility, LoseSubAbility, HeadsSubAbility and TailsSubAbility additional abilities are apparently not created somewhere in the code, and thus they are returned as "null" later on when the game queries them from SAs. Not sure why this is happening yet though.

EDIT: Ok, I figured out what was wrong, it was similar to the Charm API situation where the additional abilities were not filled for the API and that made the relevant abilities fail. I committed a similar solution in r32867, though I'm not sure about a couple things: is there a reason why we aren't just filling in additional abilities by default (for all APIs)? And also, maybe there's a better solution? :/ (not very familiar with this part of code tbh).

- Agetian

@Agetian: i removed all special cases.

i added them because of that card where the spellability tree was broken in a self-loop. and because i did rewrote that card to DONT do that anymore, the special cases are not needed anymore.

Description: [in Commander, declaring which creature to attack AI, Notably a 30/30 Minion Token]

NullPointerException | Open

Code: Select all

Forge Version:    1.5.58-SNAPSHOT-r32865
Operating System: Windows 7 6.1 amd64
Java Version:     1.8.0_25 Oracle Corporation

java.lang.NullPointerException
   at java.util.concurrent.ConcurrentHashMap.putVal(Unknown Source)
   at java.util.concurrent.ConcurrentHashMap.put(Unknown Source)
   at forge.game.combat.CombatView.addAttackingBand(CombatView.java:178)
   at forge.game.GameView.updateCombat(GameView.java:151)
   at forge.game.Game.updateCombatForView(Game.java:354)
   at forge.game.card.Card.updateAttackingForView(Card.java:575)
   at forge.game.combat.Combat.addAttacker(Combat.java:241)
   at forge.match.input.InputAttack.declareAttacker(InputAttack.java:255)
   at forge.match.input.InputAttack.onCardSelected(InputAttack.java:217)
   at forge.match.input.InputBase.selectCard(InputBase.java:106)
   at forge.match.input.InputProxy.selectCard(InputProxy.java:145)
   at forge.player.PlayerControllerHuman.selectCard(PlayerControllerHuman.java:1481)
   at forge.view.arcane.PlayArea.selectCard(PlayArea.java:564)
   at forge.view.arcane.PlayArea.mouseLeftClicked(PlayArea.java:536)
   at forge.view.arcane.CardPanelContainer$2.mouseReleased(CardPanelContainer.java:165)
   at java.awt.Component.processMouseEvent(Unknown Source)
   at javax.swing.JComponent.processMouseEvent(Unknown Source)
   at java.awt.Component.processEvent(Unknown Source)
   at java.awt.Container.processEvent(Unknown Source)
   at java.awt.Component.dispatchEventImpl(Unknown Source)
   at java.awt.Container.dispatchEventImpl(Unknown Source)
   at java.awt.Component.dispatchEvent(Unknown Source)
   at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
   at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
   at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
   at java.awt.Container.dispatchEventImpl(Unknown Source)
   at java.awt.Window.dispatchEventImpl(Unknown Source)
   at java.awt.Component.dispatchEvent(Unknown Source)
   at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
   at java.awt.EventQueue.access$400(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.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
   at java.awt.EventQueue$4.run(Unknown Source)
   at java.awt.EventQueue$4.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)


Description: [While having Archangel of Thune OTB, played, consecutively, Nomads' Assembly and Deploy to the Front ]

OutOfMemoryError | Open

Code: Select all

Forge Version:    1.5.58-SNAPSHOT-r32862
Operating System: Windows 7 6.1 amd64
Java Version:     1.8.0_101 Oracle Corporation

java.lang.OutOfMemoryError: GC overhead limit exceeded
   at javax.swing.text.WrappedPlainView.loadChildren(Unknown Source)
   at javax.swing.text.CompositeView.setParent(Unknown Source)
   at javax.swing.plaf.basic.BasicTextUI$RootView.setView(Unknown Source)
   at javax.swing.plaf.basic.BasicTextUI.setView(Unknown Source)
   at javax.swing.plaf.basic.BasicTextUI.modelChanged(Unknown Source)
   at javax.swing.plaf.basic.BasicTextUI$UpdateHandler.propertyChange(Unknown Source)
   at java.beans.PropertyChangeSupport.fire(Unknown Source)
   at java.beans.PropertyChangeSupport.firePropertyChange(Unknown Source)
   at java.beans.PropertyChangeSupport.firePropertyChange(Unknown Source)
   at java.awt.Component.firePropertyChange(Unknown Source)
   at java.awt.Component.setFont(Unknown Source)
   at java.awt.Container.setFont(Unknown Source)
   at javax.swing.JComponent.setFont(Unknown Source)
   at javax.swing.JTextArea.setFont(Unknown Source)
   at forge.toolbox.FSkin$SkinnedTextArea.setFont(FSkin.java:2600)
   at forge.toolbox.FSkin$ComponentSkin.setFont(FSkin.java:1621)
   at forge.toolbox.FSkin$SkinnedTextArea.setFont(FSkin.java:2599)
   at forge.screens.match.views.VStack$StackInstanceTextArea.<init>(VStack.java:194)
   at forge.screens.match.views.VStack.updateStack(VStack.java:134)
   at forge.screens.match.controllers.CStack.update(CStack.java:45)
   at forge.screens.match.CMatchUI$4.run(CMatchUI.java:652)
   at forge.GuiDesktop.invokeInEdtNow(GuiDesktop.java:79)
   at forge.FThreads.invokeInEdtNowOrLater(FThreads.java:30)
   at forge.screens.match.CMatchUI.updateStack(CMatchUI.java:650)
   at forge.control.FControlGameEventHandler$1.run(FControlGameEventHandler.java:93)
   at java.awt.event.InvocationEvent.dispatch(Unknown Source)
   at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
   at java.awt.EventQueue.access$500(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$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)


Apparently Dragon Presence no longer does anything useful (if anything at all). Cards like Draconic Roar, Foul-Tongue Invocation, Dragonlord's Prerogative and other cards with Dragon Presence always operate as if there is a Dragon present or revealed, even if there is none at all. :/ Trying to debug this I found out that the condition for Dragon Presence is never actually checked (the relevant code is never called at all, none of my breakpoints at anything Presence-related ever fired). Fix attempted in r32878.

- Agetian

@Agetian: i don't know if that is the right fix.

the check is done in CardTraitBase.meetsCommonRequirements ... i think it would be better to look why that part is not called.

PS: StaticAbility#checkConditions should be rewritten to use meetsCommonRequirements too.

Edit: okay now i see why that is done ... SpellAbilityRestriction & SpellAbilityCondition don't use meetsCommonRequirements .. (that seems to be only used for Replacement and Trigger)

i think some day in the future we might rework that, so that the code isn't at multiple places at once.

Yeah, sounds like it's a good target for refactoring in the future. Seems like it's working correctly so far though, should we leave it this way for now until one of us or someone else finds an opportunity to refactor this not to duplicate the checks?

- Agetian

Hanmac wrote:Edit: okay now i see why that is done ... SpellAbilityRestriction & SpellAbilityCondition don't use meetsCommonRequirements .. (that seems to be only used for Replacement and Trigger)

i think some day in the future we might rework that, so that the code isn't at multiple places at once.

IIRC Conditions for SpellAbilities existed first. Then Triggers and Replacements were redone about the same time and used lots of the same parts.

hey there, new to the forum's but been using Forge for about a year, its amazing.

I have a Gisa and Geralf commander deck, and somewhere in the last few weeks, their ability as Commander to cast zombie cards from the graveyard no longer works.

I am currently using snapshot 32887. No error or anything comes up, used to be you could click a zombie in the GY and it would prompt the casting cost popup, but now when you click, nothing happens.

Searched the forum for a Gisa and Geralf related bug post but didn't find anything, so hopefully this isn't old news!

Thanks!

toomadtoplay wrote:hey there, new to the forum's but been using Forge for about a year, its amazing.

I have a Gisa and Geralf commander deck, and somewhere in the last few weeks, their ability as Commander to cast zombie cards from the graveyard no longer works.

I am currently using snapshot 32887. No error or anything comes up, used to be you could click a zombie in the GY and it would prompt the casting cost popup, but now when you click, nothing happens.

Searched the forum for a Gisa and Geralf related bug post but didn't find anything, so hopefully this isn't old news!

Thanks!

Figured this out Fixed (r32899).
EDIT: Hmm, no cards actually use PlayerTurn$ You, so maybe a different thing is actually wrong, will try to update the fix, sec...
EDIT 2: Most likely r32901 is how it was intended to be (if there is a Defined parameter, look in it for who should be the ability activator, if not (e.g. PlayerTurn$ True and no Defined), assume Defined == You. This fixes Gisa and Geralf, but I'm not completely sure if this was the intended way for it to work.

- Agetian

Hm that PlayerTurn$ is probably for something different. I think you look for "Condition$ PlayerTurn"

Hanmac wrote:Hm that PlayerTurn$ is probably for something different. I think you look for "Condition$ PlayerTurn"

Hmm, I'm not sure about Condition... The line around StaticAbility.java:520 is making a call to look for defined players (usually the Defined parameter is passed there, which, if absent, returns "You"), but for some reason it's passing in PlayerTurn instead of Defined there, which results in "True" being processed incorrectly as a definition of a player.

Btw, it looks like the card can be implemented via Condition$ PlayerTurn as well, I don't know if either implementation is preferable though.

Looks like for Gisa and Geralf there will be no difference actually, but the actual difference (for potentially other cards) is that Condition$ PlayerTurn always assumes that it has to be *your* turn, while PlayerTurn$ True with a Defined parameter allows to specify someone else (e.g. "during opponent's turn only" or whatever).

EDIT: Grr, not sure actually There are only two cards in Forge that use PlayerTurn$ True on a static ability - Gisa and Geralf and Karador, Ghost Chieftain. I'll update both to use Condition$ PlayerTurn instead, which is what everything else seems to be using anyway, and I'll revert my change to StaticAbility for now since God only knows what that can affect (still don't understand why it's looking for the defined players in PlayerTurn though, I don't think that's ever true unless someone actually implements a card (ab)using this feature in the future) :/

- Agetian

@Agetian hm yeah but now with your change "Gisa and Geralf" might crash because the Defined parameter is missing.

So that change with "PlayerTurn$ True" might be wrong and the cards should use "Condition$ PlayerTurn" instead.

Hanmac wrote:@Agetian hm yeah but now with your change "Gisa and Geralf" might crash because the Defined parameter is missing.

So that change with "PlayerTurn$ True" might be wrong and the cards should use "Condition$ PlayerTurn" instead.

The getDefinedPlayers function is actually testing if Defined is null and then assumes that it defaults to "You" in that case, so it shouldn't crash, but anyways, I updated it to use Condition$ PlayerTurn instead and for now let the PlayerTurn parameter operate in its own weird way (apparently it's possible to code things such that PlayerTurn points to an opponent or something), I guess let it be that way for now since if it's changed to look in Defined, then it may potentially interfere with other stuff that may already use Defined for other purposes, perhaps...

- Agetian

Gisa and Geralf is confirmed casting from GY again, no crash! Thanks for such a quick reply!