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

[RA}Evacuation logic enhancements and MP fix #453

Merged
merged 3 commits into from
Jun 19, 2024

Conversation

mvdhout1992
Copy link

@mvdhout1992 mvdhout1992 commented Jan 14, 2021

The game contains evacuation logic where certain units are hard-coded to evacuate when loaded into a Chinook.

this PR changes:
-Evacuation is now disabled by default for MP.
-You can now disable evacuation logic completely via the map file [BASIC]DisableEvac=true/false keyword. This is useful for single player
-You can enable MP evacuation completely via the map file [BASIC]EvacinMP=true/false keyword. This will be useful when coop mode is added.

In the future we should add a per-infantry unit option that can be modified in rules files and map file.

@OmniBlade OmniBlade force-pushed the vanilla branch 3 times, most recently from 67febe4 to e6cd02d Compare April 1, 2021 12:32
@OmniBlade
Copy link
Contributor

Having two keywords for the same thing doesn't make sense to me, just have it default differently for SP and MP and use the single key for override in both.

@OmniBlade
Copy link
Contributor

Fixes #254

@OmniBlade
Copy link
Contributor

@mvdhout1992 I've pushed an additional commit that switches naming from DisableEvac to EnableEvac and sets the default value based on game mode in Clear_Scenario if you want to check if that covers most use cases? I've also rebased the branch.

Allows globally enabling or disabling evac logic on per map basis.
Default behaviour is enabled in SP disabled in MP if key is absent.
@OmniBlade OmniBlade merged commit 5c42ac9 into TheAssemblyArmada:vanilla Jun 19, 2024
17 checks passed
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.

GNRL and Einstein are evacuated in multiplayer when entering air transport
2 participants