Ticket #1374: errorReport.patch

gui/common/functions_civinfo.js

@@ -14 +14 @@
     for each (var filename in civFiles)
     {   // Parse data if valid file
         var data = parseJSONData(filename);
+        if (Object.keys(data).length == 0)
+            continue;
         translateObjectKeys(data, ["Name", "Description", "History", "Special"]);
         civData[data.Code] = data;
     }

simulation/components/TechnologyTemplateManager.js

@@ -19 +19 @@
 {
     if (!this.allTechs[template])
     {
-        this.allTechs[template] = Engine.ReadJSONFile("technologies/" + template + ".json");
-        if (!this.allTechs[template])
-            error("Failed to load technology \"" + template + "\"");
+        try
+        {
+            this.allTechs[template] = Engine.ReadJSONFile("technologies/" + template + ".json");
+        }
+        catch (err)
+        {
+            error(sprintf("%(error)s: parsing JSON data in %(path)s", { error: err.toString(), path: pathname }));
+            this.allTechs[template] = false;
+        }
     }
 
     return this.allTechs[template];

simulation/components/TechnologyManager.js

@@ -283 +283 @@
 
     if (!template)
     {
-        error("Tried to research invalid techonology: " + uneval(tech));
+        error("Tried to research invalid technology: " + uneval(tech));
         return;
     }
 

simulation/components/ProductionQueue.js

@@ -241 +241 @@
             // Find the template data so we can determine the build costs
             var cmpTempMan = Engine.QueryInterface(SYSTEM_ENTITY, IID_TemplateManager);
             var template = cmpTempMan.GetTemplate(templateName);
-            if (!template)
+            if (!template || Object.keys(template) == 0)
                 return;
             if (template.Promotion)
             {