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

Neutralizing Gas and Extreme Weather abilities ending after switching out #10812

Closed

Conversation

andrebastosdias
Copy link
Contributor

@andrebastosdias andrebastosdias commented Jan 9, 2025

@DaWoblefet :

When Neutralizing Gas leaves the field, the user is still relatively "active" on the field during the pending switchout (e.g. it can have its Attack dropped via Intimidate, its item revealed via Frisk, or its HP restored via Hospitality, even though it's not "on the field" anymore). So it's a bug in the same way that a Pokemon using Volt Switch shouldn't be "on the field" when choosing a target to be replaced.

Added the PostEnd event. The client should include the -switchout command (purely for aesthetic purposes).

Still missing the Volt Switch part.

@DaWoblefet
Copy link
Member

DaWoblefet commented Jan 10, 2025

This does not work. The Neutralizing Gas Pokemon is currently not receiving Intimidate drops or getting healed from Hospitality, as examples. You can add Hospitality to Eternatus and damage Weezing before switching to see that for yourself. That behavior is correct on master currently.

Assuming that part gets worked out, we would need to merge a client PR to handle the new switchout protocol message, or else all battles will have an error message printed indicating no support.
image

@andrebastosdias andrebastosdias marked this pull request as draft January 10, 2025 05:32
@andrebastosdias
Copy link
Contributor Author

andrebastosdias commented Jan 10, 2025

On the master branch, the Neutralizing Gas Pokémon is being affected by Intimidate before switching out, which shouldn't happen.

Let's set aside the fact that the Pokémon is still visually present on the field. In practice, the only issue with the current logic is that the Neutralizing Gas Pokémon is being influenced by abilities. If we simply bypass this and allow the other Pokémon to be affected by the abilities' effects, it should work.

Edit. the real problem, it is that we never let an active slot be null when it switches out or faints. We just bypass it by looking at its hp.

@andrebastosdias andrebastosdias marked this pull request as ready for review January 10, 2025 06:32
@DaWoblefet
Copy link
Member

DaWoblefet commented Jan 10, 2025

On the master branch, the Neutralizing Gas Pokémon is being affected by Intimidate before switching out, which shouldn't happen.

No, you've got it backwards - that is what should happen. The Weezing should have its Attack dropped by Intimidate before switching out. That particular interaction on master is correct right now.

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

Successfully merging this pull request may close these issues.

2 participants