# HG changeset patch
# User IBBoard <dev@ibboard.co.uk>
# Date 1240687791 0
# Node ID 780f3926abadce98949018f70a979600951ab9e2
# Parent  91cf8efbea0b1ded6e47c37f16959d211a6f2192
Re #50 - Complete core loading of WarFoundry XML files
 * Rename a "create" method to a "get"
 * Remove duplicate "add unit" call

diff -r 91cf8efbea0b -r 780f3926abad api/Factories/Xml/WarFoundryXmlRaceFactory.cs
--- a/api/Factories/Xml/WarFoundryXmlRaceFactory.cs	Sat Apr 25 19:26:12 2009 +0000
+++ b/api/Factories/Xml/WarFoundryXmlRaceFactory.cs	Sat Apr 25 19:29:51 2009 +0000
@@ -76,7 +76,7 @@
 			
 			foreach (XmlElement node in WarFoundryXmlFactoryUtils.SelectNodes(extraData, "/race:race/race:units/race:unit"))
 			{
-				CreateUnitTypeFromElement(node, race);
+				GetUnitTypeForElement(node, race);
 			}
 			
 			race.SetAsFullyLoaded();
@@ -92,10 +92,10 @@
 
 		private UnitType GetUnitTypeFromDocument(XmlDocument doc, string id, Race parentRace)
 		{
-			return CreateUnitTypeFromElement(WarFoundryXmlFactoryUtils.SelectSingleElement(doc, "/race:race/race:units/race:unit[id="+id+"]"), parentRace);
+			return GetUnitTypeForElement(WarFoundryXmlFactoryUtils.SelectSingleElement(doc, "/race:race/race:units/race:unit[id="+id+"]"), parentRace);
 		}
 						
-		private UnitType CreateUnitTypeFromElement(XmlElement elem, Race parentRace)
+		private UnitType GetUnitTypeForElement(XmlElement elem, Race parentRace)
 		{
 			string id = elem.GetAttribute("id");
 			UnitType type = parentRace.GetUnitType(id);
@@ -103,7 +103,6 @@
 			if (type==null)
 			{
 				type = CreateUnitTypeFromElement(elem, id, parentRace);
-				parentRace.AddUnitType(type);
 			}
 			
 			return type;