Merge pull request #90 from Microsoft/fix-ReportUnhandledExceptions-logic

evgeny-pol · web-flow · commit 83a2903be974 · 2018-09-12T18:47:43.000+03:00
Fix ReportUnhandledExceptions logic
diff --git a/Assets/AppCenter/Plugins/AppCenterSDK/Crashes/Shared/Crashes.cs b/Assets/AppCenter/Plugins/AppCenterSDK/Crashes/Shared/Crashes.cs
@@ -47,7 +47,7 @@ public static void TrackError(Exception exception, IDictionary<string, string> p
 
         public static void OnHandleLog(string logString, string stackTrace, LogType type)
         {
-            if (_reportUnhandledExceptions && LogType.Assert == type || LogType.Exception == type || LogType.Error == type)
+            if (_reportUnhandledExceptions && (LogType.Assert == type || LogType.Exception == type || LogType.Error == type))
             {
                 var exception = CreateWrapperException(logString, stackTrace);
                 CrashesInternal.TrackException(exception.GetRawObject());

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ public static void TrackError(Exception exception, IDictionary<string, string> p`
`47`	`47`
`48`	`48`	`public static void OnHandleLog(string logString, string stackTrace, LogType type)`
`49`	`49`	`{`
`50`		`- if (_reportUnhandledExceptions && LogType.Assert == type \|\| LogType.Exception == type \|\| LogType.Error == type)`
	`50`	`+ if (_reportUnhandledExceptions && (LogType.Assert == type \|\| LogType.Exception == type \|\| LogType.Error == type))`
`51`	`51`	`{`
`52`	`52`	`var exception = CreateWrapperException(logString, stackTrace);`
`53`	`53`	`CrashesInternal.TrackException(exception.GetRawObject());`