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

Emrakul, the Promised End's Mindslaver effect has issues passing priority #12192

Open
stevedavidmusic opened this issue Apr 28, 2024 · 6 comments

Comments

@stevedavidmusic
Copy link

stevedavidmusic commented Apr 28, 2024

The Mindslaver effect of Emrakul, the Promised End (and probably Mindlsaver in general) seems very bugged when it comes to passing priority. The only way to move forward with priority seems to be if both players constantly turn their skips on and off. We experienced this in more than one match and I just replicated it myself on my machine just to see it going on from both sides. Normally I'd post screenshots for proof, but it's not exactly going to do much in this scenario.

It'll also randomly say that you lost control of the opponent's turn, although I'm not sure if you actually do lose control of their turn.

@Susucre
Copy link
Contributor

Susucre commented Apr 28, 2024

Can confirm I've replicated the issue with [[Mindslaver]].
Not sure what is wrong and when that broke, but the "take control of player" effect still seems to have the original turn's player in control somewhat.

Copy link

Mindslaver - (Gatherer) (Scryfall) (EDHREC)

{6}
Legendary Artifact
{4}, {T}, Sacrifice Mindslaver: You control target player during that player's next turn. (You see all cards that player could see and make all decisions for the player.)

@JayDi85
Copy link
Member

JayDi85 commented Apr 28, 2024

Can be related to player skip settings (e.g. game use it from wrong player). There few places in HumanPlayer with potentially wrong usages (I added related todo comments to it some time ago).

@Susucre
Copy link
Contributor

Susucre commented Apr 28, 2024

I have looked at the PlayerImpl part and nothing felt out of place (stuff could be simplified, but the logic felt sane). But indeed that big method with a lot of TODO in HumanPlayer could have the root cause in its logic (it is long and convoluted, but so central a change on it should be carefully tested)

@alexander-novo
Copy link
Contributor

I ran into a similar issue with [[Cruel Entertainment]] the other day - iirc, what had to happen was that the controlling player had to click the "switch hands" button every time priority was passed to pass priority for themselves and the player they were controlling.

Copy link

Cruel Entertainment - (Gatherer) (Scryfall) (EDHREC)

{6}{B}
Sorcery
Choose target player and another target player. The first player controls the second player during the second player's next turn, and the second player controls the first player during the first player's next turn.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants