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

[TD+RA] MISSION_AMBUSH is busted #824

Closed
ChthonVII opened this issue Jul 29, 2022 · 0 comments · Fixed by #1008
Closed

[TD+RA] MISSION_AMBUSH is busted #824

ChthonVII opened this issue Jul 29, 2022 · 0 comments · Fixed by #1008

Comments

@ChthonVII
Copy link
Contributor

Due to a capitalization error in TechnoClass::Revealed(), revealed units check whether the house discovering them is AI-controlled when they should instead be checking whether the house that owns them is AI-controlled. Hence MISSION_AMBUSH does nothing.

Here's the fix for RA. The fix for TD is identical.

All credit to screamingchicken for this one.

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 a pull request may close this issue.

1 participant