Add null check to DropLoot() for artifact scrolls (#269)

When an artifact scroll couldn't be created, it lead to a null exception which stopped further loot templates from being processed and messed up mobs dying.

It now logs an error including the scroll data string so it can be added to item templates.

GameServer/gameobjects/GameNPC.cs

@@ -4606,7 +4606,7 @@ public virtual void DropLoot(GameObject killer)
 				foreach (ItemTemplate lootTemplate in lootTemplates)
 				{
 					if (lootTemplate == null) continue;
-					GameStaticItem loot;
+					GameStaticItem loot = null;
 					if (GameMoney.IsItemMoney(lootTemplate.Name))
 					{
 						long value = lootTemplate.Price;
@@ -4666,16 +4666,29 @@ public virtual void DropLoot(GameObject killer)
 					else if (lootTemplate.Name.StartsWith("scroll|"))
 					{
 						String[] scrollData = lootTemplate.Name.Split('|');
-						String artifactID = scrollData[1];
-						int pageNumber = UInt16.Parse(scrollData[2]);
-						loot = ArtifactMgr.CreateScroll(artifactID, pageNumber);
-						loot.X = X;
-						loot.Y = Y;
-						loot.Z = Z;
-						loot.Heading = Heading;
-						loot.CurrentRegion = CurrentRegion;
-						(loot as WorldInventoryItem).Item.IsCrafted = false;
-						(loot as WorldInventoryItem).Item.Creator = Name;
+
+						if (scrollData.Length >= 3)
+						{
+							String artifactID = scrollData[1];
+							int pageNumber = UInt16.Parse(scrollData[2]);
+							loot = ArtifactMgr.CreateScroll(artifactID, pageNumber);
+						}
+
+						if (loot == null)
+						{
+							log.Error($"Artifact scroll could not be created for data string [{lootTemplate.Name}]");
+							continue;
+						}
+						else
+						{
+							loot.X = X;
+							loot.Y = Y;
+							loot.Z = Z;
+							loot.Heading = Heading;
+							loot.CurrentRegion = CurrentRegion;
+							(loot as WorldInventoryItem).Item.IsCrafted = false;
+							(loot as WorldInventoryItem).Item.Creator = Name;
+						}
 					}
 					else
 					{