LoneGazebo / Community-Patch-DLL

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

Vassals and Domination Victory #6900

Closed RecursiveVision closed 3 years ago

RecursiveVision commented 4 years ago

I suggest that we alter the rules for Domination Victory so that capitals owned by vassal states or allied City-States count towards Domination Victory.

This prevents situations where a player has to declare war on their own vassal or allied City-State in order to win a Domination Victory.

Not only would players probably appreciate this (Domination Victory is the hardest to achieve, and it's not fun to have to declare an easy war on an ally in order to win like this) but it'd be a lot more AI friendly.

As it is the special logic I wrote to handle the special case where the AI should attack their own vassals is ineffective, probably because of that check ilteroi mentioned preventing attack operations on vassals.

LoneGazebo commented 3 years ago

Fixed. Vassal-owned capitals now count towards Dom victory.