fix: Updated ParseOptions to use original Default options

Updated ParseOptions to use original Default options and also set the default parse to DefaultParsing
barrett777 · Jan 21, 2020 · 78b77c1 · 78b77c1
1 parent ad81f18
commit 78b77c1
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 6 deletions.
diff --git a/Heroes.ReplayParser.ConsoleApplication/Program.cs b/Heroes.ReplayParser.ConsoleApplication/Program.cs
@@ -14,7 +14,7 @@ static void Main(string[] args)
 
             // Attempt to parse the replay
             // Ignore errors can be set to true if you want to attempt to parse currently unsupported replays, such as 'VS AI' or 'PTR Region' replays
-            var (replayParseResult, replay) = DataParser.ParseReplay(randomReplayFileName, deleteFile: false, ParseOptions.FullParsing);
+            var (replayParseResult, replay) = DataParser.ParseReplay(randomReplayFileName, deleteFile: false, ParseOptions.DefaultParsing);
 
             // If successful, the Replay object now has all currently available information
             if (replayParseResult == DataParser.ReplayParseResult.Success)

diff --git a/Heroes.ReplayParser/ParseOptions.cs b/Heroes.ReplayParser/ParseOptions.cs
@@ -20,9 +20,9 @@ public class ParseOptions
         public bool IgnoreErrors { get; set; } = false;
         public bool AllowPTR { get; set; } = false;
         public bool ShouldParseEvents { get; set; } = true;
-        public bool ShouldParseUnits { get; set; } = false;
-        public bool ShouldParseMouseEvents { get; set; } = false;
-        public bool ShouldParseDetailedBattleLobby { get; set; } = true;
+        public bool ShouldParseUnits { get; set; } = true;
+        public bool ShouldParseMouseEvents { get; set; } = true;
+        public bool ShouldParseDetailedBattleLobby { get; set; } = false;
         public bool ShouldParseMessageEvents { get; set; } = false;
         public bool ShouldParseStatistics { get; set; } = true;
 
@@ -34,23 +34,36 @@ public class ParseOptions
             AllowPTR = true,
             ShouldParseDetailedBattleLobby = true,
             ShouldParseMouseEvents = true,
-            ShouldParseUnits = true,
             ShouldParseMessageEvents = true,
         };
 
+        /// <summary>
+        /// Parse as excluding Units, Mouse Events, and Enabling the DetailedBattleLobby
+        /// </summary>
+        public static ParseOptions MediumParsing => new ParseOptions()
+        {
+            ShouldParseUnits = false,
+            ShouldParseMouseEvents = false,
+            ShouldParseDetailedBattleLobby = true,
+        };
+
         /// <summary>
         /// Parse as little as possible
         /// </summary>
         public static ParseOptions MinimalParsing => new ParseOptions()
         {
             ShouldParseEvents = false,
+            ShouldParseUnits = false,
+            ShouldParseMouseEvents = false,
             ShouldParseDetailedBattleLobby = false
         };
 
+
+
         /// <summary>
         /// Parsing for typical needs, excludes events, units and mouseevents.
         /// </summary>
-        public static ParseOptions TypicalParsing => new ParseOptions();
+        public static ParseOptions DefaultParsing => new ParseOptions();
 
     }
 }