Store email cache by AccountId

(cherry picked from commit 9b87da2990cef051c93eaab692aa2fa360ad9af2)

lib/features/thread/data/datasource/thread_datasource.dart

@@ -38,7 +38,7 @@ abstract class ThreadDataSource {
 
   Future<List<Email>> getAllEmailCache({MailboxId? inMailboxId, Set<Comparator>? sort, FilterMessageOption? filterOption, UnsignedInt? limit});
 
-  Future<void> update({List<Email>? updated, List<Email>? created, List<EmailId>? destroyed});
+  Future<void> update(AccountId accountId, {List<Email>? updated, List<Email>? created, List<EmailId>? destroyed});
 
   Future<List<EmailId>> emptyTrashFolder(
     Session session,

lib/features/thread/data/datasource_impl/local_thread_datasource_impl.dart

@@ -68,13 +68,15 @@ class LocalThreadDataSourceImpl extends ThreadDataSource {
   }
 
   @override
-  Future<void> update({
+  Future<void> update(
+    AccountId accountId, {
     List<Email>? updated,
     List<Email>? created,
     List<EmailId>? destroyed
   }) {
     return Future.sync(() async {
       return await _emailCacheManager.update(
+        accountId,
         updated: updated,
         created: created,
         destroyed: destroyed);

lib/features/thread/data/datasource_impl/thread_datasource_impl.dart

@@ -79,7 +79,7 @@ class ThreadDataSourceImpl extends ThreadDataSource {
   }
 
   @override
-  Future<void> update({List<Email>? updated, List<Email>? created, List<EmailId>? destroyed}) {
+  Future<void> update(AccountId accountId, {List<Email>? updated, List<Email>? created, List<EmailId>? destroyed}) {
     throw UnimplementedError();
   }
 

lib/features/thread/data/extensions/list_email_extension.dart

@@ -0,0 +1,17 @@
+
+import 'package:jmap_dart_client/jmap/account_id.dart';
+import 'package:jmap_dart_client/jmap/mail/email/email.dart';
+import 'package:model/extensions/account_id_extensions.dart';
+import 'package:model/extensions/email_id_extensions.dart';
+import 'package:tmail_ui_user/features/caching/utils/cache_utils.dart';
+import 'package:tmail_ui_user/features/thread/data/extensions/email_extension.dart';
+import 'package:tmail_ui_user/features/thread/data/model/email_cache.dart';
+
+extension ListEmailExtension on List<Email> {
+  Map<String, EmailCache> toMapCache(AccountId accountId) {
+    return {
+      for (var email in this)
+        TupleKey(email.id!.asString, accountId.asString).toString() : email.toEmailCache()
+    };
+  }
+}

lib/features/thread/data/extensions/list_email_id_extension.dart

@@ -0,0 +1,10 @@
+
+import 'package:jmap_dart_client/jmap/account_id.dart';
+import 'package:jmap_dart_client/jmap/mail/email/email.dart';
+import 'package:model/extensions/account_id_extensions.dart';
+import 'package:model/extensions/email_id_extensions.dart';
+import 'package:tmail_ui_user/features/caching/utils/cache_utils.dart';
+
+extension ListEmailIdExtension on List<EmailId> {
+  List<String> toCacheKeyList(AccountId accountId) => map((id) => TupleKey(id.asString, accountId.asString).toString()).toList();
+}

lib/features/thread/data/local/email_cache_manager.dart

@@ -1,4 +1,4 @@
-import 'package:core/core.dart';
+import 'package:jmap_dart_client/jmap/account_id.dart';
 import 'package:jmap_dart_client/jmap/core/unsigned_int.dart';
 import 'package:model/model.dart';
 import 'package:jmap_dart_client/jmap/mail/email/email.dart';
@@ -6,9 +6,8 @@ import 'package:jmap_dart_client/jmap/mail/mailbox/mailbox.dart';
 import 'package:tmail_ui_user/features/caching/email_cache_client.dart';
 import 'package:tmail_ui_user/features/cleanup/domain/model/email_cleanup_rule.dart';
 import 'package:tmail_ui_user/features/thread/data/extensions/email_cache_extension.dart';
-import 'package:tmail_ui_user/features/thread/data/extensions/email_extension.dart';
+import 'package:tmail_ui_user/features/thread/data/extensions/list_email_extension.dart';
-import 'package:tmail_ui_user/features/thread/data/extensions/list_email_cache_extension.dart';
+import 'package:tmail_ui_user/features/thread/data/extensions/list_email_id_extension.dart';
-import 'package:tmail_ui_user/features/thread/data/model/email_cache.dart';
 import 'package:jmap_dart_client/jmap/core/sort/comparator.dart';
 import 'package:tmail_ui_user/features/thread/domain/model/filter_message_option.dart';
 
@@ -43,28 +42,26 @@ class EmailCacheManager {
     return emailList;
   }
 
-  Future<void> update({List<Email>? updated, List<Email>? created, List<EmailId>? destroyed}) async {
+  Future<void> update(
+    AccountId accountId, {
+    List<Email>? updated,
+    List<Email>? created,
+    List<EmailId>? destroyed
+  }) async {
     final emailCacheExist = await _emailCacheClient.isExistTable();
-    log('EmailCacheManager::update(): emailCacheExist: $emailCacheExist');
     if (emailCacheExist) {
-      final updatedCacheEmails = updated
+      final updatedCacheEmails = updated?.toMapCache(accountId) ?? {};
-        ?.map((email) => email.toEmailCache()).toList() ?? <EmailCache>[];
+      final createdCacheEmails = created?.toMapCache(accountId) ?? {};
-      final createdCacheEmails = created
+      final destroyedCacheEmails = destroyed?.toCacheKeyList(accountId) ?? [];
-        ?.map((email) => email.toEmailCache()).toList() ?? <EmailCache>[];
-      final destroyedCacheEmails = destroyed
-        ?.map((emailId) => emailId.id.value).toList() ?? <String>[];
-
-      log('EmailCacheManager::update(): destroyedCacheEmails: ${destroyedCacheEmails.length}');
 
       await Future.wait([
-        _emailCacheClient.updateMultipleItem(updatedCacheEmails.toMap()),
+        _emailCacheClient.updateMultipleItem(updatedCacheEmails),
-        _emailCacheClient.insertMultipleItem(createdCacheEmails.toMap()),
+        _emailCacheClient.insertMultipleItem(createdCacheEmails),
         _emailCacheClient.deleteMultipleItem(destroyedCacheEmails)
       ]);
     } else {
-      final createdCacheEmails = created
+      final createdCacheEmails = created?.toMapCache(accountId) ?? {};
-        ?.map((email) => email.toEmailCache()).toList() ?? <EmailCache>[];
+      await _emailCacheClient.insertMultipleItem(createdCacheEmails);
-      await _emailCacheClient.insertMultipleItem(createdCacheEmails.toMap());
     }
   }
 

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

@@ -81,7 +81,7 @@ class ThreadRepositoryImpl extends ThreadRepository {
     }
 
     if (networkEmailResponse != null) {
-      await _updateEmailCache(newCreated: networkEmailResponse.emailList);
+      await _updateEmailCache(accountId, newCreated: networkEmailResponse.emailList);
     }
 
     if (localEmailResponse.hasState()) {
@@ -138,7 +138,7 @@ class ThreadRepositoryImpl extends ThreadRepository {
         filter: filter ?? EmailFilterCondition(inMailbox: mailboxId),
         properties: propertiesCreated,
       );
-      await _updateEmailCache(newCreated: networkEmailResponse.emailList);
+      await _updateEmailCache(accountId, newCreated: networkEmailResponse.emailList);
 
       return networkEmailResponse;
   }
@@ -176,12 +176,14 @@ class ThreadRepositoryImpl extends ThreadRepository {
     }
   }
 
-  Future<void> _updateEmailCache({
+  Future<void> _updateEmailCache(
+    AccountId accountId, {
     List<Email>? newUpdated,
     List<Email>? newCreated,
     List<EmailId>? newDestroyed
   }) async {
     await mapDataSource[DataSourceType.local]!.update(
+      accountId,
       updated: newUpdated,
       created: newCreated,
       destroyed: newDestroyed);
@@ -240,7 +242,7 @@ class ThreadRepositoryImpl extends ThreadRepository {
   @override
   Stream<EmailsResponse> loadMoreEmails(GetEmailRequest emailRequest) async* {
     final response = await _getAllEmailsWithoutLastEmailId(emailRequest);
-    await _updateEmailCache(newCreated: response.emailList);
+    await _updateEmailCache(emailRequest.accountId, newCreated: response.emailList);
     yield response;
   }
 
@@ -290,7 +292,7 @@ class ThreadRepositoryImpl extends ThreadRepository {
       accountId,
       trashMailboxId,
       (listEmailIdDeleted) async {
-        await _updateEmailCache(newDestroyed: listEmailIdDeleted);
+        await _updateEmailCache(accountId, newDestroyed: listEmailIdDeleted);
       },
     );
   }
@@ -342,6 +344,7 @@ class ThreadRepositoryImpl extends ThreadRepository {
           'destroyed = ${emailChangeResponse.destroyed?.length}');
 
       await _updateEmailCache(
+          accountId,
           newCreated: emailChangeResponse.created,
           newUpdated: newEmailUpdated,
           newDestroyed: emailChangeResponse.destroyed);