patchFreeciv - Patches: patch #3611, Ruleset setting about cancelling...

Show feedback again

patch #3611: Ruleset setting about cancelling illegal traderoutes

Submitted by:  Marko Lindqvist <cazfi>
Submitted on:  Tue 22 Jan 2013 09:03:11 PM UTC  
Category: generalPriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: Marko Lindqvist <cazfi>Open/Closed: Closed
Planned Release: 2.5.0Contains string changes: None

Add a New Comment (Rich MarkupRich Markup):

You are not logged in

Please log in, so followups can be emailed to you.


Mon 28 Jan 2013 03:48:29 PM UTC, SVN revision 22228:

Added ruleset setting to control what happens to existing traderoutes
which are no longer legal. This can happen when traderoute type changes
as a result of city changing owner or terrain transformations.

See gna patch #3611

(Browse SVN revision 22228)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Thu 24 Jan 2013 08:37:39 PM UTC, comment #1:

- Fixed keeping of traderoute over city owner change
- Get 0 trade from Inactive traderoute instead of "random" (uninitialized) amount
- Set values in ruleset "Cancel" rather than "Active" as it turned out that the old behavior was like that when city changes owner and traderoute turns illegal

(file #17009)

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Tue 22 Jan 2013 09:03:11 PM UTC, original submission:

So far only compile-tested patch, but posting already for discussion.

There's currently no handling of already established traderoutes turned illegal. With most supplied rulesets such a thing could happen if city changes owner and suddenly trading cities should be at least trademindist apart, but are not. Civ2civ3 ruleset prohibits national traderoutes altogether.
Does anybody know how civ/2 handle this? In attached patch I've added only TOOD comment to civ/2 rulesets that settings should be corrected.

Patch adds ruleset setting to control what happens to each kind (national, intercontinental, international, inic) of illegal traderoutes. Possible values are "Active" - traderoute is fully functional, "Inactive" - traderoute is not functional, but remains in case it later turns back legal, and "Cancel" - traderoute is removed.

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.


(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):

Attached Files


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by cazfi (Submitted the item)

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.


    Error: not logged in



    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 28 Jan 2013 03:48:41 PM UTCcazfiStatusReady For Test=>Done
      Assigned toNone=>cazfi
    Thu 24 Jan 2013 08:37:39 PM UTCcazfiAttached File-=>Added TraderouteCancellingSetting-2.patch, #17009
    Tue 22 Jan 2013 09:03:11 PM UTCcazfiAttached File-=>Added TraderouteCancellingSetting.patch, #16992
    Show feedback again

    Back to the top

    Powered by Savane 3.1-cleanup