I'm writing a parser that will load the DoD XML files into a SQL database for easy querying. I'm running into some bugs in the XML files that is making it difficult to parse them. Obviously they don't cause the game to crash, but it would be very helpful to my efforts if these were fixed: \expansion\game\itemDB.xml Line 884: "percentage" is misspelled as "percent":<crossbowShotBuff percent="100" name="Ignition Bolt"/>Line 268,276: Mask of the Golden God has two artifact elements.<artifact quality="2" /><artifact quality="4"/>Lines 124, 129, 134: These new reagents aren't marked with alchemical="1".They do appear as "Reagent" in the game, so I'm not sure if this attribute is actually important. However, for consistency it would be good if they were marked in the same way as all of the other reagents.EDIT: Actually, this doesn't matter. I just assume that an item without any type information is a reagent, which is exactly what the game does. The only real use for the alchemical attribute is to differentiate gems from runes. Have you considered making an explicit <rune/> element and eliminating "alchemical" altogether?\game\itemDB.xml Line 518: Three-Pronged Sword is missing type="0" which identifies it as a sword. EDIT: It turns out that if this is missing then it defaults to sword, anyway.\game\spellDB.xml Line 379: Make Puffballs has two <anim> elements.<anim sprite="sprites/sfx/glintA/glintA" frames="5" framerate="60" centerEffect="0" sfx="pong" /><anim sprite="sprites/sfx/psionic_generic/psionic_generic" frames="5" framerate="100" centerEffect="1"/>Line 2564: Armouring spell buff has an attribute, "timer", which I think is supposed to be "time",<buff brittle="5" useTimer="0" timer="0" self="1" ...>Lines 1436, 1450: "resistances" doesn't appear to be a valid element. It should be resistBuff.<resistances necromantic="-2" /><resistances necromantic="-1" />Line 1267: The halo for "The Recursive Curse" has bad="1". I don't see how a halo sprite can be "bad".\expansion\game\spellDB.xml Line 420: Same issue with "resistances".
Since the new expansion is out, there are new bugs, as well as some of these old ones, so this thread is no longer applicable. I posted all of these in the expansion feedback thread.
