forge icon indicating copy to clipboard operation
forge copied to clipboard

Cryptic Spires does not work properly

Open Goddfrey opened this issue 1 year ago • 10 comments

Describe the bug Cryptic Spires says "As you create your deck, circle two of the colors below" (out of 5 basic mana colors). In game you can tap it to add one mana of either of the circled colors.

How it's implemented here in the quest mode is you choose the two colors before the game starts. Whatever I'm fine with it.

But the real bug is it produces two colorless when tapped.

To Reproduce Play Cryptic Spires Tap it for mana

Expected behavior Should add 1 mana of one of the chosen colors. Not two colorless.

Screenshots

cryptic-spires-bug

Desktop (please complete the following information):

  • OS: Windows 10
  • Forge 1.6.57, Java 8

Goddfrey avatar Aug 18 '23 09:08 Goddfrey

This issue has not been updated in a while and has now been marked as stale. Stale messages will be auto closed.

github-actions[bot] avatar Sep 18 '23 09:09 github-actions[bot]

@tool4ever can you look into this Problem?

Hanmac avatar Apr 12 '24 12:04 Hanmac

Can confirm, still bugged. There is a color selection at the start of the game, but it still produces 2 colorless mana instead.

pfirpfel avatar May 07 '24 14:05 pfirpfel

@pfirpfel that's because @tool4ever or others hasn't looked at the problem yet

Hanmac avatar May 07 '24 14:05 Hanmac

I tried debugging this, it looks like the information about the chosen colors is lost between the time it's actually chosen and the standard game flow begins. My guess is that this information is reset somewhere when the library is prepared or something along those lines, but I haven't been able to establish the exact cause yet. This should probably be done somewhere later in the chain of game events then?

Agetian avatar May 07 '24 15:05 Agetian

Actually, I'm not sure why this is happening, since the next thing after doing the Spires color choice routine is drawing the hand, and that doesn't seem to run anything that would reset the chosen color info :/

Agetian avatar May 07 '24 15:05 Agetian

By the time GameAction calls changeZone on the Spires when you play it as a land, the game tries to grab the last known info via getLKICopy, and the source in there already has no information on the chosen colors (the relevant array is empty).

Agetian avatar May 07 '24 15:05 Agetian

This card was never implemented in a way that actually works... I might look into it if Alchemy releases a clone effect that's perpetual since that could then be used to simulate this too :D (the colors need to be copiable values)

tool4ever avatar May 07 '24 15:05 tool4ever

I had the thought that there might actually be 10 different card scripts... each instance gets locked in when drafting / constructing decks... but got a little stuck on bringing the theory to completion

This could lead to implementing Attractions in Unfinity, too

Perpetual stuff might be lost between matches, though that is an interesting tactic too

Northmoc avatar May 07 '24 15:05 Northmoc

Ah, makes sense! :+1:

Agetian avatar May 07 '24 17:05 Agetian