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

Implement CR 800.4m #189

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

tool4ever
Copy link
Contributor

@tool4ever tool4ever commented May 1, 2022

Example:

Oppononent uses Azorius Arrester trigger on your creature, then you kill him after. Now your creature would have been detained till game end

@tool4ever tool4ever added the Rules compliance Bringing the engine or cards closer to CR label May 1, 2022
@Hanmac
Copy link
Contributor

Hanmac commented May 2, 2022

i think it would be better to make a new Game Zone for Effects that stays even if the player has lost the game

@tool4ever
Copy link
Contributor Author

I can try it like that but I think we'll still need this part for stuff that is not done with effect cards (currently those all get exiled onPlayerLost anyway)

@Hanmac
Copy link
Contributor

Hanmac commented May 2, 2022

it might work for now, but some effects needs to continue after the player left the game

and for such effects i want an extra global Zone, that may not pollute the GUI

Player p = game.getRegisteredPlayers().get(oldPlayerIdx);
if (p.hasLost()) {
// CR 702.26n
Untap.doPhasing(p);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this phase out creatures with Phasing?

Copy link
Contributor Author

@tool4ever tool4ever Feb 8, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The function could do that for the AP, but since it's called for a player who's no longer around nothing will be there

As per the rule it will phase in those "phased out under the control of a player who has left the game [...] after that player's next turn would have begun"

To set up such a scenario you could:

  • steal the creature with a temporary control change effect
  • phase it out
  • proceed to lose
  • the control effect ends, the creature changes to the previous controller's battlefield (while still phased out)

I have to note the timing is slightly off because it should phase in with all the others but the cases where that matters will be even rarer
So for now I feel more comfortable with keeping this rather close together

@Hanmac
Copy link
Contributor

Hanmac commented Feb 8, 2023

i think i want to refactor the Detain effect anyway later, but for this i want to refactor the "doesn't untap" part first

@tool4ever
Copy link
Contributor Author

Well that Detain was just an example - there's lots of different effects that could get stuck in limbo like Shapesharer cloning making a mess

I think we can just take the plunge on this PR as it's a step in the right direction...?

@github-actions
Copy link

This PR has not been updated in a while nad has been marked on stale. Stale PRs will be auto closed

@Hanmac Hanmac added keep no stale and removed no-pr-activity labels Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keep no stale Rules compliance Bringing the engine or cards closer to CR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants