search

LoneGazebo / Community-Patch-DLL

Community Patch for Civilization V - Brave New World
Other
286 stars 159 forks source link

Nationalization and Vassals #7273

Closed Oscar-IronTower closed 3 years ago

Oscar-IronTower commented 3 years ago

1. Mod version (i.e Date - 12/1): 12/1 with hotfix.

2. Mod list (if using Vox Populi only, leave blank):

Gedemon's YnAEMP InfoAddict Quick Turns Really Advanced Setup UI - Improved City View (Vox Populi EUI) UI - Trade Opportunities for VP Minor tweaks to VP like allowing farms on freshwater hills, allowing both train station and seaports in a city, and modified the values of resource deposits. Playing on Dale's Earth map from YnAEMP.

3. Error description:

With Nationalization, should I be able to build franchises on vassals or not? Because it seems to imply I can, but it says it can't when selecting a trade route.

4. Steps to reproduce (optional):

In the savegame there's a trade route ready so you can check if it will create a franchise or not.

Supporting information: Please note that you can attach .zip files by dragging-and-dropping them. If possible, zip up all supporting data and post that way.

  1. Log files (always attach your Logs folder, located at My Documents/My Games/Sid Meier's Civilization 5. Make sure you have enabled logging before experiencing an error! Go here to find out how: http://forums.civfanatics.com/showthread.php?t=487482):

  2. Save game (always attach a save that was made a turn before the error; located at My Documents/My Games/Sid Meier's Civilization 5/ModdedSaves):

Nationalization Bug.zip

  1. CvMiniDump.dmp file (attach if experiencing a game crash. Located at Program Files/Steam/steamapps/common/Sid Meier's Civilization V):

  2. Screenshots (optional):

image

RecursiveVision commented 3 years ago

@LoneGazebo Which one is intended?

LoneGazebo commented 3 years ago

'except vassals' is intended.

RecursiveVision commented 3 years ago

Fixed. Bad syntax.