TF-4274 Avoid clearing all caches on refresh for non‑web platforms

docs/adr/0070-sync-strategy-disappearing-emails.md

@@ -20,7 +20,7 @@ The issue typically manifests in the following sequence, as reported in [Issue #
 * **Crucially:** The email *can* be found using the Search function (proving data existence).
 
 
-4. **Resolution:** The user is forced to **Logout and Login**, which magically restores the email to the list.
+4. **Resolution:** The user is forced to **log out and log in**, which magically restores the email to the list.
 
 ### Root Cause Analysis
 
@@ -104,4 +104,4 @@ FORCE_EMAIL_QUERY=false
 
 ## References
 
-* **Github Issue:** [linagora/tmail-flutter#4274](https://github.com/linagora/tmail-flutter/issues/4274)
+* **GitHub Issue:** [linagora/tmail-flutter#4274](https://github.com/linagora/tmail-flutter/issues/4274)

lib/features/base/base_controller.dart

@@ -506,7 +506,7 @@ abstract class BaseController extends GetxController
     if (_isFcmEnabled) {
       await _handleDeleteFCMRegistration();
     }
-    clearAllData();
+    await clearAllData();
   }
 
   Future<void> _handleDeleteFCMRegistration() async {

lib/features/caching/caching_manager.dart

@@ -70,7 +70,7 @@ class CachingManager {
       await Future.wait([
         clearMailDataCached(),
         clearAccountDataCached(),
-      ], eagerError: true);
+      ]);
     } catch (e) {
       logWarning('CachingManager::clearAll: Cannot clear all cache: $e');
     }

lib/features/login/presentation/login_controller.dart

@@ -254,7 +254,11 @@ class LoginController extends ReloadableController {
     SmartDialog.dismiss();
 
     if (PlatformInfo.isWeb) {
-      await cachingManager.clearAllEmailAndStateCache();
+      try {
+        await cachingManager.clearAllEmailAndStateCache();
+      } catch (e) {
+        logWarning('$runtimeType::handleReloaded: Failed to clear cache: $e');
+      }
     }
 
     popAndPush(

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

@@ -611,14 +611,16 @@ class MailboxDashBoardController extends ReloadableController
   @override
   Future<void> onBeforeUnloadBrowserListener(html.Event event) async {
     log('MailboxDashBoardController::onBeforeUnloadBrowserListener:event = ${event.runtimeType} | hasComposer = ${twakeAppManager.hasComposer} | isExecutingBeforeReconnect = ${twakeAppManager.isExecutingBeforeReconnect}');
-    if (PlatformInfo.isWeb) {
+    final shouldPrevent = event is html.BeforeUnloadEvent &&
-      await cachingManager.clearAllEmailAndStateCache();
+        twakeAppManager.hasComposer &&
+        !twakeAppManager.isExecutingBeforeReconnect;
+
+    if (shouldPrevent) {
+      event.preventDefault();
     }
 
-    if (event is html.BeforeUnloadEvent &&
+    if (PlatformInfo.isWeb) {
-        twakeAppManager.hasComposer &&
+      await cachingManager.clearAllEmailAndStateCache();
-        !twakeAppManager.isExecutingBeforeReconnect) {
-      event.preventDefault();
     }
   }
 

lib/features/thread/presentation/thread_controller.dart

@@ -355,7 +355,7 @@ class ThreadController extends BaseController with EmailActionController {
       if (action is RefreshChangeEmailAction) {
         _refreshEmailChanges(newState: action.newState);
       } else if (action is RefreshAllEmailAction) {
-        refreshAllEmail(forceEmailQuery: true);
+        refreshAllEmail(shouldClearCache: PlatformInfo.isWeb);
         mailboxDashBoardController.clearEmailUIAction();
       }
     });
@@ -604,8 +604,8 @@ class ThreadController extends BaseController with EmailActionController {
     }
   }
 
-  Future<void> refreshAllEmail({bool forceEmailQuery = false}) async {
+  Future<void> refreshAllEmail({bool shouldClearCache = false}) async {
-    if (forceEmailQuery) {
+    if (shouldClearCache) {
       await cachingManager.clearAllEmailAndStateCache();
     }
 
@@ -622,7 +622,7 @@ class ThreadController extends BaseController with EmailActionController {
     if (searchController.isSearchEmailRunning) {
       _searchEmail(limit: limitEmailFetched);
     } else {
-      getAllEmailAction(forceEmailQuery: forceEmailQuery);
+      getAllEmailAction();
     }
   }