FirebaseService: Perform version check when service runs

OatmealDome · Jan 23, 2023 · a6b7659 · a6b7659
1 parent 3d591d5
commit a6b7659
Showing 1 changed file with 13 additions and 9 deletions.
diff --git a/Source/iOS/App/Common/Services/FirebaseService.mm b/Source/iOS/App/Common/Services/FirebaseService.mm
@@ -17,7 +17,9 @@ @interface FirebaseService () <AnalyticsNoticeViewControllerDelegate>
 
 @end
 
-@implementation FirebaseService
+@implementation FirebaseService {
+  BOOL _shouldSendInitialEvent;
+}
 
 - (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary<UIApplicationLaunchOptionsKey,id>*)launchOptions {
   if ([VersionManager shared].buildSource != DOLBuildSourceOfficial) {
@@ -44,7 +46,14 @@ - (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(
   [[FIRCrashlytics crashlytics] setCrashlyticsCollectionEnabled:analyticsEnabled];
 
   if (analyticsEnabled) {
-    [self sendInitialAnalyticsEvents];
+    NSUserDefaults* defaults = [NSUserDefaults standardUserDefaults];
+
+    NSString* lastVersion = [defaults stringForKey:@"last_version"];
+    NSString* currentVersion = [VersionManager shared].userFacingVersion;
+
+    _shouldSendInitialEvent = ![lastVersion isEqualToString:currentVersion];
+  } else {
+    _shouldSendInitialEvent = false;
   }
 
   return true;
@@ -57,12 +66,7 @@ - (void)didFinishAnalyticsNoticeWithResult:(BOOL)result sender:(id)sender {
 }
 
 - (void)sendInitialAnalyticsEvents {
-  NSUserDefaults* defaults = [NSUserDefaults standardUserDefaults];
-
-  NSString* lastVersion = [defaults stringForKey:@"last_version"];
-  NSString* currentVersion = [VersionManager shared].userFacingVersion;
-
-  if (![lastVersion isEqualToString:currentVersion]) {
+  if (_shouldSendInitialEvent) {
     NSString* appType;
 #ifdef NONJAILBROKEN
     appType = @"non-jailbroken";
@@ -74,7 +78,7 @@ - (void)sendInitialAnalyticsEvents {
 
     [FIRAnalytics logEventWithName:@"version_start" parameters:@{
       @"type" : appType,
-      @"version" : currentVersion
+      @"version" : [VersionManager shared].userFacingVersion
     }];
   }
 }