Use default properties when update email from email changed

Signed-off-by: dab246 <tdvu@linagora.com>

lib/features/thread/data/repository/thread_repository_impl.dart

@@ -176,32 +176,40 @@ class ThreadRepositoryImpl extends ThreadRepository {
     Properties? updatedProperties,
     List<Email>? emailCacheList
   }) async {
-    if (emailUpdated != null && emailUpdated.isNotEmpty) {
-      if (updatedProperties == null) {
-        return null;
-      }
-      final newEmailUpdated = emailUpdated
-        .map((updatedEmail) => _combineUpdatedWithEmailInCache(updatedEmail, emailCacheList))
-        .where((tuple) => tuple.value2 != null)
-        .map((tuple) => tuple.value2!.combineEmail(tuple.value1, updatedProperties))
+    if (emailUpdated == null || emailUpdated.isEmpty) return emailUpdated;
+
+    if (updatedProperties == null) return null;
+
+    if (updatedProperties.value.containsAll(ThreadConstants.propertiesDefault.value)) {
+      log('ThreadRepositoryImpl::_combineEmailCache(): Update use properties default');
+      return emailUpdated;
+    }
+
+    final combinedEmails = emailUpdated
+        .map((email) => _combineUpdatedWithEmailInCache(email, emailCacheList))
+        .where((record) => record.oldEmail != null)
+        .map((record) => record.oldEmail!.combineEmail(
+          record.updatedEmail,
+          updatedProperties,
+        ))
         .toList();
 
-      return newEmailUpdated;
-    }
-    return emailUpdated;
+    return combinedEmails;
   }
 
-  dartz.Tuple2<Email, Email?> _combineUpdatedWithEmailInCache(Email updatedEmail, List<Email>? emailCacheList) {
-    final emailOld = updatedEmail.id != null
-      ? emailCacheList?.findEmailById(updatedEmail.id!)
-      : null;
-    if (emailOld != null) {
-      log('ThreadRepositoryImpl::_combineUpdatedWithEmailInCache(): cache hit');
-      return dartz.Tuple2(updatedEmail, emailOld);
+  ({Email updatedEmail, Email? oldEmail}) _combineUpdatedWithEmailInCache(
+    Email updatedEmail,
+    List<Email>? emailCacheList,
+  ) {
+    final oldEmail = updatedEmail.id != null
+        ? emailCacheList?.findEmailById(updatedEmail.id!)
+        : null;
+    if (oldEmail != null) {
+      log('ThredRepositoryImpl::_combineUpdatedWithEmailInCache(): cache hit for this email -> ${oldEmail.id} - ${oldEmail.subject} - ${oldEmail.keywords} - ${oldEmail.mailboxIds} - new update in $updatedEmail');
     } else {
-      log('ThreadRepositoryImpl::_combineUpdatedWithEmailInCache(): cache miss');
-      return dartz.Tuple2(updatedEmail, null);
+      log('ThreadRepositoryImpl::_combineUpdatedWithEmailInCache(): cache miss for emailId ${updatedEmail.id}');
     }
+    return (oldEmail: oldEmail, updatedEmail: updatedEmail);
   }
 
   Future<void> _updateEmailCache(

lib/features/thread/domain/constants/thread_constants.dart

@@ -26,10 +26,7 @@ class ThreadConstants {
     IndividualHeaderIdentifier.importanceHeader.value,
     IndividualHeaderIdentifier.priorityHeader.value,
   });
-  static final propertiesUpdatedDefault = Properties({
-    EmailProperty.keywords,
-    EmailProperty.mailboxIds,
-  });
+  static final propertiesUpdatedDefault = propertiesDefault;
 
   static final propertiesGetEmailContent = Properties({
     EmailProperty.bodyValues,

lib/features/thread/presentation/extensions/handle_pull_to_refresh_list_email_extension.dart

@@ -48,7 +48,10 @@ extension HandlePullToRefreshListEmailExtension on ThreadController {
         mailboxId: selectedMailboxId,
       ),
       propertiesCreated: EmailUtils.getPropertiesForEmailGetMethod(session, accountId),
-      propertiesUpdated: ThreadConstants.propertiesUpdatedDefault,
+      propertiesUpdated: EmailUtils.getPropertiesForEmailChangeMethod(
+        session,
+        accountId,
+      ),
       getLatestChanges: true,
     ));
   }

lib/features/thread/presentation/thread_controller.dart

@@ -554,7 +554,10 @@ class ThreadController extends BaseController with EmailActionController {
           mailboxId: selectedMailboxId
         ),
         propertiesCreated: EmailUtils.getPropertiesForEmailGetMethod(_session!, _accountId!),
-        propertiesUpdated: ThreadConstants.propertiesUpdatedDefault,
+        propertiesUpdated: EmailUtils.getPropertiesForEmailChangeMethod(
+          _session!,
+          _accountId!,
+        ),
         getLatestChanges: getLatestChanges,
       ));
     } else {
@@ -709,7 +712,10 @@ class ThreadController extends BaseController with EmailActionController {
         _session!,
         _accountId!,
       ),
-      propertiesUpdated: ThreadConstants.propertiesUpdatedDefault,
+      propertiesUpdated: EmailUtils.getPropertiesForEmailChangeMethod(
+        _session!,
+        _accountId!,
+      ),
       emailFilter: EmailFilter(
         filter: getFilterCondition(mailboxIdSelected: selectedMailboxId),
         filterOption: mailboxDashBoardController.filterMessageOption.value,