Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possible bug with Kozilek, the Broken Reality and Ulalek, Fused Atrocity's copy ability #5340

Open
TheLastNarwhal opened this issue May 31, 2024 · 6 comments
Assignees
Labels
bug Something isn't working keep no stale

Comments

@TheLastNarwhal
Copy link
Contributor

TheLastNarwhal commented May 31, 2024

Describe the bug
I'm not sure this is actually a bug, but it feels like it could be one, especially since Ulalek has this bit of text: You may choose new targets for the copies.

When you are copying an ability, like Kozilek, the Broken Reality, Forge isn't allowing you to pick new targets. It just keeps the same targets that you chose for the original cast trigger.

Expected behavior
To be able to choose new targets for the copied abilities (assuming that this is correct.)

Desktop (please complete the following information):

  • Forge Version: forge-gui-desktop-1.6.63-SNAPSHOT-05-30

EDIT: I was able to choose new targets when I copied Ulamog, the Defiler's trigger, so perhaps it's just Kozilek that has the issue?

@TheLastNarwhal TheLastNarwhal changed the title Possible bug with Ulalek, Fused Atrocity Possible bug with Kozilek, the Broken Reality and Ulalek, Fused Atrocity's copy ability May 31, 2024
@Hanmac Hanmac added bug Something isn't working labels May 31, 2024
@tool4ever
Copy link
Contributor

I'm not sure if it's legal to change the number of targets as part of such effects. Were you trying to do that?

@Hanmac
Copy link
Contributor

Hanmac commented May 31, 2024

@tool4ever Rules wise, you can't change the number of targets

So if you targeted 1 Player, you should be able to change the target.
If you targeted both player, then you should not be able too.

But maybe the issue is related to #5334 ?

@TheLastNarwhal
Copy link
Contributor Author

@Hanmac

So if you targeted 1 Player, you should be able to change the target.
If you targeted both player, then you should not be able too.

I was in a four player EDH game and targeted two of the players with the original effect. For the copy effect I should be able to change the targets to whoever I want as long as I target two players, right?

@Glorax
Copy link
Contributor

Glorax commented Jun 12, 2024

In my own testing, Ulalek hasn't let me change the targets on numerous effects. With Ulamog, the Ceaseless Hunger, the copy's cast trigger doesn't do anything and doesn't let me pick new targets for removal--presumably because it's trying to target the same two permanents that just got exiled and coming up with nothing.

Ulalek doesn't seem to copy Spawn-Gang Commander's cast trigger either. It only creates three tokens total, with the copy creating no tokens.

For other effects, like Hideous Taskmaster, it actually has let me pick new targets for the copy. About 50% of the time it either won't let me pick new targets, or fails to copy the cast triggers at all.

Copy link

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

@dracontes
Copy link
Contributor

Just tested the problematic cases for this issue, Kozilek, the Broken Reality and Ulamog, the Ceaseless Hunger, and they seem to be working fine now allowing me to change targets.

ulalek_targeting_1
The two target case for Ulamog

ulalek_targeting_2
The two target case for Kozilek

ulalek_targeting_3
The single target case for Kozilek

However since my experience was that the transition between prompts of copies of a triggered ability from the same source is pretty abrupt, especially the times I only chose one target for Kozilek's cast trigger, it's my impression that the current state of affairs regarding how copied abilities are handled by the game engine is prone to being confusing. Perhaps ability copies should be disambiguated with numbering. Also it would be useful to have the targets that might carry over listed on the prompt as a measure of convenience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working keep no stale
Projects
None yet
Development

No branches or pull requests

5 participants