Ticket #3894: marketVisibility-v2.1.patch

binaries/data/mods/public/simulation/components/Fogging.js

@@ -129 +129 @@
     if (cmpResourceSupply)
         cmpMirage.CopyResourceSupply(cmpResourceSupply);
 
+    var cmpMarket = Engine.QueryInterface(this.entity, IID_Market);
+    if (cmpMarket)
+        cmpMirage.CopyMarket(cmpMarket);
+
     // Notify the GUI the entity has been replaced by a mirage, in case it is selected at this moment
     var cmpGuiInterface = Engine.QueryInterface(SYSTEM_ENTITY, IID_GuiInterface);
     cmpGuiInterface.AddMiragedEntity(player, this.entity, this.mirages[player]);

binaries/data/mods/public/simulation/components/GuiInterface.js

@@ -289 +289 @@
     if (cmpBuilder)
         ret.builder = true;
 
-    let cmpMarket = Engine.QueryInterface(ent, IID_Market);
+    let cmpMarket = QueryMiragedInterface(ent, IID_Market);
     if (cmpMarket)
         ret.market = {
             "land": cmpMarket.HasType("land"),

binaries/data/mods/public/simulation/components/Market.js

@@ -41 +41 @@
     return this.tradeType.has(type);
 };
 
+Market.prototype.GetType = function()
+{
+    return this.tradeType;
+};
+
+Market.prototype.GetTraders = function()
+{
+    return this.traders;
+};
+
 /**
- * Check if all traders with a route on this market can still trade
+ * Check if the traders attached to this market can still trade with it
+ * Warning traders currently trading with a mirage of this market are dealt with in Mirage.js
  */
+
 Market.prototype.OnDiplomacyChanged = function(msg)
 {
-    for (let ent of this.traders)
+    for (let trader of this.traders)
     {
-        let cmpTrader = Engine.QueryInterface(ent, IID_Trader);
+        let cmpTrader = Engine.QueryInterface(trader, IID_Trader);
         if (!cmpTrader)
-            this.RemoveTrader(ent);
-        else if (!cmpTrader.CanTrade(this.entity))
         {
-            this.RemoveTrader(ent);
-            cmpTrader.RemoveMarket(this.entity);
+            this.RemoveTrader(trader);
+            continue;
         }
+        if (!cmpTrader.HasMarket(this.entity) || cmpTrader.CanTrade(this.entity))
+            continue;
+        // this trader can no more trade  TODO warn UnitAI
+        this.RemoveTrader(trader);
+        cmpTrader.RemoveMarket(this.entity);
     }
 };
 
 Market.prototype.OnOwnershipChanged = function(msg)
 {
-    for (let ent of this.traders)
+    for (let trader of this.traders)
     {
-        let cmpTrader = Engine.QueryInterface(ent, IID_Trader);
+        let cmpTrader = Engine.QueryInterface(trader, IID_Trader);
         if (!cmpTrader)
-            this.RemoveTrader(ent);
-        else if (msg.to == -1)
-            cmpTrader.RemoveMarket(this.entity);
-        else if (!cmpTrader.CanTrade(this.entity))
         {
-            this.RemoveTrader(ent);
-            cmpTrader.RemoveMarket(this.entity);
+            this.RemoveTrader(trader);
+            continue;
         }
+        if (!cmpTrader.HasMarket(this.entity) || (msg.to != -1 && cmpTrader.CanTrade(this.entity)))
+            continue;
+        // this trader can no more trade  TODO warn UnitAI
+        this.RemoveTrader(trader);
+        cmpTrader.RemoveMarket(this.entity);
     }
 };
 

binaries/data/mods/public/simulation/components/Mirage.js

@@ -34 +34 @@
     this.killBeforeGather = null;
     this.maxGatherers = null;
     this.numGatherers = null;
+
+    this.traders = null;
+    this.marketType = null;
+    this.internationalBonus = null;
 };
 
 Mirage.prototype.SetParent = function(ent)
@@ -125 +129 @@
 Mirage.prototype.GetMaxGatherers = function() { return this.maxGatherers; };
 Mirage.prototype.GetNumGatherers = function() { return this.numGatherers; };
 
+// Market data
+
+Mirage.prototype.CopyMarket = function(cmpMarket)
+{
+    this.miragedIids.add(IID_Market);
+    this.traders = new Set();
+    for (let trader of cmpMarket.GetTraders())
+    {
+        let cmpTrader = Engine.QueryInterface(trader, IID_Trader);
+        let cmpOwnership = Engine.QueryInterface(trader, IID_Ownership);
+        if (!cmpTrader || !cmpOwnership)
+        {
+            cmpMarket.RemoveTrader(trader);
+            continue;
+        }
+        if (this.player != cmpOwnership.GetOwner())
+            continue;
+        cmpTrader.SwitchMarket(cmpMarket.entity, this.entity);
+        cmpMarket.RemoveTrader(trader);
+        this.AddTrader(trader);
+    }
+    this.marketType = cmpMarket.GetType();
+    this.internationalBonus = cmpMarket.GetInternationalBonus();
+};
+
+Mirage.prototype.HasType = function(type) { return this.marketType.has(type); };
+Mirage.prototype.GetInternationalBonus = function() { return this.internationalBonus; };
+Mirage.prototype.AddTrader = function(trader) { this.traders.add(trader); };
+Mirage.prototype.RemoveTrader = function(trader) { this.traders.delete(trader); };
+
+Mirage.prototype.UpdateTraders = function(msg)
+{
+    let cmpMarket = Engine.QueryInterface(this.parent, IID_Market);
+    if (!cmpMarket) // The parent market does not exist anymore
+    {
+        for (let trader of this.traders)
+        {
+            // TODO warn UnitAI
+            let cmpTrader = Engine.QueryInterface(trader, IID_Trader);
+            if (cmpTrader)
+                cmpTrader.RemoveMarket(this.entity);
+        }
+        return;
+    }
+
+    // The market becomes visible, switch all traders from the mirage to the market
+    for (let trader of this.traders)
+    {
+        let cmpTrader = Engine.QueryInterface(trader, IID_Trader);
+        if (!cmpTrader)
+            continue;
+        cmpTrader.SwitchMarket(this.entity, cmpMarket.entity);
+        this.RemoveTrader(trader);
+        cmpMarket.AddTrader(trader);
+    }
+};
+
 // ============================
 
 Mirage.prototype.OnVisibilityChanged = function(msg)
@@ -132 +193 @@
     if (msg.player != this.player || msg.newVisibility != VIS_HIDDEN)
         return;
 
+    if (this.miragedIids.has(IID_Market))
+        this.UpdateTraders(msg);
+
     if (this.parent == INVALID_ENTITY)
         Engine.DestroyEntity(this.entity);
     else

binaries/data/mods/public/simulation/components/Trader.js

@@ -44 +44 @@
         {
             var garrisonMultiplier = 1;
             var garrisonedTradersCount = 0;
-            for each (var entity in cmpGarrisonHolder.GetEntities())
+            for (var entity of cmpGarrisonHolder.GetEntities())
             {
                 var cmpGarrisonedUnitTrader = Engine.QueryInterface(entity, IID_Trader);
                 if (cmpGarrisonedUnitTrader)
@@ -68 +68 @@
 // Return true if at least one of markets was changed.
 Trader.prototype.SetTargetMarket = function(target, source)
 {
-    var cmpTargetMarket = Engine.QueryInterface(target, IID_Market);
+    let cmpTargetMarket = QueryMiragedInterface(target, IID_Market);
     if (!cmpTargetMarket)
         return false;
 
@@ -75 +75 @@
     if (source)
     {
         // Establish a trade route with both markets in one go.
-        let cmpSourceMarket = Engine.QueryInterface(source, IID_Market);
+        let cmpSourceMarket = QueryMiragedInterface(source, IID_Market);
         if (!cmpSourceMarket)
             return false;
         this.markets = [source];
-        cmpSourceMarket.AddTrader(this.entity);
     }
     if (this.markets.length >= 2)
     {
@@ -87 +86 @@
         // and use the target as first market
         for (let market of this.markets)
         {
-            let cmpMarket = Engine.QueryInterface(market, IID_Market);
+            let cmpMarket = QueryMiragedInterface(market, IID_Market);
             if (cmpMarket)
                 cmpMarket.RemoveTrader(this.entity);
         }
@@ -160 +159 @@
 {
     var cmpTraderIdentity = Engine.QueryInterface(this.entity, IID_Identity);
     // Check that the target exists
-    var cmpTargetMarket = Engine.QueryInterface(target, IID_Market);
+    var cmpTargetMarket = QueryMiragedInterface(target, IID_Market);
     if (!cmpTargetMarket)
         return false;
     // Check that the target is not a foundation
@@ -252 +251 @@
 };
 
 /**
- * Called when this trader can no longer trade with the market
+ * Returns true if the trader has the given market (can be either a market or a mirage)
  */
+Trader.prototype.HasMarket = function(market)
+{
+    return this.markets.indexOf(market) != -1; 
+};
+
+/**
+ * Remove a market when this trader can no longer trade with it
+ */
 Trader.prototype.RemoveMarket = function(market)
 {
     let index = this.markets.indexOf(market);
     if (index != -1)
         this.markets.splice(index, 1);
+    else
+        error(" RemoveMarket problem: " + market);
 };
 
+/**
+ * Switch between a market and its mirage according to visibility
+ */
+Trader.prototype.SwitchMarket = function(oldMarket, newMarket)
+{
+    let index = this.markets.indexOf(oldMarket);
+    if (index == -1)
+        return;
+    this.markets[index] = newMarket;
+    let cmpUnitAI = Engine.QueryInterface(this.entity, IID_UnitAI);
+    if (cmpUnitAI)
+        cmpUnitAI.SwitchMarketOrder(oldMarket, newMarket);
+};
+
 Trader.prototype.StopTrading = function()
 {
     for (let market of this.markets)
     {
-        let cmpMarket = Engine.QueryInterface(market, IID_Market);
+        let cmpMarket = QueryMiragedInterface(market, IID_Market);
         if (cmpMarket)
             cmpMarket.RemoveTrader(this.entity);
     }

binaries/data/mods/public/simulation/components/UnitAI.js

@@ -648 +648 @@
     "Order.Trade": function(msg) {
         // We must check if this trader has both markets in case it was a back-to-work order
         var cmpTrader = Engine.QueryInterface(this.entity, IID_Trader);
-        if (!cmpTrader || ! cmpTrader.HasBothMarkets())
+        if (!cmpTrader || !cmpTrader.HasBothMarkets())
         {
             this.FinishOrder();
             return;
@@ -5344 +5344 @@
         error("UnitAI: Setting to invalid stance '"+stance+"'");
 };
 
+UnitAI.prototype.SwitchMarketOrder = function(oldMarket, newMarket)
+{
+    if (this.order.data && this.order.data.target && this.order.data.target == oldMarket)
+    {
+        this.order.data.target == newMarket;
+        // Mirages have no obstruction and can lead to strange path, so when the mirage is destroyed
+        // and the market becomes visible, redo the pathing
+        if (Engine.QueryInterface(newMarket, IID_Market) && this.GetCurrentState().split(".").pop() == "APPROACHINGMARKET")
+            this.MoveToMarket(newMarket);
+    }
+};
+
 UnitAI.prototype.SwitchToStance = function(stance)
 {
     var cmpPosition = Engine.QueryInterface(this.entity, IID_Position);

binaries/data/mods/public/simulation/helpers/TraderGain.js

@@ -1 +1 @@
 function CalculateTraderGain(firstMarket, secondMarket, traderTemplate, trader)
 {
-    let cmpMarket1 = Engine.QueryInterface(firstMarket, IID_Market);
-    let cmpMarket2 = Engine.QueryInterface(secondMarket, IID_Market);
+    let cmpMarket1 = QueryMiragedInterface(firstMarket, IID_Market);
+    let cmpMarket2 = QueryMiragedInterface(secondMarket, IID_Market);
     if (!cmpMarket1 || !cmpMarket2)
         return null;