Ticket #4239: PRIOR_broken_serialization_patch.diff

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

@@ -6 +6 @@
     "</attribute>" +
     "<text a:help='A whitespace-separated list of aura files, placed under simulation/data/auras/'/>";
 
+Auras.prototype.Serialize = function()
+{
+    let ret = {};
+    for (let key in this)
+    {
+        if (!this.hasOwnProperty(key))
+            continue;
+
+        // Don't serialize the templates themselves
+        if (key == "auras")
+            continue;
+
+        ret[key] = this[key];
+    }
+
+    return ret;
+};
+
+Auras.prototype.Deserialize = function(data)
+{
+    for (let key in data)
+        if (data.hasOwnProperty(key))
+            this[key] = data[key];
+
+    // Reload aura templates from the disk
+    this.auras = {};
+    let cmpDataTemplateManager = Engine.QueryInterface(SYSTEM_ENTITY, IID_DataTemplateManager);
+    for (let name of this.GetAuraNames())
+        this.auras[name] = cmpDataTemplateManager.GetAuraTemplate(name);
+};
+
 Auras.prototype.Init = function()
 {
     let cmpDataTemplateManager = Engine.QueryInterface(SYSTEM_ENTITY, IID_DataTemplateManager);
     this.auras = {};
     this.affectedPlayers = {};
-    let auraNames = this.GetAuraNames();
-    for (let name of auraNames)
+    for (let name of this.GetAuraNames())
     {
         this.affectedPlayers[name] = [];
         this.auras[name] = cmpDataTemplateManager.GetAuraTemplate(name);

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

@@ -15 +15 @@
         this.GetTechnologyTemplate(techNames[i]);
 };
 
+DataTemplateManager.prototype.Serialize = null; // we have no dynamic state to save
+
+DataTemplateManager.prototype.Deserialize = function()
+{
+    this.Init();
+};
+
 DataTemplateManager.prototype.GetTechnologyTemplate = function(template)
 {
     if (!this.allTechs[template])

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

@@ -5 +5 @@
 
 TechnologyManager.prototype.Serialize = function()
 {
-    // The modifications cache will be affected by property reads from the GUI and other places so we shouldn't 
-    // serialize it.
+    let ret = {};
+    for (let key in this)
+        if (this.hasOwnProperty(key))
+            ret[key] = this[key];
 
-    var ret = {};
-    for (var i in this)
-    {
-        if (this.hasOwnProperty(i))
-            ret[i] = this[i];
-    }
+    // The modifications cache will be affected by property reads from the GUI and
+    // other places so we shouldn't serialize it.
     ret.modificationCache = {};
+
     return ret;
 };
 
@@ -45 +44 @@
     for (var key in allTechs)
     {
         if (allTechs[key].autoResearch || allTechs[key].top)
-            this.autoResearchTech[key] = allTechs[key];
+            this.autoResearchTech[key] = true;
     }
 };