[Vote] (9-46) Move Map Trading To Navigation

Include in VP?


  • Total voters
    75
  • Poll closed .
Status
Not open for further replies.

hokath

Deity
Joined
Oct 3, 2013
Messages
3,311
Location
London
Motivation:
Map trading is a staple civ mechanic, but in VP at the moment it is locked behind Military Science
By the time one reaches Military Science, much of the map is often unlocked.
This is a shame!
As has been pointed out elsewhere, the Age of Discovery is when the trading of maps was in full swing. It should cap that off instead.

Proposal:
Move Map Trading to the Navigation tech

Implementation notes:
Need to do
Code:
UPDATE Technologies SET MapTrading = 0 WHERE Type = 'TECH_MILITARY_SCIENCE';
UPDATE Technologies SET MapTrading = 1 WHERE Type = 'TECH_NAVIGATION';
and then update these strings
Code:
        <Row Tag="TXT_KEY_DIPLO_TRADE_MAPS_NO_TECH_BOTH">
            <Text>Both players don't yet have the Technology required to trade this item (Military Science).</Text>
        </Row>
        <Row Tag="TXT_KEY_DIPLO_TRADE_MAPS_NO_TECH_OTHER_PLAYER">
            <Text>They don't yet have the Technology required to trade this item (Military Science).</Text>
        </Row>
        <Row Tag="TXT_KEY_DIPLO_TRADE_MAPS_NO_TECH_PLAYER">
            <Text>You don't yet have the Technology required to trade this item (Military Science).</Text>

References:
This proposal would rename the Navigation tech
This has no effect on this proposal, except what you update those strings to!
 
and then update these strings
<Row Tag="TXT_KEY_DIPLO_TRADE_MAPS_NO_TECH_BOTH"> <Text>Both players don't yet have the Technology required to trade this item (Military Science).</Text> </Row> <Row Tag="TXT_KEY_DIPLO_TRADE_MAPS_NO_TECH_OTHER_PLAYER"> <Text>They don't yet have the Technology required to trade this item (Military Science).</Text> </Row> <Row Tag="TXT_KEY_DIPLO_TRADE_MAPS_NO_TECH_PLAYER"> <Text>You don't yet have the Technology required to trade this item (Military Science).</Text>
Why are these hardcoded :c5angry:
 
I sponsor this
If someone has a smart way to construct these strings so they will dynamically update, let me know
 
The strings are only used in DLL, so it's easy to just grab the tech description there and pass it into the GetLocalizedText call. Will probably need to assume there's only one tech unlocking it though.
Change the database text to
Code:
Both players don't yet have the Technology required to trade this item ({1_Tech:textkey}).
 
Status
Not open for further replies.
Back
Top Bottom