Merge pull request #226 from Laupetin/fix/t6-material-json-errors

fix: not properly handling errors when loading json

src/ObjCommon/Json/JsonExtension.h

@@ -18,7 +18,7 @@ namespace nlohmann
     template<class T> void optional_from_json(const nlohmann::json& j, const char* name, std::optional<T>& value)
     {
         const auto it = j.find(name);
-        if (it != j.end())
+        if (it != j.end() && !it->is_null())
             value = it->get<T>();
         else
             value = std::nullopt;

src/ObjLoading/Game/IW4/Leaderboard/JsonLeaderboardDefLoader.cpp

@@ -36,8 +36,17 @@ namespace
                 return false;
             }
 
-            const auto jLeaderboard = jRoot.get<JsonLeaderboardDef>();
-            return CreateLeaderboardFromJson(jLeaderboard, leaderboardDef);
+            try
+            {
+                const auto jLeaderboard = jRoot.get<JsonLeaderboardDef>();
+                return CreateLeaderboardFromJson(jLeaderboard, leaderboardDef);
+            }
+            catch (const json::exception& e)
+            {
+                std::cerr << std::format("Failed to parse json of leaderboard: {}\n", e.what());
+            }
+
+            return false;
         }
 
     private:

src/ObjLoading/Game/IW5/Leaderboard/JsonLeaderboardDefLoader.cpp

@@ -36,8 +36,17 @@ namespace
                 return false;
             }
 
-            const auto jLeaderboard = jRoot.get<JsonLeaderboardDef>();
-            return CreateLeaderboardFromJson(jLeaderboard, leaderboardDef);
+            try
+            {
+                const auto jLeaderboard = jRoot.get<JsonLeaderboardDef>();
+                return CreateLeaderboardFromJson(jLeaderboard, leaderboardDef);
+            }
+            catch (const json::exception& e)
+            {
+                std::cerr << std::format("Failed to parse json of leaderboard: {}\n", e.what());
+            }
+
+            return false;
         }
 
     private:

src/ObjLoading/Game/IW5/Weapon/JsonWeaponAttachmentLoader.cpp

@@ -44,8 +44,17 @@ namespace
                 return false;
             }
 
-            const auto jAttachment = jRoot.get<JsonWeaponAttachment>();
-            return CreateWeaponAttachmentFromJson(jAttachment, attachment);
+            try
+            {
+                const auto jAttachment = jRoot.get<JsonWeaponAttachment>();
+                return CreateWeaponAttachmentFromJson(jAttachment, attachment);
+            }
+            catch (const json::exception& e)
+            {
+                std::cerr << std::format("Failed to parse json of attachment: {}\n", e.what());
+            }
+
+            return false;
         }
 
     private:

src/ObjLoading/Game/T6/Leaderboard/JsonLeaderboardDefLoader.cpp

@@ -37,8 +37,17 @@ namespace
                 return false;
             }
 
-            const auto jLeaderboard = jRoot.get<JsonLeaderboardDef>();
-            return CreateLeaderboardFromJson(jLeaderboard, leaderboardDef);
+            try
+            {
+                const auto jLeaderboard = jRoot.get<JsonLeaderboardDef>();
+                return CreateLeaderboardFromJson(jLeaderboard, leaderboardDef);
+            }
+            catch (const json::exception& e)
+            {
+                std::cerr << std::format("Failed to parse json of leaderboard: {}\n", e.what());
+            }
+
+            return false;
         }
 
     private:

src/ObjLoading/Game/T6/Material/JsonMaterialLoader.cpp

@@ -39,8 +39,17 @@ namespace
                 return false;
             }
 
-            const auto jMaterial = jRoot.get<JsonMaterial>();
-            return CreateMaterialFromJson(jMaterial, material);
+            try
+            {
+                const auto jMaterial = jRoot.get<JsonMaterial>();
+                return CreateMaterialFromJson(jMaterial, material);
+            }
+            catch (const json::exception& e)
+            {
+                std::cerr << std::format("Failed to parse json of material: {}\n", e.what());
+            }
+
+            return false;
         }
 
     private:

src/ObjLoading/Game/T6/WeaponCamo/JsonWeaponCamoLoader.cpp

@@ -39,8 +39,17 @@ namespace
                 return false;
             }
 
-            const auto jWeaponCamo = jRoot.get<JsonWeaponCamo>();
-            return CreateWeaponCamoFromJson(jWeaponCamo, weaponCamo);
+            try
+            {
+                const auto jWeaponCamo = jRoot.get<JsonWeaponCamo>();
+                return CreateWeaponCamoFromJson(jWeaponCamo, weaponCamo);
+            }
+            catch (const json::exception& e)
+            {
+                std::cerr << std::format("Failed to parse json of weapon camo: {}\n", e.what());
+            }
+
+            return false;
         }
 
     private:

src/ObjLoading/Game/T6/XModel/JsonXModelLoader.cpp

@@ -40,8 +40,17 @@ namespace
                 return false;
             }
 
-            const auto jXModel = jRoot.get<JsonXModel>();
-            return CreateXModelFromJson(jXModel, xmodel);
+            try
+            {
+                const auto jXModel = jRoot.get<JsonXModel>();
+                return CreateXModelFromJson(jXModel, xmodel);
+            }
+            catch (const json::exception& e)
+            {
+                std::cerr << std::format("Failed to parse json of xmodel: {}\n", e.what());
+            }
+
+            return false;
         }
 
     private:

thirdparty/json.lua

@@ -2,6 +2,9 @@ json = {}
 
 function json:include(includes)
 	if includes:handle(self:name()) then
+		defines {
+			"JSON_DIAGNOSTICS=1"
+		}
 		includedirs {
 			path.join(ThirdPartyFolder(), "json", "single_include")
 		}