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

[BUG] Unable to build infantry #92

Closed
Libeccio-DD opened this issue Jan 26, 2020 · 0 comments · Fixed by #94
Closed

[BUG] Unable to build infantry #92

Libeccio-DD opened this issue Jan 26, 2020 · 0 comments · Fixed by #94
Assignees
Labels
bug Something isn't working

Comments

@Libeccio-DD
Copy link
Contributor

System Information:

  • Mod Version v0.4.6 - Hotfix 1
  • Mod Checksum DICN
  • Vanilla Checksum WNVU

Describe the bug
Every time an uncivilised nation westernises, they become unable to build infantry. This can be fixed by reloading the game. However since this bug affects the AI as well, if the player doesn't reload their game it can result in westernised AI nations like Japan unable to build proper armies.

To Reproduce
Westernise as any uncivilised nation or wait until an unciv westernises and then tag switch to them. Open the military tab and you'll see that you can't build infantry. Reload the game to fix the bug.

Expected behavior
Westernised nations should be able to build infantry.

Additional context
It looks like the cause of this bug is that infantry is now activated through the flintlock_rifle_armament invention instead of the flintlock_rifles technology like it was done in the past. Moving activate_unit = infantry back to flintlock_rifles solves this problem however I understand this change happened in order to fix a vanilla bug.

@Libeccio-DD Libeccio-DD added the bug Something isn't working label Jan 26, 2020
arkhometha added a commit that referenced this issue Jan 26, 2020
* Separated Finnish and Sami from Scandinavian culture group and Ugric from the East Slavic and joined them in the Finno-Ugric culture group. Adapted the Russian decision to integrate Finland to include Scandinavia as one of the potential countries that can integrate Finland. Closes #87
* Small localization fixes.
* Slight increased the effects of the Dissolving the upper house decisions.
* Made the Propaganda decision money requirement scale with population.
* Removed duplicate decisions to release Galicia-Lodomeria for AI Austria. Improved the Bukovina decision that does the same.
* Fix uncivs not being able to build infantry after westernization due to a fix for the vanilla bug that locks created ships in ports. Closes #92
* Adapted one of the Muslim decisions to apply the jizya tax modifier if the country is missing the modifier. Fixed the decision so it only applies to Muslims. Closes #91
* Changed to the UK Liberal party by @uk Liberal Party by @Libeccio-DD  - #85
* Added missing localization to reactionary rebels. By @Libeccio-DD - #90
* Made propaganda costs scale with population.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants