Merge pull request #479 from punksta/master

safe FirebaseInstanceId.getInstance() access.

Merge pull request #479 from punksta/master
safe FirebaseInstanceId.getInstance() access.
edbf2ac3 · Libin Lu · GitHub · dfc91524 · 3412255e · edbf2ac3
Commit edbf2ac3 authored Jul 31, 2017 by Libin Lu Committed by GitHub Jul 31, 2017
Show whitespace changes
Inline Side-by-side

Showing with 7 additions and 2 deletions

android/src/main/java/com/evollu/react/fcm/FIRMessagingModule.java ...rc/main/java/com/evollu/react/fcm/FIRMessagingModule.java +7 -2

No files found.
--- a/android/src/main/java/com/evollu/react/fcm/FIRMessagingModule.java
+++ b/android/src/main/java/com/evollu/react/fcm/FIRMessagingModule.java
@@ -70,8 +70,13 @@ public class FIRMessagingModule extends ReactContextBaseJavaModule implements Li

    @ReactMethod
    public void getFCMToken(Promise promise) {
+        try {
             Log.d(TAG, "Firebase token: " + FirebaseInstanceId.getInstance().getToken());
             promise.resolve(FirebaseInstanceId.getInstance().getToken());
+        } catch (Throwable e) {
+             e.printStackTrace();
+             promise.reject(null,e.getMessage());
+        }
    }

    @ReactMethod