diff options
author | Ryan Wilson <wilsonryan@google.com> | 2018-05-25 09:00:03 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-05-25 09:00:03 -0400 |
commit | dd5c3c41b91a8642807303c99ba0c39e56f11e25 (patch) | |
tree | c396e0d01e50f56ac49e1b6c659f784e77aca8a3 /Firebase | |
parent | 39e7f6515bec8895b2ee56c5003f15c577094b22 (diff) |
Temporary fix for Messaging's data collection bit (#1334) (#1336)
* 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
Diffstat (limited to 'Firebase')
-rw-r--r-- | Firebase/Messaging/FIRMessaging.m | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/Firebase/Messaging/FIRMessaging.m b/Firebase/Messaging/FIRMessaging.m index d5e0a01..3c1c730 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; } |