search

LoneGazebo / Community-Patch-DLL

Community Patch for Civilization V - Brave New World
Other
285 stars 158 forks source link

Broken expansion promise after conquest #8146

Closed Chase-star closed 3 years ago

Chase-star commented 3 years ago

1. Mod version (i.e Date - 4/14): 7/1 Hotfix 3

_2. Mod list (if using Vox Populi only, leave blank): Vox Populi Trade Opportunities mixn'match (Global - City Forest Bonus, Global - City Jungle Bonus, Global - City State Airbases, Global - Local Generals, Global - Passable Forts, Improvements - Airbases) Promotion Flags EUI for VP Reforestation TopPanel Replace (EUI Only) UI - Destination UI - Filters in Trade Route Overview UI - Improved City View (Vox Populi EUI) UI - Religion Spread (Vox Populi)

These mods are compiled into a modpack though so please tell if the modpack also needs to be uploaded. 3. Error description: After conquering the capital of Songhai, Denmark shows up telling me that I am a thief for breaking the promise of NOT expanding.

4. Steps to reproduce (optional): Capture the city (I only tried using the knight) Select Annex or Puppet End turn Denmark will message you in his turn

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. For instructions, go to the repository's main page, under "To enable logging for bug reports"):

  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; you can change autosave frequency in the game's Options menu): Horde.Civ5Save.zip

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

  4. Screenshots (optional):

2021-07-13 21_46_50-Greenshot 2021-07-13 21_50_16-Greenshot 2021-07-13 21_52_19-Greenshot 2021-07-13 21_53_47-Greenshot 2021-07-13 21_54_45-Greenshot

LoneGazebo commented 3 years ago

that counts as expansion

RecursiveVision commented 3 years ago

Intended

Neborya commented 3 years ago

Just reported this case, because it seems like a 99% bug, so people are most likely going to report it again and again. If this is intentional, the promise should should say something about conquering too.

RecursiveVision commented 3 years ago

@Neborya I updated the text to make it more clear.

Neborya commented 3 years ago

At least it would be clear, thanks!

I'd argue, that gaining city through conquering shouldn't be considered expansion in all cases (at least because you cannot raze the conquered city instantly to keep the promise), but maybe a forum thread would suit better for this discussion.