fix(messaging): deprecate onTokenRefresh(event => event.token) fixes #…

…2889
invertase · Nov 25, 2019 · 1940d6c · 1940d6c
1 parent 547d0a2
commit 1940d6c
Showing 1 changed file with 15 additions and 1 deletion.
diff --git a/packages/messaging/lib/index.js b/packages/messaging/lib/index.js
@@ -128,7 +128,21 @@ class FirebaseMessagingModule extends FirebaseModule {
     // TODO(salakar) rework internals as without this native module will never be ready (therefore never subscribes)
     this.native;
 
-    const subscription = this.emitter.addListener('messaging_token_refresh', listener);
+    const subscription = this.emitter.addListener('messaging_token_refresh', event => {
+      // TODO remove after v7.0.0, see: https://github.com/invertase/react-native-firebase/issues/2889
+      const { token } = event;
+      const tokenStringWithTokenAccessor = new String(token);
+      Object.defineProperty(tokenStringWithTokenAccessor, 'token', {
+        enumerable: false,
+        get() {
+          console.warn(
+            'firebase.messaging().onTokenRefresh(event => event.token) is deprecated, use onTokenRefresh(token => token) or call getToken() instead',
+          );
+          return token;
+        },
+      });
+      listener(tokenStringWithTokenAccessor);
+    });
     return () => subscription.remove();
   }