Ticket #2243: actorTech.diff

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

@@ -355 +355 @@
     {
         var cmpTemplateManager = Engine.QueryInterface(SYSTEM_ENTITY, IID_TemplateManager);
         var templateName = cmpTemplateManager.GetCurrentTemplateName(ent);
-        // Ensure that preview entites have the same properties as the final building
+        // Ensure that preview and construction entites have the same properties as the final building
         if (templateName.indexOf("preview|") != -1)
             templateName = templateName.slice(8);
+        if (templateName.indexOf("construction|") != -1)
+            templateName = templateName.slice(13);
         this.modificationCache[valueName][ent] = GetTechModifiedProperty(this.modifications, cmpTemplateManager.GetTemplate(templateName), valueName, curValue);
     }
     

binaries/data/mods/public/simulation/data/technologies/phase_town.json

@@ -2 +2 @@
     "genericName": "Town Phase",
     "requirements": { "any": [{"tech":"phase_town_generic"}, {"tech":"phase_town_athen"}]},
     "autoResearch": true,
-    "modifications": [{"value": "TerritoryInfluence/Radius", "multiplier": 1.30}],
+    "modifications": [{"value": "TerritoryInfluence/Radius", "multiplier": 1.30},{"value": "VisualActor/Actor", "replace":"structures/athenians/gymnasion.xml"}],
     "affects": ["CivCentre"],
     "soundComplete": "interface/alarm/alarm_phase.xml"
 }

source/simulation2/components/CCmpVisualActor.cpp

@@ -30 +30 @@
 #include "ICmpTemplateManager.h"
 #include "ICmpTerrain.h"
 #include "ICmpUnitMotion.h"
+#include "ICmpValueModificationManager.h"
 #include "ICmpVision.h"
 
 #include "graphics/Decal.h"
@@ -56 +57 @@
         componentManager.SubscribeToMessageType(MT_Interpolate);
         componentManager.SubscribeToMessageType(MT_RenderSubmit);
         componentManager.SubscribeToMessageType(MT_OwnershipChanged);
+        componentManager.SubscribeToMessageType(MT_ValueModification);
         componentManager.SubscribeGloballyToMessageType(MT_TerrainChanged);
     }
 
@@ -62 +64 @@
     DEFAULT_COMPONENT_ALLOCATOR(VisualActor)
 
 private:
-    std::wstring m_ActorName;
+    std::wstring m_OriginalActorName, m_ActorName;
     CUnit* m_Unit;
 
     fixed m_R, m_G, m_B; // shading colour
@@ -182 +184 @@
         m_Seed = GetEntityId();
 
         if (paramNode.GetChild("Foundation").IsOk() && paramNode.GetChild("FoundationActor").IsOk())
+        {
             m_ActorName = paramNode.GetChild("FoundationActor").ToString();
+            m_OriginalActorName = L"";
+        }
         else
-            m_ActorName = paramNode.GetChild("Actor").ToString();
+        {
+            m_OriginalActorName = m_ActorName = paramNode.GetChild("Actor").ToString();
+        }
 
         m_VisibleInAtlasOnly = paramNode.GetChild("VisibleInAtlasOnly").ToBool();
 
@@ -297 +304 @@
         {
             const CMessageOwnershipChanged& msgData = static_cast<const CMessageOwnershipChanged&> (msg);
             m_Unit->GetModel().SetPlayerID(msgData.to);
+
+            // also apply the technology changes to preview models
+            if (m_OriginalActorName.length() == 0)
+                break; // don't change foundation actors 
+            CmpPtr<ICmpValueModificationManager> cmpValueModificationManager(GetSystemEntity());
+            std::wstring newActorName = cmpValueModificationManager->ApplyModifications(L"VisualActor/Actor", m_OriginalActorName, GetEntityId());
+            if (newActorName != m_ActorName)
+            {
+                m_ActorName = newActorName;
+                ReloadActor();
+            }
             break;
         }
         case MT_TerrainChanged:
@@ -305 +323 @@
             m_Unit->GetModel().SetTerrainDirty(msgData.i0, msgData.j0, msgData.i1, msgData.j1);
             break;
         }
+        case MT_ValueModification:
+        {
+            const CMessageValueModification& msgData = static_cast<const CMessageValueModification&> (msg);
+            if (msgData.component == L"VisualActor")
+            {
+                if (m_OriginalActorName.length() == 0)
+                    break; // don't change foundation actors 
+                CmpPtr<ICmpValueModificationManager> cmpValueModificationManager(GetSystemEntity());
+                std::wstring newActorName = cmpValueModificationManager->ApplyModifications(L"VisualActor/Actor", m_OriginalActorName, GetEntityId());
+                if (newActorName != m_ActorName)
+                {
+                    m_ActorName = newActorName;
+                    ReloadActor();
+                }
+            }
         }
+        }
     }
 
     virtual CBoundingBoxAligned GetBounds()

source/simulation2/components/ICmpValueModificationManager.cpp

@@ -39 +39 @@
     {
         return m_Script.Call<u32>("ApplyModifications", valueName, currentValue, entity);
     }
+
+    virtual std::wstring ApplyModifications(std::wstring valueName, std::wstring currentValue, entity_id_t entity)
+    {
+        return m_Script.Call<std::wstring>("ApplyModifications", valueName, currentValue, entity);
+    }
 };
 
 REGISTER_COMPONENT_SCRIPT_WRAPPER(ValueModificationManagerScripted)

source/simulation2/components/ICmpValueModificationManager.h

@@ -32 +32 @@
 public:
     virtual fixed ApplyModifications(std::wstring valueName, fixed currentValue, entity_id_t entity) = 0;
     virtual u32 ApplyModifications(std::wstring valueName, u32 currentValue, entity_id_t entity) = 0;
+    virtual std::wstring ApplyModifications(std::wstring valueName, std::wstring currentValue, entity_id_t entity) = 0;
 
     DECLARE_INTERFACE_TYPE(ValueModificationManager)
 };