TF-1360 Fix can not load new email until reload mail list

lib/features/mailbox_dashboard/presentation/controller/mailbox_dashboard_controller.dart

@@ -165,6 +165,9 @@ class MailboxDashBoardController extends ReloadableController {
   final StreamController<RefreshActionViewEvent> _refreshActionEventController =
     StreamController<RefreshActionViewEvent>.broadcast();
 
+  final _notificationManager = LocalNotificationManager.instance;
+  final _fcmController = FcmController.instance;
+
   MailboxDashBoardController(
     LogoutOidcInteractor logoutOidcInteractor,
     DeleteAuthorityOidcInteractor deleteAuthorityOidcInteractor,
@@ -197,11 +200,6 @@ class MailboxDashBoardController extends ReloadableController {
     _registerPendingFileInfo();
     _getSessionCurrent();
     _getAppVersion();
-    if (!BuildUtils.isWeb) {
-      _handleOpenAppByNotification();
-    } else {
-      dispatchRoute(DashboardRoutes.thread);
-    }
     super.onReady();
   }
 
@@ -272,12 +270,13 @@ class MailboxDashBoardController extends ReloadableController {
           clearState();
         } else if (failure is SaveEmailAsDraftsFailure
             || failure is RemoveEmailDraftsFailure
-            || failure is UpdateEmailDraftsFailure
-            || failure is GetEmailByIdFailure) {
+            || failure is UpdateEmailDraftsFailure) {
           clearState();
         } else if (failure is MarkAsMailboxReadAllFailure ||
             failure is MarkAsMailboxReadFailure) {
           _markAsReadMailboxFailure(failure);
+        } else if (failure is GetEmailByIdFailure) {
+          _handleGetEmailDetailedFailed(failure);
         }
       },
       (success) {
@@ -367,16 +366,21 @@ class MailboxDashBoardController extends ReloadableController {
     _refreshActionEventController.stream
       .debounceTime(const Duration(milliseconds: FcmService.durationMessageComing))
       .listen(_handleRefreshActionWhenBackToApp);
-    
-    LocalNotificationManager.instance.localNotificationStream.listen(_handleLocalNotificationResponse);
+
+    _notificationManager.localNotificationStream.listen(_handleClickLocalNotificationOnForeground);
   }
 
-  void _handleOpenAppByNotification() async {
-    log("_handleOpenAppByNotification(): isOpenAppByNotification: ${LocalNotificationManager.instance.isOpenAppByNotification}");
-    await LocalNotificationManager.instance.setUp();
-    if (LocalNotificationManager.instance.isOpenAppByNotification) {
-      final emailId = EmailId(Id(LocalNotificationManager.instance.getEmailIdFromNotification()!));
-      _getPresentationEmailFromEmailId(emailId);
+  void _handleClickLocalNotificationOnTerminated() async {
+    await _notificationManager.getNotificationDetails();
+    final notificationResponse = _notificationManager.currentNotificationResponse;
+    log('MailboxDashBoardController::_handleClickLocalNotificationOnTerminated():payload: ${notificationResponse?.payload}');
+    final emailIdFromPayload = notificationResponse?.payload;
+    final currentAccountId = accountId.value;
+    if (emailIdFromPayload?.isNotEmpty == true && currentAccountId != null) {
+      _getPresentationEmailFromEmailIdAction(
+        EmailId(Id(emailIdFromPayload!)),
+        currentAccountId
+      );
     } else {
       dispatchRoute(DashboardRoutes.thread);
     }
@@ -387,6 +391,9 @@ class MailboxDashBoardController extends ReloadableController {
   }
 
   void _getSessionCurrent() {
+    if (BuildUtils.isWeb) {
+      dispatchRoute(DashboardRoutes.thread);
+    }
     final arguments = Get.arguments;
     log('MailboxDashBoardController::_getSessionCurrent(): arguments = $arguments');
     if (arguments is Session) {
@@ -398,6 +405,10 @@ class MailboxDashBoardController extends ReloadableController {
       injectVacationBindings(sessionCurrent, accountId.value);
       injectFCMBindings(sessionCurrent, accountId.value);
       _getVacationResponse();
+
+      if (!BuildUtils.isWeb) {
+        _handleClickLocalNotificationOnTerminated();
+      }
     } else {
       reload();
     }
@@ -1371,36 +1382,62 @@ class MailboxDashBoardController extends ReloadableController {
     }
   }
 
-  Future<bool> haveLocalNotificationPress() async {
-    return !(await LocalNotificationManager.instance.localNotificationStream.isEmpty);
-  }
+  void _handleClickLocalNotificationOnForeground(NotificationResponse? response) {
+    log('MailboxDashBoardController::_handleClickLocalNotificationOnForeground():payload: ${response?.payload}');
+    final emailIdFromPayload = response?.payload;
+    final currentAccountId = accountId.value;
+    if (emailIdFromPayload?.isNotEmpty == true && currentAccountId != null) {
+      popAllRouteIfHave();
+      dispatchRoute(DashboardRoutes.waiting);
 
-  void _handleLocalNotificationResponse(NotificationResponse response) {
-    if(response.payload != null) {
-      final emailId = EmailId(Id(response.payload!));
-      _getPresentationEmailFromEmailId(emailId);
+      _getPresentationEmailFromEmailIdAction(
+        EmailId(Id(emailIdFromPayload!)),
+        currentAccountId
+      );
+    } else {
+      dispatchRoute(DashboardRoutes.thread);
     }
   }
 
-  void _getPresentationEmailFromEmailId(EmailId emailId) {
-    if(accountId.value != null) {
-      log('MailboxDashBoardController: _getPresentationEmailFromEmailId: $emailId');
-      consumeState(_getEmailByIdInteractor.execute(accountId.value!, emailId, properties: ThreadConstants.propertiesDefault));
-    }
+  void _getPresentationEmailFromEmailIdAction(EmailId emailId, AccountId accountId) {
+    log('MailboxDashBoardController:_getPresentationEmailFromEmailIdAction:emailId: $emailId');
+    consumeState(_getEmailByIdInteractor.execute(
+      accountId,
+      emailId,
+      properties: ThreadConstants.propertiesDefault
+    ));
   }
 
   void _moveToEmailDetailedView(GetEmailByIdSuccess success) {
     log('MailboxDashBoardController::_moveToEmailDetailedView(): ${success.email}');
     setSelectedEmail(success.email);
     dispatchRoute(DashboardRoutes.emailDetailed);
-    // move to inbox so that user can back to inbox view
-    dispatchAction(ClearSearchEmailAction());
+  }
+
+  void _handleGetEmailDetailedFailed(GetEmailByIdFailure failure) {
+    logError('MailboxDashBoardController::_handleGetEmailDetailedFailed(): $failure');
+    dispatchRoute(DashboardRoutes.thread);
   }
 
   void popAllRouteIfHave() {
     Get.until((route) => Get.currentRoute == AppRoutes.dashboard);
   }
-  
+
+  void handleOnFocusGained() {
+    log('MailboxDashBoardController::handleOnFocusGained():');
+    if (!BuildUtils.isWeb) {
+      _updateTheme();
+    }
+    refreshActionWhenBackToApp();
+  }
+
+  void _updateTheme() {
+    ThemeUtils.setSystemDarkUIStyle();
+    if (isDrawerOpen) {
+      ThemeUtils.setStatusBarTransparentColor();
+    }
+  }
+
   @override
   void onClose() {
     _emailReceiveManager.closeEmailReceiveManagerStream();
@@ -1409,8 +1446,8 @@ class MailboxDashBoardController extends ReloadableController {
     _progressStateController.close();
     _refreshActionEventController.close();
     Get.delete<DownloadController>();
-    FcmController.instance.dispose();
-    LocalNotificationManager.instance.dispose();
+    _notificationManager.dispose();
+    _fcmController.dispose();
     super.onClose();
   }
 }

lib/features/mailbox_dashboard/presentation/mailbox_dashboard_view.dart

@@ -1,6 +1,5 @@
 import 'package:core/presentation/extensions/color_extension.dart';
 import 'package:core/presentation/utils/responsive_utils.dart';
-import 'package:core/presentation/utils/theme_utils.dart';
 import 'package:core/presentation/views/responsive/responsive_widget.dart';
 import 'package:flutter/cupertino.dart';
 import 'package:flutter/material.dart';
@@ -30,17 +29,7 @@ class MailboxDashBoardView extends BaseMailboxDashBoardView {
     );
 
     return FocusDetector(
-      onFocusGained: () async {
-        ThemeUtils.setSystemDarkUIStyle();
-        if (controller.isDrawerOpen) {
-          ThemeUtils.setStatusBarTransparentColor();
-        }
-        if(await controller.haveLocalNotificationPress()) {
-          controller.popAllRouteIfHave();
-          controller.dispatchRoute(DashboardRoutes.waiting);
-        }
-        controller.refreshActionWhenBackToApp();
-      },
+      onFocusGained: controller.handleOnFocusGained,
       child: Scaffold(
         drawerEnableOpenDragGesture: responsiveUtils.hasLeftMenuDrawerActive(context),
         body: Obx(() {

lib/features/push_notification/presentation/notification/local_notification_manager.dart

@@ -6,7 +6,7 @@ import 'package:core/presentation/extensions/html_extension.dart';
 import 'package:core/utils/app_logger.dart';
 import 'package:flutter_local_notifications/flutter_local_notifications.dart';
 import 'package:jmap_dart_client/jmap/mail/email/email_address.dart';
-import 'package:model/model.dart';
+import 'package:model/extensions/email_address_extension.dart';
 import 'package:tmail_ui_user/features/push_notification/presentation/notification/local_notification_config.dart';
 
 class LocalNotificationManager {
@@ -21,14 +21,13 @@ class LocalNotificationManager {
   bool notificationsEnabled = false;
 
   final StreamController<NotificationResponse> localNotificationsController = StreamController<NotificationResponse>.broadcast();
-  
+
   Stream<NotificationResponse> get localNotificationStream => localNotificationsController.stream;
 
   NotificationAppLaunchDetails? _notificationAppLaunchDetails;
 
   Future<void> setUp() async {
     try {
-      _notificationAppLaunchDetails = await _localNotificationsPlugin.getNotificationAppLaunchDetails();
       await _initLocalNotification();
       _checkLocalNotificationPermission();
       if (Platform.isAndroid) {
@@ -41,19 +40,17 @@ class LocalNotificationManager {
     return Future.value();
   }
 
-  bool get isOpenAppByNotification => _notificationAppLaunchDetails?.didNotificationLaunchApp ?? false;
-
-  bool get isLocalNotificationStreamClosed => localNotificationsController.isClosed;
-
-  String? getEmailIdFromNotification() {
-    if(_notificationAppLaunchDetails?.didNotificationLaunchApp ?? false) {
-      log('LocalNotificationManager::handleNotificationAppLaunch(): ${_notificationAppLaunchDetails!.notificationResponse}');
-      final selectedNotificationPayload = _notificationAppLaunchDetails!.notificationResponse?.payload;
-      return selectedNotificationPayload;
+  Future<void> getNotificationDetails() async {
+    try {
+      _notificationAppLaunchDetails = await _localNotificationsPlugin.getNotificationAppLaunchDetails();
+    } catch (e) {
+      logError('LocalNotificationManager::getNotificationDetails(): ERROR: ${e.toString()}');
     }
-    return null;
+    return Future.value();
   }
 
+  NotificationResponse? get currentNotificationResponse => _notificationAppLaunchDetails?.notificationResponse;
+
   Future<bool?> _initLocalNotification() async {
     return await _localNotificationsPlugin.initialize(
       const InitializationSettings(
@@ -65,8 +62,8 @@ class LocalNotificationManager {
   }
 
   void _handleReceiveNotificationResponse(NotificationResponse response) {
-    log('LocalNotificationManager::handleReceiveNotificationResponse(): ${response.payload}');
-    if(response.notificationResponseType == NotificationResponseType.selectedNotification) {
+    log('LocalNotificationManager::handleReceiveNotificationResponse():payload: ${response.payload}');
+    if (response.notificationResponseType == NotificationResponseType.selectedNotification) {
       localNotificationsController.add(response);
     }
   }