Added output cache (massive performance boost)

Now each check is only executed once instead of once once for each def xml in each mod.
Also avoids the issue where the second call will make that call execute twice, the third 3 times (1+2+3=6 times) etc.
Those two combined meant a million executions of each check with around 250 mods

This also fixes the issue where log entries could be written many times

Author: Nightinggale

About/About.xml

@@ -3,7 +3,7 @@
 	<name>ModCheck[B18]</name>
 	<author>Nightinggale</author>
 	<url>https://ludeon.com/forums/index.php?topic=36534.0</url>
-	<targetVersion>0.18.2</targetVersion>
+	<targetVersion>0.18.3</targetVersion>
 	<description>Adds the ability to create errors if required mods aren't loaded or incompatible mods are loaded as well as adding patch conditions related to other mods.
 
 For players:

Assemblies/ModCheck.dll

@@ -20,6 +20,8 @@ public abstract class ModCheckBase : PatchOperation
 
         protected List<string> altModNames = new List<string>();
 
+        private bool internalHasCache = false;
+        private bool internalSuccess  = false;
 
 
         protected bool isModLoaded(string name)
@@ -106,6 +108,14 @@ protected virtual void handleError(ArgumentException ex)
 
         protected override bool ApplyWorker(XmlDocument xml)
         {
+            // use cached output if it exist
+            // the reason is that the calculations should be done once, not once for each def xml file
+            if (internalHasCache)
+            {
+                return internalSuccess;
+            }
+            internalHasCache = true;
+            
             try
             {
                 if (modName.NullOrEmpty()) throw new ArgumentException("MissingModName");
@@ -114,13 +124,14 @@ protected override bool ApplyWorker(XmlDocument xml)
                 // include the modname in the alt names. That way all names will be used if alt names are looped
                 altModNames.Add(modName);
 
-                bool testPassed = isTestPassed();
-                writeLogEntry(testPassed);
-                return testPassed;
+                bool internalSuccess = isTestPassed();
+                writeLogEntry(internalSuccess);
+                return internalSuccess;
             }
             catch (ArgumentException ex)
             {
                 handleError(ex);
+                internalSuccess = false;
                 return false;
             }
         }

Source/ModCheck.cs

@@ -1,6 +1,10 @@
 Changelog for ModCheck
 Updating ModCheck will not break existing xml files.
 
+v1.3
+- Fix: checks are now only executed once (massive performance boost)
+- Fix: writing to the log will now always only write one line and never repeat the same many times
+
 v1.2
 - Added custom message support (like logging: My mod detected modX and will patch itself accordingly)
 - Added ability to detect another mod by more than one name (like name v1.3 and name v1.4)