Browse Source

Try catch in analytics reporter

Krzysztof Krysiński 2 weeks ago
parent
commit
3c509effca

+ 1 - 1
src/Drawie

@@ -1 +1 @@
-Subproject commit 38fe5456e1e82664ece3eb9f742a29ee60744ed5
+Subproject commit 592622a39992789d96d3ebba580eb3c24298c285

+ 4 - 1
src/PixiEditor/Models/AnalyticsAPI/AnalyticsPeriodicReporter.cs

@@ -33,7 +33,10 @@ public class AnalyticsPeriodicReporter
         _client = client;
         _client = client;
         _performanceReporter = new PeriodicPerformanceReporter(this);
         _performanceReporter = new PeriodicPerformanceReporter(this);
 
 
-        PixiEditorSettings.Analytics.AnalyticsEnabled.ValueChanged += EnableAnalyticsOnValueChanged;
+        if (PixiEditorSettings.Analytics.AnalyticsEnabled != null)
+        {
+            PixiEditorSettings.Analytics.AnalyticsEnabled.ValueChanged += EnableAnalyticsOnValueChanged;
+        }
     }
     }
 
 
     public void Start(Guid? sessionId)
     public void Start(Guid? sessionId)

+ 9 - 2
src/PixiEditor/Views/MainWindow.axaml.cs

@@ -81,8 +81,15 @@ internal partial class MainWindow : Window
         
         
         StartupPerformance.ReportToMainViewModel();
         StartupPerformance.ReportToMainViewModel();
 
 
-        var analytics = services.GetService<AnalyticsPeriodicReporter>();
-        analytics?.Start(analyticsSessionId);
+        try
+        {
+            var analytics = services.GetService<AnalyticsPeriodicReporter>();
+            analytics?.Start(analyticsSessionId);
+        }
+        catch (Exception e)
+        {
+            CrashHelper.SendExceptionInfo(e);
+        }
 
 
         InitializeComponent();
         InitializeComponent();
     }
     }