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 a player option to prevent Fokker Aeroplanbau from moving Germany's capital #151

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rogerburks
Copy link
Collaborator

References #150

Adds an option to event 99814 "Fokker Aeroplanbau," to avoid the effect where it moves the capital back and forth. This is done in the original event because that is the only way to place a factory at the desired location, although it does not always anticipate where to place the capital once it is done. If Germany has been formed by the player in an unusual way, such as with Baden, the player may not wish to have the capital moved to Berlin, for instance. This gives the player an option to refuse to move the capital.

I examined other code to find other events that may do this. I found some for the United Kingdom that I left alone, because London should be the only valid option for the capital to be placed at. Other cases for Germany were vanilla decisions, which probably require a different fix and arguably are a different issue.

An ai chance is added that should choose the original option every time, so that it does not miss out on its airplane factory.

… where it moves the capital back and forth. This is done in the original event because that is the only way to place a factory at the desired location, although it does not always anticipate where to place the capital once it is done.If Germany has been formed by the player in an unusual way, such as with Baden, the player may not wish to have the capital moved to Berlin, for instance. This gives the player an option to refuse to move the capital.
@rogerburks rogerburks self-assigned this Feb 20, 2022
@rogerburks rogerburks added the enhancement New feature or request label Feb 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Events that move the capital in formables need an alternative option
1 participant