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

Add MarkPlotUsed call for missions that don't use the dropship #1207

Merged
merged 2 commits into from
Aug 10, 2023

Conversation

Tedster59
Copy link
Contributor

Fixes issue #1082

Since nothing uses Seamless traveling anymore (per a discord message from robojumper), Missions that don't involve a dropship travel animation don't mark plots as used, leading to all UFOs and Chosen Avenger Defenses for a campaign being on the same plot. This PR moves the MarkPlotsUsed code outside of the bShowDropshipInteriorWhileGeneratingMap check so that all missions have plots marked as used properly.

@Tedster59 Tedster59 changed the title Add MarkPlotUsed call for missions that don't use the dropship Fix issue #1082: Add MarkPlotUsed call for missions that don't use the dropship Jun 27, 2023
@Tedster59 Tedster59 changed the title Fix issue #1082: Add MarkPlotUsed call for missions that don't use the dropship Fix Issue #1082: Add MarkPlotUsed call for missions that don't use the dropship Jun 27, 2023
@Iridar Iridar added bug-basegame ready-to-review A pull request is ready to be reviewed labels Jun 29, 2023
@Iridar Iridar added this to the 1.26.0 milestone Jun 29, 2023
@Iridar Iridar added waiting-on-author A pull request is waiting on changes from the author and removed ready-to-review A pull request is ready to be reviewed labels Jun 29, 2023
@Iridar Iridar changed the title Fix Issue #1082: Add MarkPlotUsed call for missions that don't use the dropship Add MarkPlotUsed call for missions that don't use the dropship Jun 29, 2023
@Tedster59 Tedster59 requested a review from Iridar June 29, 2023 02:57
@BlackDog86 BlackDog86 added ready-to-review A pull request is ready to be reviewed and removed waiting-on-author A pull request is waiting on changes from the author labels Jul 25, 2023
@Iridar Iridar added ready-for-merge the pull request was reviewed and is ready to be merged. and removed ready-to-review A pull request is ready to be reviewed labels Aug 10, 2023
@Iridar Iridar merged commit f4283d2 into X2CommunityCore:master Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-basegame ready-for-merge the pull request was reviewed and is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants