Advanced Diplomacy 2

stolenrays · Mar 13, 2012

@Orion. I'm not sure about the diplomacy options. Sorry about the non-reply, I've been so busy at school. I'm glad to hear it has no bugs. I always try and release them without any bugs/crashes.

OrionVeteran · Mar 14, 2012

stolenrays said:
@Orion. I'm not sure about the diplomacy options.

Testing shows that you can conduct trades for tech, resources and cities, without first establishing an embassy. This is very good, as the historian would have had a major problem with the mod, if an embassy was required to trade for those items. The mod is realistic and therefore, will make a fine addition to the next version of OGI.

Thanks for your hard work in expanding this mod.

stolenrays · Mar 14, 2012

Glad to hear and thanks for the feedback. There might be some future additions as well.

Roamty · Mar 14, 2012

ripple01 said:
Also, stolenrays, this is really being anal, but you can also remove the CONTACT_SECRETARY_GENERAL_VOTE and CONTACT_TRADE_CORPORATION from CvEnums.h, CvInfos.cpp, CyEnumsInterfaces.cpp and the GlobalTypes.XML. A harmless assert throws up if you don't do this, because the list in CvLeaderHeadsInfos.XML doesn't match the GlobalTypes.

What I did was add in the missing CONTACT_SECRETARY_GENERAL_VOTE and CONTACT_TRADE_CORPORATION to CIV4LeaderHeadInfos


<ContactRand>
<ContactType>CONTACT_PEACE_PRESSURE</ContactType>
<iContactRand>100</iContactRand>
</ContactRand>
<ContactRand>
<ContactType>CONTACT_EMBASSY</ContactType>
<iContactRand>25</iContactRand>
</ContactRand>
<ContactRand>
<ContactType>CONTACT_TRADE_STOP_TRADING</ContactType>
<iContactRand>20</iContactRand>
</ContactRand>
<ContactRand>
<ContactType>CONTACT_TRADE_JOIN_WAR</ContactType>
<iContactRand>10</iContactRand>
</ContactRand>
<ContactRand>
<ContactType>CONTACT_SECRETARY_GENERAL_VOTE</ContactType>
<iContactRand>10</iContactRand>
</ContactRand>
<ContactRand>
<ContactType>CONTACT_TRADE_MILITARY_UNITS</ContactType>
<iContactRand>15</iContactRand>
</ContactRand>
<ContactRand>
<ContactType>CONTACT_TRADE_WORKERS</ContactType>
<iContactRand>25</iContactRand>
</ContactRand>
<ContactRand>
<ContactType>CONTACT_TRADE_CONTACTS</ContactType>
<iContactRand>25</iContactRand>
</ContactRand>
<ContactRand>
<ContactType>CONTACT_TRADE_CORPORATION</ContactType>
<iContactRand>25</iContactRand>
</ContactRand>



<ContactDelay>
<ContactType>CONTACT_PEACE_PRESSURE</ContactType>
<iContactDelay>100</iContactDelay>
</ContactDelay>
<ContactDelay>
<ContactType>CONTACT_EMBASSY</ContactType>
<iContactDelay>25</iContactDelay>
</ContactDelay>
<ContactDelay>
<ContactType>CONTACT_TRADE_STOP_TRADING</ContactType>
<iContactDelay>20</iContactDelay>
</ContactDelay>
<ContactDelay>
<ContactType>CONTACT_TRADE_JOIN_WAR</ContactType>
<iContactDelay>20</iContactDelay>
</ContactDelay>
<ContactDelay>
<ContactType>CONTACT_SECRETARY_GENERAL_VOTE</ContactType>
<iContactDelay>15</iContactDelay>
</ContactDelay>
<ContactDelay>
<ContactType>CONTACT_TRADE_MILITARY_UNITS</ContactType>
<iContactDelay>20</iContactDelay>
</ContactDelay>
<ContactDelay>
<ContactType>CONTACT_TRADE_WORKERS</ContactType>
<iContactDelay>20</iContactDelay>
</ContactDelay>
<ContactDelay>
<ContactType>CONTACT_TRADE_CONTACTS</ContactType>
<iContactDelay>15</iContactDelay>
</ContactDelay>
<ContactDelay>
<ContactType>CONTACT_TRADE_CORPORATION</ContactType>
<iContactDelay>20</iContactDelay>
</ContactDelay>



<ContactType>CONTACT_PEACE_PRESSURE</ContactType>
<ContactType>CONTACT_EMBASSY</ContactType>
<ContactType>CONTACT_TRADE_STOP_TRADING</ContactType>
<ContactType>CONTACT_TRADE_JOIN_WAR</ContactType>
<ContactType>CONTACT_SECRETARY_GENERAL_VOTE</ContactType>
<ContactType>CONTACT_TRADE_MILITARY_UNITS</ContactType>
<ContactType>CONTACT_TRADE_WORKERS</ContactType>
<ContactType>CONTACT_TRADE_CONTACTS</ContactType>
<ContactType>CONTACT_TRADE_CORPORATION</ContactType>


So now is the same as in GlobalTypes

OrionVeteran · Mar 15, 2012

Roamty said:
What I did was add in the missing CONTACT_SECRETARY_GENERAL_VOTE and CONTACT_TRADE_CORPORATION to CIV4LeaderHeadInfos

So now is the same as in GlobalTypes

Nice work Roamty! :goodjob:

stolenrays · Mar 15, 2012

Ripple could you attach those changed sdk files for me to quickly update the mod?

OrionVeteran · Mar 15, 2012

stolenrays said:
Ripple could you attach those changed sdk files for me to quickly update the mod?

Why would you want to update the SDK to remove CONTACT_SECRETARY_GENERAL_VOTE and CONTACT_TRADE_CORPORATION, when you have Roamty's fix in the XML to make it work?

stolenrays · Mar 15, 2012

ripple01 said:
Stolenrays:

Please see the following post re: some code from CvDeal that was left out of this modcomp. Afforess didn't comment it and the .dll compiled without it, so very easy to miss. I only discovered it after some extensive debugging:

http://forums.civfanatics.com/showpost.php?p=11316454&postcount=3

and thanks again for putting this together!

I was referring to this post actually.

OrionVeteran · Mar 15, 2012

stolenrays said:
I was referring to this post actually.

Yes. I read that too and would be very interested to see these SDK changes.

stolenrays · Mar 16, 2012

OK, I've changed the leaderheadinfos.xml file now (no update yet). I can't seem to find the CvDeal code strings that was mentioned here.

OrionVeteran · Mar 16, 2012

stolenrays said:
OK, I've changed the leaderheadinfos.xml file now (no update yet). I can't seem to find the CvDeal code strings that was mentioned here.

In fact, I checked all the SDK source files in Advanced Diplomacy ver 1.05 and in ver 2.0 for "removeFirstTradeNode" and could not find it anywhere. If this function is not there, I assume Afforess must have removed it. Ripple01 said the code was left out, but I don't see how it can affect game play since nothing in Advanced Diplomacy references that function. :confused:

Lastly, I checked the original Civ4 SDK in both CvDeal files and didn't find it there either. Therefore, I don't beleve we have a problem in ver 2.0.

Reading the whole post seems to confirm my findings: http://forums.civfanatics.com/showthread.php?p=11327971

ripple01 · Mar 16, 2012

Orion is right, that "missing" code was not what was causing my issue. If nothing references it, it must have just been old, left over code and is not needed.

stolenrays · Mar 16, 2012

Good news then. Version 2.1 should be released soon.

Roamty · Mar 17, 2012

CyEnumsInterface I found this and change it to the second one double

.value("TRADE_SECRETARY_GENERAL_VOTE", TRADE_SECRETARY_GENERAL_VOTE)
.value("TRADE_PLEDGE_VOTE", TRADE_PLEDGE_VOTE)
.value("TRADE_SECRETARY_GENERAL_VOTE", TRADE_SECRETARY_GENERAL_VOTE)
.value("TRADE_PLEDGE_VOTE", TRADE_PLEDGE_VOTE)

/************************************************************************************************/
/* Afforess Start 06/16/10 */
/* */
/* Advanced Diplomacy */
/************************************************************************************************/
.value("TRADE_EMBASSY", TRADE_EMBASSY)
.value("TRADE_WORKER", TRADE_WORKER)
.value("TRADE_MILITARY_UNIT", TRADE_MILITARY_UNIT)
.value("TRADE_CONTACT", TRADE_CONTACT)
.value("TRADE_CORPORATION", TRADE_CORPORATION)
.value("TRADE_SECRETARY_GENERAL_VOTE", TRADE_SECRETARY_GENERAL_VOTE)
.value("TRADE_PLEDGE_VOTE", TRADE_PLEDGE_VOTE)
.value("TRADE_SECRETARY_GENERAL_VOTE", TRADE_SECRETARY_GENERAL_VOTE)
.value("TRADE_PLEDGE_VOTE", TRADE_PLEDGE_VOTE)
.value("TRADE_RITE_OF_PASSAGE", TRADE_RITE_OF_PASSAGE)
.value("TRADE_FREE_TRADE_ZONE", TRADE_FREE_TRADE_ZONE)
/************************************************************************************************/
/* Afforess END */
/************************************************************************************************/

/************************************************************************************************/
/* Afforess Start 06/16/10 */
/* */
/* Advanced Diplomacy */
/************************************************************************************************/
.value("TRADE_EMBASSY", TRADE_EMBASSY)
.value("TRADE_WORKER", TRADE_WORKER)
.value("TRADE_MILITARY_UNIT", TRADE_MILITARY_UNIT)
.value("TRADE_CONTACT", TRADE_CONTACT)
.value("TRADE_CORPORATION", TRADE_CORPORATION)
.value("TRADE_SECRETARY_GENERAL_VOTE", TRADE_SECRETARY_GENERAL_VOTE)
.value("TRADE_PLEDGE_VOTE", TRADE_PLEDGE_VOTE)
.value("TRADE_RITE_OF_PASSAGE", TRADE_RITE_OF_PASSAGE)
.value("TRADE_FREE_TRADE_ZONE", TRADE_FREE_TRADE_ZONE)
/************************************************************************************************/
/* Afforess END */
/************************************************************************************************/

OrionVeteran · Mar 17, 2012

Roamty said:
CyEnumsInterface I found this and change it to the second one double

Yup. We don't need any duplicate information. :goodjob:

Thanks again Roamty.

stolenrays · Mar 17, 2012

Thanks again roamty.

keldath · Mar 30, 2012

stolenrays,

hi - will v 2.1 will be released soon?

stolenrays · Mar 30, 2012

Yes, my busy schedule get in the way though.

Archid · Apr 10, 2012

Just been looking through the code and it looks like the m_iSecretaryGeneralVoteRefuseAttitudeThreshold variable from CvInfos.cpp does not have a default set and is not part of the CvLeaderHeadInfo XML file, though the code to read the entry from the XML file exists.

There is a call to CvLeaderHeadInfo::getSecretaryGeneralVoteRefuseAttitudeThreshold() from CvPlayer.cpp so the value is used.

The following changes should resolve it:

-------------------
CvInfos.cpp
-------------------

Code:

/***********************************************************************************************/
/* Afforess	                  Start		 07/29/10                                               */
/*                                                                                              */
/* Advanced Diplomacy                                                                           */
/***********************************************************************************************/
,m_iMilitaryUnitRefuseAttitudeThreshold(ATTITUDE_ANNOYED)
,m_iWorkerRefuseAttitudeThreshold(ATTITUDE_ANNOYED)
,m_iCorporationRefuseAttitudeThreshold(ATTITUDE_CAUTIOUS)
,m_iEmbassyRefuseAttitudeThreshold(ATTITUDE_ANNOYED)
/***********************************************************************************************/
/* Afforess	                     END                                                            */
/***********************************************************************************************/

Should be changed to:

/***********************************************************************************************/
/* Afforess	                  Start		 07/29/10                                               */
/*                                                                                              */
/* Advanced Diplomacy                                                                           */
/***********************************************************************************************/
,m_iMilitaryUnitRefuseAttitudeThreshold(ATTITUDE_ANNOYED)
,m_iWorkerRefuseAttitudeThreshold(ATTITUDE_ANNOYED)
,m_iCorporationRefuseAttitudeThreshold(ATTITUDE_CAUTIOUS)
,m_iEmbassyRefuseAttitudeThreshold(ATTITUDE_ANNOYED)
,m_iSecretaryGeneralVoteRefuseAttitudeThreshold(ATTITUDE_ANNOYED)
/***********************************************************************************************/
/* Afforess	                     END                                                            */
/***********************************************************************************************/

-------------------------------
CIV4CivilizationsSchema.xml
-------------------------------

Code:

<!-- Afforess Advanced Diplomacy -->
<ElementType name="MilitaryUnitRefuseAttitudeThreshold" content="textOnly"/>
<ElementType name="WorkerRefuseAttitudeThreshold" content="textOnly"/>
<ElementType name="CorporationRefuseAttitudeThreshold" content="textOnly"/>
<ElementType name="EmbassyRefuseAttitudeThreshold" content="textOnly"/>
<!-- Afforess Advanced Diplomacy -->

becomes

<!-- Afforess Advanced Diplomacy -->
<ElementType name="MilitaryUnitRefuseAttitudeThreshold" content="textOnly"/>
<ElementType name="WorkerRefuseAttitudeThreshold" content="textOnly"/>
<ElementType name="CorporationRefuseAttitudeThreshold" content="textOnly"/>
<ElementType name="EmbassyRefuseAttitudeThreshold" content="textOnly"/>
<ElementType name="SecretaryGeneralVoteRefuseAttitudeThreshold" content="textOnly"/>
<!-- Afforess Advanced Diplomacy -->

AND

<!-- Afforess Advanced Diplomacy -->
<element type="MilitaryUnitRefuseAttitudeThreshold" minOccurs="0"/>
<element type="WorkerRefuseAttitudeThreshold" minOccurs="0"/>
<element type="CorporationRefuseAttitudeThreshold" minOccurs="0"/>
<element type="EmbassyRefuseAttitudeThreshold" minOccurs="0"/>
<!-- Afforess Advanced Diplomacy -->

becomes

<!-- Afforess Advanced Diplomacy -->
<element type="MilitaryUnitRefuseAttitudeThreshold" minOccurs="0"/>
<element type="WorkerRefuseAttitudeThreshold" minOccurs="0"/>
<element type="CorporationRefuseAttitudeThreshold" minOccurs="0"/>
<element type="EmbassyRefuseAttitudeThreshold" minOccurs="0"/>
<element type="SecretaryGeneralVoteRefuseAttitudeThreshold" minOccurs="0"/>
<!-- Afforess Advanced Diplomacy -->

-----------------------------
CIV4LeaderHeadInfos.xml
-----------------------------

All instances of

Code:

<!-- Afforess Advanced Diplomacy-->
<MilitaryUnitRefuseAttitudeThreshold>ATTITUDE_ANNOYED</MilitaryUnitRefuseAttitudeThreshold>
<WorkerRefuseAttitudeThreshold>ATTITUDE_ANNOYED</WorkerRefuseAttitudeThreshold>
<CorporationRefuseAttitudeThreshold>ATTITUDE_CAUTIOUS</CorporationRefuseAttitudeThreshold>
<EmbassyRefuseAttitudeThreshold>ATTITUDE_ANNOYED</EmbassyRefuseAttitudeThreshold>
<!-- Afforess Advanced Diplomacy-->

should become

<!-- Afforess Advanced Diplomacy-->
<MilitaryUnitRefuseAttitudeThreshold>ATTITUDE_ANNOYED</MilitaryUnitRefuseAttitudeThreshold>
<WorkerRefuseAttitudeThreshold>ATTITUDE_ANNOYED</WorkerRefuseAttitudeThreshold>
<CorporationRefuseAttitudeThreshold>ATTITUDE_CAUTIOUS</CorporationRefuseAttitudeThreshold>
<EmbassyRefuseAttitudeThreshold>ATTITUDE_ANNOYED</EmbassyRefuseAttitudeThreshold>
<SecretaryGeneralVoteRefuseAttitudeThreshold>ATTITUDE_ANNOYED</SecretaryGeneralVoteRefuseAttitudeThreshold>
<!-- Afforess Advanced Diplomacy-->

I making an assumption that the correct threshold value is ATTITUDE_ANNOYED here of course.

keldath · Apr 10, 2012

Archid,
GREAT JOB,

i often wondered about this while looking at the code, i dunno how to code, but im a good merger.

thank you friend.

Advanced Diplomacy 2

Deity

Deity

Deity

BTS

Deity

Deity

Deity

Deity

Deity

Deity

Deity

Emperor

Deity

BTS

Deity

Deity

LivE LonG AnD PrOsPeR

Deity

Warlord

LivE LonG AnD PrOsPeR

Similar threads