Improved dictionary access inside GetItemName to prevent issues where now-removed modded items are checked

Libraries/SPTarkov.Server.Core/Controllers/InsuranceController.cs

@@ -704,7 +704,7 @@ public class InsuranceController(
         var roll = returnChance >= traderReturnChance;
 
         // Log the roll with as much detail as possible.
-        var itemName = insuredItem is not null ? $"{itemHelper.GetItemName(insuredItem.Template)}" : "";
+        var itemName = insuredItem is not null ? $"{itemHelper.GetItemName(insuredItem.Template)}" : string.Empty;
         var status = roll ? "Delete" : "Keep";
         if (logger.IsLogEnabled(LogLevel.Debug))
         {

Libraries/SPTarkov.Server.Core/Helpers/ItemHelper.cs

@@ -1571,13 +1571,20 @@ public class ItemHelper(
     public string GetItemName(MongoId itemTpl)
     {
         var localeDb = localeService.GetLocaleDb();
-        var result = localeDb[$"{itemTpl} Name"];
-        if (result?.Length > 0)
+
+        // Key exists and it's not empty
+        if (localeDb.TryGetValue($"{itemTpl} Name", out var result) && result.Length > 0)
         {
             return result;
         }
 
-        return localeDb[$"{itemTpl} ShortName"];
+        // Main item "name" property not found, try the backup
+        if (localeDb.TryGetValue($"{itemTpl} ShortName", out result))
+        {
+            return result;
+        }
+
+        return string.Empty;
     }
 
     /// <summary>