From 1030d5ec9da07ff8bf0b18beae95558797b26a5b Mon Sep 17 00:00:00 2001 From: Ryan Wilson Date: Fri, 25 May 2018 08:55:46 -0400 Subject: Temporary fix for Messaging's data collection bit (#1334) * Temporary fix for Messaging's data collection bit This ensures that existing behavior still works - InstanceID can sometimes fetch stored values before `FirebaseApp.configure()` is called, causing the global flag to be `NO`. Will file an issue shortly. * Remove extra whitespaces Thanks, GitHub editor. * Fix method being called --- Firebase/Messaging/FIRMessaging.m | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'Firebase/Messaging') diff --git a/Firebase/Messaging/FIRMessaging.m b/Firebase/Messaging/FIRMessaging.m index 972b4fa..3c4d999 100644 --- a/Firebase/Messaging/FIRMessaging.m +++ b/Firebase/Messaging/FIRMessaging.m @@ -173,6 +173,10 @@ NSString *const kFIRMessagingPlistAutoInitEnabled = _loggedMessageIDs = [NSMutableSet set]; _instanceID = instanceID; _messagingUserDefaults = defaults; + + // TODO: Remove this once the race condition with FIRApp configuring and InstanceID + // is fixed. This must be fixed before Core's flag becomes public. + _globalAutomaticDataCollectionEnabled = YES; } return self; } -- cgit v1.2.3