It is currently 26 Aug 2025, 13:51
   
Text Size

Bug Reports (snapshot builds)

Post MTG Forge Related Programming Questions Here

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

Re: Bug Reports (snapshot builds)

Postby stormcat » 01 Jun 2016, 04:42

In Eternal Masters, Xantid Swarm's Rarity is wrong.
Not Common, it's Rare.
stormcat
 
Posts: 361
Joined: 17 Jun 2015, 05:32
Has thanked: 0 time
Been thanked: 6 times

Re: Bug Reports (snapshot builds)

Postby stormcat » 01 Jun 2016, 06:47

:r31258
I found another mistake in Eternal Masters.

Ticking Gnomes R→U
Yavimaya Enchantress R→C
Sensei's Divining Top U→R
Winter Orb U→R
Prismatic Lens C→U
stormcat
 
Posts: 361
Joined: 17 Jun 2015, 05:32
Has thanked: 0 time
Been thanked: 6 times

Re: Bug Reports (snapshot builds)

Postby Hanmac » 01 Jun 2016, 08:47

@stormcat: thanks for pointing that out for me, i had some typos while i did type it from hand.

===
r31259:

Problem with SplitCards:
1) if i cast them with Fuse: it does only show the color of the left card but it should show the combined. so while on the stack i cant see from the color if the spell is red or not.

2) while in Deck Editor Splitcards are shown with the Transform Swirl, while in the game they do not. Imo they should either don't show the swirl for them or the first page should be the left one and not the combined. (and even if they show the comined one, they show to have the wrong color)

3) Interaction with Harness the Storm: i can only use it when i cast the spell with Fuse. that has something to do with sharesNameWith, it should look at both Sides. Also when i cast a Spell with Fuse with Harness the Storm, i can select to cast it with Fuse what should not work.
(i asked a Judge about if Harness the Storm does work with split cards)
Magic Judge:
magichanmac asked: Question about interaction of Harness the Storm with Split Cards. For example with Alive // Well one in my hand, the other in Graveyard. Whatever i cast from my hand does it have consequences what i can cast from my graveyard? or can only cast something when i cast the spell with Fuse? (because otherwise it doesn't share a name with the one in the stack?) Also i know what i cant cast Fuse spells from my Graveyard with Fuse.
You’ll be able to cast either half of Alive // Well from your graveyard with Harness the Storm’s trigger no matter which side(s) of it you cast from your hand.

For example, you can cast Well, then use Harness’s trigger to cast Alive from your graveyard.
i have a fix for sharesNameWith but not commited yet.

4) Other cost done with Harness the Storm like Surge and Awaken should work, but currently they do not.

===
Cards with Alternate State like Split cards:
Naming a Card for something Council of the Absolute does not work correclty for cards with Alternate State.
Alive // Well does appear twice, but when i select it the chosen card is "Alive // Well", but the ruling says:
4/15/2013: You can name either half of a split card, but not both. If you name half of a split card, your opponents can cast the other half. If that split card has fuse, your opponents can't cast that card as a fused split card.
means i should only be able to choice Alive or Well, not the combined one.
also should be able to cast the other half of the named card. (if opponent does control it)

i don't know how Alhammarret, High Arbiter works with Split cards. i think i also should only be able to name one half of a split card. And my Opponent should be able to cast the other half.

This might be a problem from chooseCardName and later the magic Db with getMagicDb().getCommonCards().getUniqueCards();
Last edited by Hanmac on 01 Jun 2016, 14:14, edited 2 times in total.
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby Marek14 » 01 Jun 2016, 10:11

Another bug with Psychogenic Probe: it's not triggered by Mindblaze, for some reason.

Stealing opponent's Dragonlord Silumgar should cause whatever permanent IT stole to be returned.
Marek14
Tester
 
Posts: 2773
Joined: 07 Jun 2008, 07:54
Has thanked: 0 time
Been thanked: 303 times

Re: Bug Reports (snapshot builds)

Postby friarsol » 01 Jun 2016, 12:21

Marek14 wrote:Another bug with Psychogenic Probe: it's not triggered by Mindblaze, for some reason.
As far as I can tell, Mindblaze never actually shuffles a library (even though it should).
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 Hanmac » 01 Jun 2016, 13:16

friarsol wrote:
Marek14 wrote:Another bug with Psychogenic Probe: it's not triggered by Mindblaze, for some reason.
As far as I can tell, Mindblaze never actually shuffles a library (even though it should).
yeah i was surprised too and i added it.

Sad that my problems with SplitCards are not that easy to fix.
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby Hanmac » 01 Jun 2016, 16:31

Found this combo: Neck Breaker + Stonebrow, Krosan Hero

Magic Judge:
goat-shield asked: In my Stonebrow, Krosan Hero EDH deck, I'm wondering what the interaction with Neck Breaker would be... Are both abilities triggers that I can stack to my benefit?
Neck Breaker’s ability isn’t a triggered ability. It’s a static ability that applies to all attacking creatures. A creature without trample that attacks will gain trample from Neck Breaker in just enough time for Stonebrow to be able to trigger and give it an extra +2/+2.

(I had to go to @tabakrules for clarification on this one, as the rules as written aren’t clear here.)
Forge currently doesn't support that. (it doesn't trigger for Neck Breaker)


===

Edit:

Magic Judge:
snowskeleton asked: Back on the split card thing, what is their CMC when not on the stack? For example, how do they interact with Seasons Past?
A split card not on the stack has two sets of characteristics, so it has two different mana costs and the associated converted mana costs.

For Seasons Past, if you choose to return a split card, it takes the slots for both of its CMCs. So if you returned Armed // Dangerous, then you couldn’t return any other cards with CMCs of 2 or 4 to your hand with that Seasons Past.
i implemented "sharesCMCWith" for Seasons Past, you guys might look at it or if it breaks something.

Edit2:
Magic Judge:
docmannyonly asked: If my Fleshbag Marauder is exiled by SilkWrap, and I have no other creatures on play, If I cast Compelling Deterrence Targeting silk wrap, would my opponent discard?
Yes.

You get back Fleshbag Marauder immediately when Silkwrap leaves the battlefield. Fleshbag’s sacrifice trigger triggers when it returns, but that trigger doesn’t even go on the stack until after Compelling Deterrence finishes resolving. You control a zombie when that check is made and your opponent will discard a card.
Forge currently doesn't support that. similar to the first thing might have something to do with the trigger/checks :/

seems Forge have some problems with "just in time" things ...
Last edited by Hanmac on 02 Jun 2016, 04:37, edited 1 time in total.
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby Marek14 » 01 Jun 2016, 23:28

Freyalise's Winds / Seedborn Muse interaction is broken. Seems that Freyalise's Winds replaces any untap during untap step with removal of wind counters, but it should happen ONLY in its controller's untap step -- if the permanent untaps in another player's untap step, it should untap as normal.
Marek14
Tester
 
Posts: 2773
Joined: 07 Jun 2008, 07:54
Has thanked: 0 time
Been thanked: 303 times

Re: Bug Reports (snapshot builds)

Postby Hanmac » 02 Jun 2016, 06:40

Marek14 wrote:Freyalise's Winds / Seedborn Muse interaction is broken. Seems that Freyalise's Winds replaces any untap during untap step with removal of wind counters, but it should happen ONLY in its controller's untap step -- if the permanent untaps in another player's untap step, it should untap as normal.
fixed in r31263 with adding the Controller to the is Phase check.
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby Marek14 » 02 Jun 2016, 07:51

A bug that has persisted for a long time: when AI uses the first ability of Jace, the Mind Sculptor, I have no way to see whether it put the card on top or bottom of library.
Marek14
Tester
 
Posts: 2773
Joined: 07 Jun 2008, 07:54
Has thanked: 0 time
Been thanked: 303 times

Re: Bug Reports (snapshot builds)

Postby Hanmac » 02 Jun 2016, 10:40

Copy something with Madness cost:

r31264: casting Avacyn's Judgment for its madness cost, copy it with Pyromancer's Goggles should make a copy which does have copied Madness and X too. but currently it does not.

same true for Djinn Illuminatus and its Replicate.

i try to add CardFactory#copyCard this:
Code: Select all
out.setMadness(in.isMadness());
but when i do Pyromancer's Goggles works, but Replicate from Djinn Illuminatus does fizzle ...

After some Investigate, the problem is that getXManaCostPaid of that Madness X spells. when i copy the spell with Pyromancer's Goggles it works, but with Replicate it did not.

Debug output did show me that getXManaCostPaid is not set yet when the spell is copied by/with Replicate, but is later when copied with Googles.

So it does look like that Replicate somehow does get done before the Mana is set correct ... i need to investigate more.

===
Also i noticed when i does press [cancel] for the Relicate it does UNTAP the Googles ... happens sometimes for other spells too.
Last edited by Hanmac on 02 Jun 2016, 21:19, edited 1 time in total.
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby Marek14 » 02 Jun 2016, 18:21

Spells that distribute damage (like Violent Eruption), don't seem to show the actual damage distribution anywhere when they are on stack, only the targets.
The ability of Mirror-Sigil Sergeant should be optional.
Strange bug happened when AI discarded Gibbering Descent to Kindle the Carnage; no damage was dealt and when AI didn't cast Gibbering Descent for madness cost, it stayed in exile permanently. Maybe it was caused by the fact that it was random discard?
Marek14
Tester
 
Posts: 2773
Joined: 07 Jun 2008, 07:54
Has thanked: 0 time
Been thanked: 303 times

Re: Bug Reports (snapshot builds)

Postby Hanmac » 03 Jun 2016, 11:26

Marek14: fixed Mirror-Sigil Sergeant, the other things might be a bit more complicated. for the AI and madness thing, yeah it seems the AI does use the cards wrong and thats why the cards stay in exile.

===

hm i found "This Exalted handler should be converted to script" (some other things or TODOs might want to be scripted too) and i did try this in CardFactoryUtil:

Code: Select all
// Exalted
final int exalted = card.getAmountOfKeyword("Exalted");
//card.removeIntrinsicKeyword("Exalted");
final StringBuilder trigExalted = new StringBuilder(
        "Mode$ Attacks | ValidCard$ Creature.YouCtrl | Alone$ True | "
                + "Execute$ ExaltedPump | TriggerZones$ Battlefield | TriggerDescription$ "
                + "Exalted (Whenever a creature you control attacks alone, that creature "
                + "gets +1/+1 until end of turn.)");

final String abStringExalted = "DB$ Pump | Defined$ TriggeredAttacker | NumAtt$ +1 | NumDef$ +1";
card.getCurrentState().setSVar("ExaltedPump", abStringExalted);
final Trigger exaltedTrigger = TriggerHandler.parseTrigger(trigExalted.toString(), card, true);
for (int i = 0; i < exalted; i++) {
    card.getCurrentState().addTrigger(exaltedTrigger);
} // Exalted
it does work a bit, but not with Sublime Archangel.

i know this is not crittical, but for other or similar things which does add special Keywords to other cards we might have a common place so we don't need to implement everything twice.

hm i might implement it like Bushido, but thats not much different than it is currently.
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

Re: Bug Reports (snapshot builds)

Postby Marek14 » 03 Jun 2016, 12:51

Suggestion: When a deck that includes Vanguard Avatar is selected in constructed (a few of gos's decks do), Forge should notify you and offer to switch to Vanguard mode.
Marek14
Tester
 
Posts: 2773
Joined: 07 Jun 2008, 07:54
Has thanked: 0 time
Been thanked: 303 times

Re: Bug Reports (snapshot builds)

Postby Hanmac » 04 Jun 2016, 08:16

something is funky with Autopayment with Mana Bloom.
if i pay with auto, it does remove more than one counter from it.

AI does cast Lavaball Trap for the Trap cost even i didn't play lands this turn.
Hanmac
 
Posts: 954
Joined: 06 May 2013, 18:44
Has thanked: 229 times
Been thanked: 158 times

PreviousNext

Return to Developer's Corner

Who is online

Users browsing this forum: No registered users and 41 guests

Main Menu

User Menu

Our Partners


Who is online

In total there are 41 users online :: 0 registered, 0 hidden and 41 guests (based on users active over the past 10 minutes)
Most users ever online was 7303 on 15 Jul 2025, 20:46

Users browsing this forum: No registered users and 41 guests

Login Form