It is currently 11 Nov 2025, 18:17
   
Text Size

Bug Reports (snapshot builds)

Post MTG Forge Related Programming Questions Here

Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, CCGHQ Admins

Re: Bug Reports (snapshot builds)

Postby Agetian » 26 Sep 2015, 10:35

Re: Crackleburr + Smokebraider - I can't reproduce this, everything I tried seems to have correctly asked me to tap 2 creatures (after using Smokebraider's mana ability) :/

Nightly SVN builds used to be pushed here -> http://krazyweb.net/forge/ but something's currently wrong with it because no new builds went up since September 5, so right now I think the only way to get a fresh nightly is to compile it from source... I may be wrong though.

- Agetian
Agetian
Programmer
 
Posts: 3490
Joined: 14 Mar 2011, 05:58
Has thanked: 684 times
Been thanked: 572 times

Re: Bug Reports (snapshot builds)

Postby Agetian » 26 Sep 2015, 10:39

Hanmac wrote:hm for testing with transformed cards, specially the ones that does return to the battlefield Unhallowed Cathar does have a white border right in the cardframe, so does it have white color or is something else wrong?
Pretty sure it's a visual bug :/ I mean, I tried casting Terror on Unhallowed Cathar and was correctly not allowed to, meaning that the card is indeed black despite appearing with a white border. Must be another case of a forgotten view update or something, someone more UI-savvy should be able to figure out where to look for a potential solution for this issue...

- Agetian
Agetian
Programmer
 
Posts: 3490
Joined: 14 Mar 2011, 05:58
Has thanked: 684 times
Been thanked: 572 times

Re: Bug Reports (snapshot builds)

Postby Hanmac » 26 Sep 2015, 11:05

about Crackleburr + Smokebraider i need to test it again ...
probably with your recent fixes it might be fixed too

there are also packages on http://www.cardforge.link/snapshots/for ... i-desktop/ but they are also not so much newer
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby Sloth » 26 Sep 2015, 11:12

Agetian wrote:Oh, makes sense, thanks for clarifying it! Yes, that might be a good idea indeed, though I'm not quite sure if I'm up for the task at the moment, especially considering that my scripting skills are still rather limited...
I will try to take care of Echo. Its current clunky implementation has bothered me for ages.
User avatar
Sloth
Programmer
 
Posts: 3498
Joined: 23 Jun 2009, 19:40
Has thanked: 125 times
Been thanked: 507 times

Re: Bug Reports (snapshot builds)

Postby Agetian » 26 Sep 2015, 14:50

Hanmac wrote:hm for testing with transformed cards, specially the ones that does return to the battlefield Unhallowed Cathar does have a white border right in the cardframe, so does it have white color or is something else wrong?
Fixed (r30123).

- Agetian
Agetian
Programmer
 
Posts: 3490
Joined: 14 Mar 2011, 05:58
Has thanked: 684 times
Been thanked: 572 times

Re: Bug Reports (snapshot builds)

Postby friarsol » 27 Sep 2015, 03:47

r20143
Echo seems to be triggering every upkeep for AIs Mogg War Marshall
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Bug Reports (snapshot builds)

Postby Agetian » 27 Sep 2015, 05:57

friarsol wrote:r20143
Echo seems to be triggering every upkeep for AIs Mogg War Marshall
There are actually several problems with the new Echo... One is that it's triggering recurringly, actually not only for the AI but for the human player too, but this is easily fixed:

PhaseHandler.java:422
Code: Select all
     if (playerTurn.equals(c.getController())) {
          c.setCameUnderControlSinceLastUpkeep(false); // this method is then best renamed setCameUnder... to avoid confusion
     }
The bigger problem, however, is that it doesn't interact correctly with stuff, most notably with the change of control. The old Echo used to correctly fire if e.g. the opponent used Control Magic on an Echo creature. The new one is treated pretty much like a one-time ETB effect and thus it fires only once when the creature enters the battlefield, but nothing happens if the control is changed (which it should). I tried fixing this bigger problem but couldn't get my head around it, nothing I tried has worked... :(

Because we have an imminent release (possibly today?) I'll have to revert 30117:30118 for now (the old Echo is implemented in an obsolete way but seems to work in a rule-correct way...). Please feel free to revert my revert (r30144) and update the new Echo system so that it interacts correctly with controller change and possibly other things (am I missing anything important for Echo interaction?...) when the time permits.

- Agetian
Agetian
Programmer
 
Posts: 3490
Joined: 14 Mar 2011, 05:58
Has thanked: 684 times
Been thanked: 572 times

Re: Bug Reports (snapshot builds)

Postby Hanmac » 28 Sep 2015, 20:00

the second ability of Jace, Telepath Unbound does allow spells that got exiled by it to be cast again ... i dont think that should be possible (might have been fixed with recent changes ...)
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby sharkus » 29 Sep 2015, 03:26

R30161:

When Rush of Ice is cast for it's awaken cost the opponent's creature untaps during the next untap step. When it is cast without awaken it works properly.
sharkus
 
Posts: 14
Joined: 19 Sep 2014, 18:18
Has thanked: 0 time
Been thanked: 3 times

Re: Bug Reports (snapshot builds)

Postby Sloth » 29 Sep 2015, 07:17

sharkus wrote:When Rush of Ice is cast for it's awaken cost the opponent's creature untaps during the next untap step. When it is cast without awaken it works properly.
Fixed! Thanks sharkus.
User avatar
Sloth
Programmer
 
Posts: 3498
Joined: 23 Jun 2009, 19:40
Has thanked: 125 times
Been thanked: 507 times

Re: Bug Reports (snapshot builds)

Postby drdev » 01 Oct 2015, 13:47

This was posted in the Android app thread, but I wanted to post it here so it can get fixed before tomorrow if possible, as it seems like a potentially serious issue with how Devoid works. We should make sure Devoid cards are both considered colorless as well as not colored.

ABrighterShadow wrote:Bug report: All Is Dust destroys Herald of Kozilek. This shouldn't be happening, as Herald of Kozilek has Devoid (This card has no color.)
drdev
Programmer
 
Posts: 1958
Joined: 27 Jul 2013, 02:07
Has thanked: 189 times
Been thanked: 565 times

Re: Bug Reports (snapshot builds)

Postby friarsol » 01 Oct 2015, 14:19

drdev wrote:This was posted in the Android app thread, but I wanted to post it here so it can get fixed before tomorrow if possible, as it seems like a potentially serious issue with how Devoid works. We should make sure Devoid cards are both considered colorless as well as not colored.

ABrighterShadow wrote:Bug report: All Is Dust destroys Herald of Kozilek. This shouldn't be happening, as Herald of Kozilek has Devoid (This card has no color.)
I already fixed it. That card was just missing the Devoid keyword.
friarsol
Global Moderator
 
Posts: 7593
Joined: 15 May 2010, 04:20
Has thanked: 243 times
Been thanked: 965 times

Re: Bug Reports (snapshot builds)

Postby matrix4767 » 03 Oct 2015, 13:49

r30197:
Some (maybe all) cards that make an Eldrazi Scion token use the same picture, instead of picking from one of three.
matrix4767
 
Posts: 125
Joined: 09 Sep 2014, 08:10
Has thanked: 5 times
Been thanked: 3 times

Re: Bug Reports (snapshot builds)

Postby Agetian » 03 Oct 2015, 16:43

matrix4767 wrote:r30197:
Some (maybe all) cards that make an Eldrazi Scion token use the same picture, instead of picking from one of three.
Fixed (r30202).

- Agetian
Agetian
Programmer
 
Posts: 3490
Joined: 14 Mar 2011, 05:58
Has thanked: 684 times
Been thanked: 572 times

Re: Bug Reports (snapshot builds)

Postby linXzta » 03 Oct 2015, 22:38

Description: I was trying to play constructed when I hit start this comes on

NullPointerException | Open
Code: Select all
Forge Version:    1.5.46-r-1
Operating System: Windows 8 6.2 amd64
Java Version:     1.7.0_45 Oracle Corporation

java.lang.NullPointerException
   at forge.screens.match.VMatchUI.populate(VMatchUI.java:119)
   at forge.screens.match.CMatchUI.updatePlayerControl(CMatchUI.java:613)
   at forge.screens.match.CMatchUI.initialize(CMatchUI.java:483)
   at forge.control.FControl.setCurrentScreen(FControl.java:323)
   at forge.control.FControl.setCurrentScreen(FControl.java:287)
   at forge.screens.match.CMatchUI.openView(CMatchUI.java:819)
   at forge.match.HostedMatch.startGame(HostedMatch.java:183)
   at forge.match.HostedMatch.startMatch(HostedMatch.java:115)
   at forge.match.HostedMatch.startMatch(HostedMatch.java:83)
   at forge.match.HostedMatch.startMatch(HostedMatch.java:77)
   at forge.limited.GauntletMini.startRound(GauntletMini.java:144)
   at forge.limited.GauntletMini.launch(GauntletMini.java:131)
   at forge.screens.home.sanctioned.CSubmenuSealed.startGame(CSubmenuSealed.java:118)
   at forge.screens.home.sanctioned.CSubmenuSealed.access$100(CSubmenuSealed.java:34)
   at forge.screens.home.sanctioned.CSubmenuSealed$3.actionPerformed(CSubmenuSealed.java:68)
   at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
   at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
   at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
   at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
   at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
   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$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.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)
linXzta
 
Posts: 4
Joined: 08 Mar 2015, 20:26
Has thanked: 0 time
Been thanked: 0 time

PreviousNext

Return to Developer's Corner

Who is online

Users browsing this forum: No registered users and 10 guests

Main Menu

User Menu

Our Partners


Who is online

In total there are 10 users online :: 0 registered, 0 hidden and 10 guests (based on users active over the past 10 minutes)
Most users ever online was 9824 on 10 Nov 2025, 04:33

Users browsing this forum: No registered users and 10 guests

Login Form