TF-599 Handle mark as read action for mailbox

2022-06-01 15:37:07 +07:00
parent 3b3da513e4
commit 30b7d046f9
16 changed files with 301 additions and 48 deletions
@@ -3,8 +3,17 @@ import 'package:jmap_dart_client/jmap/account_id.dart';
 import 'package:jmap_dart_client/jmap/core/properties/properties.dart';
 import 'package:jmap_dart_client/jmap/core/session/session.dart';
 import 'package:jmap_dart_client/jmap/core/state.dart';
+import 'package:jmap_dart_client/jmap/core/unsigned_int.dart';
+import 'package:jmap_dart_client/jmap/core/utc_date.dart';
+import 'package:jmap_dart_client/jmap/mail/email/email.dart';
+import 'package:jmap_dart_client/jmap/mail/email/email_comparator.dart';
+import 'package:jmap_dart_client/jmap/mail/email/email_comparator_property.dart';
+import 'package:jmap_dart_client/jmap/mail/email/email_filter_condition.dart';
+import 'package:jmap_dart_client/jmap/mail/email/keyword_identifier.dart';
 import 'package:jmap_dart_client/jmap/mail/mailbox/mailbox.dart';
+import 'package:jmap_dart_client/jmap/core/sort/comparator.dart';
 import 'package:model/model.dart';
+import 'package:tmail_ui_user/features/email/data/datasource/email_datasource.dart';
 import 'package:tmail_ui_user/features/mailbox/data/datasource/mailbox_datasource.dart';
 import 'package:tmail_ui_user/features/mailbox/data/datasource/state_datasource.dart';
 import 'package:tmail_ui_user/features/mailbox/data/extensions/state_extension.dart';
@@ -13,13 +22,22 @@ import 'package:tmail_ui_user/features/mailbox/domain/model/create_new_mailbox_r
 import 'package:tmail_ui_user/features/mailbox/domain/model/mailbox_response.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/rename_mailbox_request.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/repository/mailbox_repository.dart';
+import 'package:tmail_ui_user/features/thread/data/datasource/thread_datasource.dart';
+import 'package:tmail_ui_user/features/thread/domain/model/email_response.dart';

 class MailboxRepositoryImpl extends MailboxRepository {

  final Map<DataSourceType, MailboxDataSource> mapDataSource;
  final StateDataSource stateDataSource;
+  final ThreadDataSource threadDataSource;
+  final EmailDataSource emailDataSource;

-  MailboxRepositoryImpl(this.mapDataSource, this.stateDataSource);
+  MailboxRepositoryImpl(
+    this.mapDataSource,
+    this.stateDataSource,
+    this.threadDataSource,
+    this.emailDataSource
+  );

  @override
  Stream<MailboxResponse> getAllMailbox(AccountId accountId, {Properties? properties}) async* {
@@ -152,4 +170,54 @@ class MailboxRepositoryImpl extends MailboxRepository {
  Future<bool> renameMailbox(AccountId accountId, RenameMailboxRequest request) {
    return mapDataSource[DataSourceType.network]!.renameMailbox(accountId, request);
  }
+
+  @override
+  Future<bool> markAsMailboxRead(AccountId accountId, MailboxId mailboxId, MailboxName mailboxName) async {
+    bool mailboxHasEmails = true;
+    UTCDate? lastReceivedDate;
+    EmailId? lastEmailId;
+
+    while(mailboxHasEmails) {
+      final emailResponse = await threadDataSource.getAllEmail(accountId,
+        limit: UnsignedInt(20),
+        filter: EmailFilterCondition(
+            inMailbox: mailboxId,
+            notKeyword: KeyWordIdentifier.emailSeen.value,
+            before: lastReceivedDate),
+        sort: <Comparator>{}
+          ..add(EmailComparator(EmailComparatorProperty.receivedAt)
+            ..setIsAscending(false)),
+        properties: Properties({
+          EmailProperty.id,
+          EmailProperty.keywords,
+          EmailProperty.receivedAt,
+        })
+      ).then((response) {
+        var listEmails = response.emailList;
+        if (listEmails != null && listEmails.isNotEmpty && lastEmailId != null) {
+          listEmails = listEmails
+              .where((email) => email.id != lastEmailId)
+              .toList();
+        }
+        return EmailsResponse(emailList: listEmails, state: response.state);
+      });
+      log('MailboxRepositoryImpl::markAsMailboxRead(): listEmails: ${emailResponse.emailList?.length}');
+      final listEmailUnread = emailResponse.emailList;
+
+      if (listEmailUnread == null || listEmailUnread.isEmpty) {
+        mailboxHasEmails = false;
+      } else {
+        log('MailboxRepositoryImpl::markAsMailboxRead(): listEmailUnread: ${listEmailUnread.length}');
+        final result = await emailDataSource.markAsRead(accountId,
+            listEmailUnread, ReadActions.markAsRead);
+
+        if (result.length != listEmailUnread.length) {
+          mailboxHasEmails = false;
+          return false;
+        }
+      }
+    }
+
+    return true;
+  }
 }
@@ -17,4 +17,6 @@ abstract class MailboxRepository {
  Future<bool> deleteMultipleMailbox(Session session, AccountId accountId, List<MailboxId> mailboxIds);

  Future<bool> renameMailbox(AccountId accountId, RenameMailboxRequest request);
+
+  Future<bool> markAsMailboxRead(AccountId accountId, MailboxId mailboxId, MailboxName mailboxName);
 }
@@ -3,6 +3,9 @@ import 'package:get/get.dart';
 import 'package:shared_preferences/shared_preferences.dart';
 import 'package:tmail_ui_user/features/base/base_bindings.dart';
 import 'package:tmail_ui_user/features/caching/state_cache_client.dart';
+import 'package:tmail_ui_user/features/email/data/datasource/email_datasource.dart';
+import 'package:tmail_ui_user/features/email/data/datasource_impl/email_datasource_impl.dart';
+import 'package:tmail_ui_user/features/email/data/network/email_api.dart';
 import 'package:tmail_ui_user/features/login/data/repository/credential_repository_impl.dart';
 import 'package:tmail_ui_user/features/login/domain/repository/credential_repository.dart';
 import 'package:tmail_ui_user/features/mailbox/data/datasource/mailbox_datasource.dart';
@@ -23,6 +26,9 @@ import 'package:tmail_ui_user/features/mailbox/domain/usecases/search_mailbox_in
 import 'package:tmail_ui_user/features/mailbox/presentation/mailbox_controller.dart';
 import 'package:tmail_ui_user/features/mailbox/presentation/model/mailbox_tree_builder.dart';
 import 'package:tmail_ui_user/features/mailbox_creator/domain/usecases/verify_name_interactor.dart';
+import 'package:tmail_ui_user/features/thread/data/datasource/thread_datasource.dart';
+import 'package:tmail_ui_user/features/thread/data/datasource_impl/thread_datasource_impl.dart';
+import 'package:tmail_ui_user/features/thread/data/network/thread_api.dart';
 import 'package:uuid/uuid.dart';

 class MailboxBindings extends BaseBindings {
@@ -57,6 +63,8 @@ class MailboxBindings extends BaseBindings {
  void bindingsDataSource() {
    Get.lazyPut<MailboxDataSource>(() => Get.find<MailboxDataSourceImpl>());
    Get.lazyPut<StateDataSource>(() => Get.find<StateDataSourceImpl>());
+    Get.lazyPut<EmailDataSource>(() => Get.find<EmailDataSourceImpl>());
+    Get.lazyPut<ThreadDataSource>(() => Get.find<ThreadDataSourceImpl>());
  }

  @override
@@ -64,6 +72,8 @@ class MailboxBindings extends BaseBindings {
    Get.lazyPut(() => MailboxDataSourceImpl(Get.find<MailboxAPI>()));
    Get.lazyPut(() => MailboxCacheDataSourceImpl(Get.find<MailboxCacheManager>()));
    Get.lazyPut(() => StateDataSourceImpl(Get.find<StateCacheClient>()));
+    Get.lazyPut(() => EmailDataSourceImpl(Get.find<EmailAPI>()));
+    Get.lazyPut(() => ThreadDataSourceImpl(Get.find<ThreadAPI>()));
  }

  @override
@@ -92,6 +102,8 @@ class MailboxBindings extends BaseBindings {
        DataSourceType.local: Get.find<MailboxCacheDataSourceImpl>()
      },
      Get.find<StateDataSource>(),
+      Get.find<ThreadDataSource>(),
+      Get.find<EmailDataSource>(),
    ));
  }
 }
@@ -49,6 +49,7 @@ import 'package:tmail_ui_user/features/mailbox_creator/domain/usecases/verify_na
 import 'package:tmail_ui_user/features/mailbox_creator/presentation/extensions/validator_failure_extension.dart';
 import 'package:tmail_ui_user/features/mailbox_creator/presentation/model/mailbox_creator_arguments.dart';
 import 'package:tmail_ui_user/features/mailbox_creator/presentation/model/new_mailbox_arguments.dart';
+import 'package:tmail_ui_user/features/mailbox_dashboard/domain/state/mark_as_mailbox_read_state.dart';
 import 'package:tmail_ui_user/features/mailbox_dashboard/domain/state/remove_email_drafts_state.dart';
 import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/mailbox_dashboard_controller.dart';
 import 'package:tmail_ui_user/features/thread/domain/model/search_query.dart';
@@ -215,6 +216,7 @@ class MailboxController extends BaseMailboxController {
              success is SaveEmailAsDraftsSuccess ||
              success is RemoveEmailDraftsSuccess ||
              success is SendEmailSuccess ||
+              success is MarkAsMailboxReadAllSuccess ||
              success is UpdateEmailDraftsSuccess) {
            refreshMailboxChanges();
          }
@@ -478,6 +480,9 @@ class MailboxController extends BaseMailboxController {
      case MailboxActions.rename:
        _openDialogRenameMailboxAction(context, selectionMailbox.first);
        break;
+      case MailboxActions.markAsRead:
+        _markAsReadMailboxAction(selectionMailbox.first);
+        break;
      default:
        break;
    }
@@ -650,6 +655,15 @@ class MailboxController extends BaseMailboxController {
    _cancelSelectMailbox();
  }

+  void _markAsReadMailboxAction(PresentationMailbox presentationMailbox) {
+    final accountId = mailboxDashBoardController.accountId.value;
+    final mailboxId = presentationMailbox.id;
+    final mailboxName = presentationMailbox.name;
+    if (accountId != null && mailboxName != null) {
+      mailboxDashBoardController.markAsReadMailbox(accountId, mailboxId, mailboxName);
+    }
+  }
+
  void _createListMailboxNameAsStringInMailboxParent(PresentationMailbox mailboxRenamed) {
    if (mailboxRenamed.parentId == null) {
      final allChildrenAtMailboxLocation = (defaultMailboxTree.value.root.childrenItems ?? <MailboxNode>[]) + (folderMailboxTree.value.root.childrenItems ?? <MailboxNode>[]);
@@ -704,7 +718,7 @@ class MailboxController extends BaseMailboxController {
        _appToast.showToast(AppLocalizations.of(context).the_feature_is_under_development);
        break;
      case MailboxActions.markAsRead:
-        _appToast.showToast(AppLocalizations.of(context).the_feature_is_under_development);
+        _markAsReadMailboxAction(mailbox);
        break;
      default:
        break;
@@ -373,18 +373,20 @@ class MailboxView extends GetWidget<MailboxController> with AppLoaderMixin, Popu
  void _openMailboxMenuAction(BuildContext context, RelativeRect position,
      PresentationMailbox mailbox) {
    final listMailboxActions = [
-      MailboxActions.markAsRead,
+      if (mailbox.getCountUnReadEmails().isNotEmpty) MailboxActions.markAsRead,
      if (!mailbox.hasRole()) MailboxActions.move,
      if (!mailbox.hasRole()) MailboxActions.rename,
      if (!mailbox.hasRole()) MailboxActions.delete,
    ];

-    if (_responsiveUtils.isScreenWithShortestSide(context)) {
-      controller.openContextMenuAction(context,
-          _bottomSheetIdentityActionTiles(context, mailbox, listMailboxActions));
-    } else {
-      controller.openPopupMenuAction(context, position,
-          _popupMenuMailboxActionTiles(context, mailbox, listMailboxActions));
+    if (listMailboxActions.isNotEmpty) {
+      if (_responsiveUtils.isScreenWithShortestSide(context)) {
+        controller.openContextMenuAction(context,
+            _bottomSheetIdentityActionTiles(context, mailbox, listMailboxActions));
+      } else {
+        controller.openPopupMenuAction(context, position,
+            _popupMenuMailboxActionTiles(context, mailbox, listMailboxActions));
+      }
    }
  }

@@ -54,8 +54,12 @@ class BottomBarSelectionMailboxWidget {
            ..paddingIcon(const EdgeInsets.all(8))
            ..textStyle(TextStyle(
                fontSize: 12,
-                color: _isRenameMailboxValid ? AppColor.colorTextButton : AppColor.colorTextButton.withOpacity(0.3)))
-            ..iconColor(_isRenameMailboxValid ? AppColor.colorTextButton : AppColor.colorTextButton.withOpacity(0.3))
+                color: _isRenameMailboxValid
+                    ? AppColor.colorTextButton
+                    : AppColor.colorTextButton.withOpacity(0.3)))
+            ..iconColor(_isRenameMailboxValid
+                ? AppColor.colorTextButton
+                : AppColor.colorTextButton.withOpacity(0.3))
            ..onPressActionClick(() {
              if (_isRenameMailboxValid) {
                _onMailboxActionsClick?.call(MailboxActions.rename, _listSelectionMailbox);
@@ -66,9 +70,19 @@ class BottomBarSelectionMailboxWidget {
        Expanded(child: (ButtonBuilder(_imagePaths.icRead)
            ..key(const Key('button_mark_read_all_mailbox'))
            ..paddingIcon(const EdgeInsets.all(8))
-            ..textStyle(TextStyle(fontSize: 12, color: AppColor.colorTextButton.withOpacity(0.3)))
-            ..iconColor(AppColor.colorTextButton.withOpacity(0.3))
-            ..onPressActionClick(() => {})
+            ..textStyle(TextStyle(
+                fontSize: 12,
+                color: _isDeleteMailboxValid
+                    ? AppColor.colorTextButton
+                    : AppColor.colorTextButton.withOpacity(0.3)))
+            ..iconColor(_isDeleteMailboxValid
+                ? AppColor.colorTextButton
+                : AppColor.colorTextButton.withOpacity(0.3))
+            ..onPressActionClick(() {
+              if (_isMarkAsReadMailboxValid) {
+                _onMailboxActionsClick?.call(MailboxActions.markAsRead, _listSelectionMailbox);
+              }
+            })
            ..text(AppLocalizations.of(_context).mark_as_read, isVertical: true))
          .build()),
        Expanded(child: (ButtonBuilder(_imagePaths.icDelete)
@@ -76,8 +90,12 @@ class BottomBarSelectionMailboxWidget {
            ..paddingIcon(const EdgeInsets.all(8))
            ..textStyle(TextStyle(
                fontSize: 12,
-                color: _isDeleteMailboxValid ? AppColor.colorTextButton : AppColor.colorTextButton.withOpacity(0.3)))
-            ..iconColor(_isDeleteMailboxValid ? AppColor.colorTextButton : AppColor.colorTextButton.withOpacity(0.3))
+                color: _isDeleteMailboxValid
+                    ? AppColor.colorTextButton
+                    : AppColor.colorTextButton.withOpacity(0.3)))
+            ..iconColor(_isDeleteMailboxValid
+                ? AppColor.colorTextButton
+                : AppColor.colorTextButton.withOpacity(0.3))
            ..onPressActionClick(() {
              if (_isDeleteMailboxValid) {
                _onMailboxActionsClick?.call(MailboxActions.delete, _listSelectionMailbox);
@@ -89,9 +107,15 @@ class BottomBarSelectionMailboxWidget {
    );
  }

-  bool get _isDeleteMailboxValid => _listSelectionMailbox.length == 1 && _isAllFolderMailbox;
+  bool get _isDeleteMailboxValid => _listSelectionMailbox.length == 1
+      && _isAllFolderMailbox;

-  bool get _isRenameMailboxValid => _listSelectionMailbox.length == 1 && _isAllFolderMailbox;
+  bool get _isRenameMailboxValid => _listSelectionMailbox.length == 1
+      && _isAllFolderMailbox;

-  bool get _isAllFolderMailbox => _listSelectionMailbox.every((mailbox) => !mailbox.hasRole());
+  bool get _isMarkAsReadMailboxValid => _listSelectionMailbox.length == 1
+      && _listSelectionMailbox.first.getCountUnReadEmails().isNotEmpty;
+
+  bool get _isAllFolderMailbox =>
+      _listSelectionMailbox.every((mailbox) => !mailbox.hasRole());
 }