Merge pull request #358 from wix/betterIntentHandling

Resolve intent by extra key and not by `google.message_id` string

Author: yogevbd

.gitignore

@@ -188,4 +188,5 @@ package-lock.json
 
 .history
 android/.idea
-android/build
+android/build
+android/.gradle

android/.gradle/4.4/fileHashes/fileHashes.bin

@@ -1,2 +1,2 @@
-#Tue Mar 05 16:05:24 EET 2019
+#Mon Aug 12 14:22:42 IDT 2019
 gradle.version=4.4

android/.gradle/4.4/fileHashes/resourceHashesCache.bin

@@ -18,6 +18,7 @@
 import com.wix.reactnativenotifications.core.AppLifecycleFacade;
 import com.wix.reactnativenotifications.core.AppLifecycleFacadeHolder;
 import com.wix.reactnativenotifications.core.InitialNotificationHolder;
+import com.wix.reactnativenotifications.core.NotificationIntentAdapter;
 import com.wix.reactnativenotifications.core.ReactAppLifecycleFacade;
 import com.wix.reactnativenotifications.core.notification.IPushNotification;
 import com.wix.reactnativenotifications.core.notification.PushNotification;
@@ -62,8 +63,8 @@ public void onActivityResult(Activity activity, int requestCode, int resultCode,
 
     @Override
     public void onNewIntent(Intent intent) {
-        Bundle notificationData = intent.getExtras();
-        if (notificationData != null) {
+        if (NotificationIntentAdapter.canHandleIntent(intent)) {
+            Bundle notificationData = intent.getExtras();
             final IPushNotification notification = PushNotification.get(getReactApplicationContext().getApplicationContext(), notificationData);
             if (notification != null) {
                 notification.onOpened();

android/.gradle/4.4/javaCompile/classAnalysis.bin

@@ -14,6 +14,7 @@
 import com.google.firebase.FirebaseApp;
 import com.wix.reactnativenotifications.core.AppLifecycleFacade;
 import com.wix.reactnativenotifications.core.AppLifecycleFacadeHolder;
+import com.wix.reactnativenotifications.core.NotificationIntentAdapter;
 import com.wix.reactnativenotifications.core.notification.IPushNotification;
 import com.wix.reactnativenotifications.core.notification.PushNotification;
 import com.wix.reactnativenotifications.core.notificationdrawer.IPushNotificationsDrawer;
@@ -64,13 +65,11 @@ public void onActivityCreated(Activity activity, Bundle savedInstanceState) {
         notificationsDrawer.onNewActivity(activity);
 
         Intent intent = activity.getIntent();
-        if (intent != null) {
+        if (NotificationIntentAdapter.canHandleIntent(intent)) {
             Bundle notificationData = intent.getExtras();
-            if (notificationData != null) {
-                final IPushNotification pushNotification = PushNotification.get(mApplication.getApplicationContext(), notificationData);
-                if (pushNotification != null) {
-                    pushNotification.onOpened();
-                }
+            final IPushNotification pushNotification = PushNotification.get(mApplication.getApplicationContext(), notificationData);
+            if (pushNotification != null) {
+                pushNotification.onOpened();
             }
         }
     }

android/.gradle/4.4/javaCompile/jarAnalysis.bin

@@ -18,4 +18,15 @@ public static PendingIntent createPendingNotificationIntent(Context appContext,
     public static Bundle extractPendingNotificationDataFromIntent(Intent intent) {
         return intent.getBundleExtra(PUSH_NOTIFICATION_EXTRA_NAME);
     }
+
+    public static boolean canHandleIntent(Intent intent) {
+        if (intent != null) {
+            Bundle notificationData = intent.getExtras();
+            if (notificationData != null && intent.hasExtra(PUSH_NOTIFICATION_EXTRA_NAME)) {
+                return true;
+            }
+        }
+
+        return false;
+    }
 }

android/.gradle/4.4/javaCompile/taskHistory.bin

@@ -8,6 +8,8 @@
 import android.content.Intent;
 import android.os.Build;
 import android.os.Bundle;
+import android.os.Debug;
+import android.util.Log;
 
 import com.facebook.react.bridge.ReactContext;
 import com.wix.reactnativenotifications.core.AppLaunchHelper;
@@ -43,10 +45,6 @@ public void onAppNotVisible() {
     };
 
     public static IPushNotification get(Context context, Bundle bundle) {
-        if (verifyNotificationBundle(bundle) == false) {
-            return null;
-        }
-
         Context appContext = context.getApplicationContext();
         if (appContext instanceof INotificationsApplication) {
             return ((INotificationsApplication) appContext).getPushNotification(context, bundle, AppLifecycleFacadeHolder.get(), new AppLaunchHelper());
@@ -62,14 +60,6 @@ protected PushNotification(Context context, Bundle bundle, AppLifecycleFacade ap
         mNotificationProps = createProps(bundle);
     }
 
-    private static boolean verifyNotificationBundle(Bundle bundle) {
-        if (bundle.getString("google.message_id") != null) {
-            return true;
-        }
-
-        return false;
-    }
-
     @Override
     public void onReceived() throws InvalidNotificationException {
         postNotification(null);
@@ -168,13 +158,13 @@ protected Notification.Builder getNotificationBuilder(PendingIntent intent) {
 
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
             NotificationChannel channel = new NotificationChannel(CHANNEL_ID,
-                                                                  CHANNEL_NAME,
-                                                                  NotificationManager.IMPORTANCE_DEFAULT);
+                    CHANNEL_NAME,
+                    NotificationManager.IMPORTANCE_DEFAULT);
             final NotificationManager notificationManager = (NotificationManager) mContext.getSystemService(Context.NOTIFICATION_SERVICE);
             notificationManager.createNotificationChannel(channel);
             notification.setChannelId(CHANNEL_ID);
         }
-        
+
         return notification;
     }