notification bug fixed

Author: divshekhar

android/src/main/java/com/devsonflutter/reflex/notification/NotificationListener.java

@@ -30,7 +30,6 @@
 public class NotificationListener extends NotificationListenerService {
 
     private static final String TAG = ReflexPlugin.getPluginTag();
-    private final Map<String, Object> autoReply = ReflexPlugin.autoReply;
 
     @RequiresApi(api = VERSION_CODES.N)
     @Override
@@ -42,7 +41,6 @@ public void onNotificationPosted(StatusBarNotification notification) {
 
         // Package name as title
         String packageName = notification.getPackageName();
-        ReflexPlugin.debugPrint("Notification Received From: " + packageName);
 
         // Extra Payload
         Bundle extras = notification.getNotification().extras;
@@ -70,6 +68,7 @@ public void onNotificationPosted(StatusBarNotification notification) {
         // Sending AutoReply
         if(NotificationUtils.canReply(notification))
         {
+            final Map<String, Object> autoReply = ReflexPlugin.autoReply;
             String message = (String) autoReply.get("message");
             // Reply to notification
             new AutoReply(ReflexPlugin.context).sendReply(notification, packageName, title, message);

android/src/main/java/com/devsonflutter/reflex/notification/NotificationReceiver.java

@@ -63,6 +63,8 @@ else if(packageNameList != null) {
             data.put("title", title);
             data.put("message", message);
 
+            ReflexPlugin.debugPrint("Notification Received From: " + packageName);
+
             eventSink.success(data);
         }
     }

android/src/main/java/com/devsonflutter/reflex/notification/NotificationUtils.java

@@ -37,8 +37,6 @@ public class NotificationUtils {
     public static String NOTIFICATION_MESSAGE = "notification_message";
     public static String NOTIFICATION_TITLE = "notification_title";
 
-    private static final Map<String, Object> autoReply = ReflexPlugin.autoReply;
-
     private static final List<String> listeningPackageNameList = ReflexPlugin.packageNameList;
     private static final List<String> listeningExceptionPackageNameList = ReflexPlugin.packageNameExceptionList;
 
@@ -114,47 +112,50 @@ public static String getTitleRaw(StatusBarNotification sbn) {
     static boolean canReply(StatusBarNotification notification) {
         String notificationPackageName = notification.getPackageName();
 
-        boolean isServiceEnabled = false;
         List<String> autoReplyPackageNameList = null;
 
+        final Map<String, Object> autoReply = ReflexPlugin.autoReply;
+
         if(autoReply != null) {
-            isServiceEnabled = true;
+
             autoReplyPackageNameList = (List<String>) autoReply.get("packageNameList");
+
+            return isSupportedPackage(notificationPackageName,autoReplyPackageNameList) &&
+                    checkListeningAndReplyPackages(notificationPackageName,autoReplyPackageNameList) &&
+                    NotificationUtils.isNewNotification(notification);
         }
 
         // If AutoReply object coming from flutter side is null, AutoReply feature is disabled
-        if (!isServiceEnabled) return false;
-
-        return isSupportedPackage(notificationPackageName,autoReplyPackageNameList) &&
-                checkListeningAndReplyPackages(notificationPackageName,autoReplyPackageNameList) &&
-                NotificationUtils.isNewNotification(notification);
+        return false;
     }
 
     private static boolean checkListeningAndReplyPackages(String notificationPackageName,
                                                           List<String> replyPackageNameList) {
-        if(listeningPackageNameList == null && listeningExceptionPackageNameList == null && replyPackageNameList == null){
-            return true;
-        }else if(listeningPackageNameList == null && listeningExceptionPackageNameList == null){
-            return  replyPackageNameList.contains(notificationPackageName);
-        }
-        else if(listeningPackageNameList == null ){
 
-            return  !listeningExceptionPackageNameList.contains(notificationPackageName);
-        }
-        else {
+        if (listeningPackageNameList == null && listeningExceptionPackageNameList == null &&
+                replyPackageNameList == null) {
             return true;
+        } else if (listeningExceptionPackageNameList == null && listeningPackageNameList == null) {
+            return replyPackageNameList.contains(notificationPackageName);
+        } else if (listeningPackageNameList == null && replyPackageNameList == null) {
+            return !listeningExceptionPackageNameList.contains(notificationPackageName);
+        } else if (listeningPackageNameList == null) {
+            return replyPackageNameList.contains(notificationPackageName);
         }
+
+        return listeningPackageNameList.contains(notificationPackageName);
     }
 
     @RequiresApi(api = Build.VERSION_CODES.JELLY_BEAN_MR2)
-    private static boolean isSupportedPackage(String notificationPackageName, List<String> packageList) {
+    private static boolean isSupportedPackage(String notificationPackageName,
+                                              List<String> replyPackageNameList) {
         // If packageNameList coming from flutter is null,
         // then AutoReply is enabled for all packageNames
-        if(packageList == null) {
+        if(replyPackageNameList == null) {
             return true;
         }
 
         // Check notification's package name contained in AutoReply's PackageNameList
-        return packageList.contains(notificationPackageName);
+        return replyPackageNameList.contains(notificationPackageName);
     }
 }

example/lib/main.dart

@@ -25,9 +25,9 @@ class _MyAppState extends State<MyApp> {
   Reflex reflex = Reflex(
     debug: true,
     packageNameList: ["com.whatsapp", "com.tyup"],
-    packageNameExceptionList: ["com.android.systemui"],
+    packageNameExceptionList: ["com.facebook"],
     autoReply: AutoReply(
-      packageNameList: ["com.whatsapp"],
+      // packageNameList: ["com.whatsapp"],
       message: "[Reflex] This is an automated reply.",
     ),
   );