TF-79 Mark as star/unstar selected multiple email

This commit is contained in:
dab246
2021-09-20 14:29:02 +07:00
committed by Dat H. Pham
parent 1b061d4af9
commit c7a3e90a68
30 changed files with 565 additions and 98 deletions
@@ -31,4 +31,9 @@ abstract class EmailDataSource {
Future<List<EmailId>> moveToMailbox(AccountId accountId, MoveRequest moveRequest);
Future<bool> markAsImportant(AccountId accountId, EmailId emailId, ImportantAction importantAction);
Future<List<Email>> markAsStar(
AccountId accountId,
List<Email> emails,
MarkStarAction markStarAction
);
}
@@ -79,9 +79,9 @@ class EmailDataSourceImpl extends EmailDataSource {
}
@override
Future<bool> markAsImportant(AccountId accountId, EmailId emailId, ImportantAction importantAction) {
Future<List<Email>> markAsStar(AccountId accountId, List<Email> emails, MarkStarAction markStarAction) {
return Future.sync(() async {
return await emailAPI.markAsImportant(accountId, emailId, importantAction);
return await emailAPI.markAsStar(accountId, emails, markStarAction);
}).catchError((error) {
throw error;
});
+18 -12
View File
@@ -221,27 +221,33 @@ class EmailAPI {
}
Future<bool> markAsImportant(AccountId accountId, EmailId emailId, ImportantAction importantAction) async {
Future<List<Email>> markAsStar(AccountId accountId, List<Email> emails, MarkStarAction markStarAction) async {
final emailIds = emails.map((email) => email.id).toList();
final setEmailMethod = SetEmailMethod(accountId)
..addUpdates({
emailId.id: KeyWordIdentifier.emailFlagged.generateImportantActionPath(importantAction)
});
..addUpdates(emailIds.generateMapUpdateObjectMarkAsStar(markStarAction));
final getEmailMethod = GetEmailMethod(accountId)
..addIds(emailIds.toIds().toSet())
..addProperties(Properties({'keywords'}));
final requestBuilder = JmapRequestBuilder(_httpClient, ProcessingInvocation());
final setEmailInvocation = requestBuilder.invocation(setEmailMethod);
requestBuilder.invocation(setEmailMethod);
final getEmailInvocation = requestBuilder.invocation(getEmailMethod);
final response = await (requestBuilder
..usings(setEmailMethod.requiredCapabilities))
.build()
.execute();
..usings(setEmailMethod.requiredCapabilities))
.build()
.execute();
final setEmailResponse = response.parse<SetEmailResponse>(
setEmailInvocation.methodCallId,
SetEmailResponse.deserialize);
final getEmailResponse = response.parse<GetEmailResponse>(
getEmailInvocation.methodCallId,
GetEmailResponse.deserialize);
return Future.sync(() async {
final emailUpdated = setEmailResponse!.updated![emailId.id];
return emailUpdated == null;
return getEmailResponse!.list;
}).catchError((error) {
throw error;
});
@@ -61,7 +61,11 @@ class EmailRepositoryImpl extends EmailRepository {
}
@override
Future<bool> markAsImportant(AccountId accountId, EmailId emailId, ImportantAction importantAction) {
return emailDataSource.markAsImportant(accountId, emailId, importantAction);
Future<List<Email>> markAsStar(
AccountId accountId,
List<Email> emails,
MarkStarAction markStarAction
) {
return emailDataSource.markAsStar(accountId, emails, markStarAction);
}
}
@@ -30,4 +30,9 @@ abstract class EmailRepository {
Future<List<EmailId>> moveToMailbox(AccountId accountId, MoveRequest moveRequest);
Future<bool> markAsImportant(AccountId accountId, EmailId emailId, ImportantAction importantAction);
Future<List<Email>> markAsStar(
AccountId accountId,
List<Email> emails,
MarkStarAction markStarAction
);
}
@@ -1,23 +0,0 @@
import 'package:core/core.dart';
import 'package:jmap_dart_client/jmap/mail/email/email.dart';
import 'package:model/model.dart';
class MarkAsEmailImportantSuccess extends UIState {
final EmailId emailId;
final ImportantAction importantAction;
MarkAsEmailImportantSuccess(this.emailId, this.importantAction);
@override
List<Object?> get props => [emailId, importantAction];
}
class MarkAsEmailImportantFailure extends FeatureFailure {
final exception;
final ImportantAction importantAction;
MarkAsEmailImportantFailure(this.exception, this.importantAction);
@override
List<Object> get props => [exception, importantAction];
}
@@ -0,0 +1,23 @@
import 'package:core/core.dart';
import 'package:jmap_dart_client/jmap/mail/email/email.dart';
import 'package:model/model.dart';
class MarkAsStarEmailSuccess extends UIState {
final Email updatedEmail;
final MarkStarAction markStarAction;
MarkAsStarEmailSuccess(this.updatedEmail, this.markStarAction);
@override
List<Object?> get props => [updatedEmail, markStarAction];
}
class MarkAsStarEmailFailure extends FeatureFailure {
final exception;
final MarkStarAction markStarAction;
MarkAsStarEmailFailure(this.exception, this.markStarAction);
@override
List<Object> get props => [exception, markStarAction];
}
@@ -1,22 +0,0 @@
import 'package:core/core.dart';
import 'package:dartz/dartz.dart';
import 'package:jmap_dart_client/jmap/account_id.dart';
import 'package:jmap_dart_client/jmap/mail/email/email.dart';
import 'package:model/model.dart';
import 'package:tmail_ui_user/features/email/domain/repository/email_repository.dart';
import 'package:tmail_ui_user/features/email/domain/state/mark_as_email_important_state.dart';
class MarkAsEmailImportantInteractor {
final EmailRepository emailRepository;
MarkAsEmailImportantInteractor(this.emailRepository);
Stream<Either<Failure, Success>> execute(AccountId accountId, EmailId emailId, ImportantAction importantAction) async* {
try {
final result = await emailRepository.markAsImportant(accountId, emailId, importantAction);
yield result ? Right(MarkAsEmailImportantSuccess(emailId, importantAction)) : Left(MarkAsEmailImportantFailure(null, importantAction));
} catch (e) {
yield Left(MarkAsEmailImportantFailure(e, importantAction));
}
}
}
@@ -0,0 +1,27 @@
import 'package:core/core.dart';
import 'package:dartz/dartz.dart';
import 'package:jmap_dart_client/jmap/account_id.dart';
import 'package:jmap_dart_client/jmap/mail/email/email.dart';
import 'package:model/model.dart';
import 'package:tmail_ui_user/features/email/domain/repository/email_repository.dart';
import 'package:tmail_ui_user/features/email/domain/state/mark_as_email_star_state.dart';
class MarkAsStarEmailInteractor {
final EmailRepository emailRepository;
MarkAsStarEmailInteractor(this.emailRepository);
Stream<Either<Failure, Success>> execute(AccountId accountId, Email email, MarkStarAction markStarAction) async* {
try {
final result = await emailRepository.markAsStar(accountId, [email], markStarAction);
if (result.isNotEmpty) {
final updatedEmail = email.updatedEmail(newKeywords: result.first.keywords);
yield Right(MarkAsStarEmailSuccess(updatedEmail, markStarAction));
} else {
yield Left(MarkAsStarEmailFailure(null, markStarAction));
}
} catch (e) {
yield Left(MarkAsStarEmailFailure(e, markStarAction));
}
}
}
@@ -10,9 +10,9 @@ import 'package:tmail_ui_user/features/email/domain/repository/email_repository.
import 'package:tmail_ui_user/features/email/domain/usecases/download_attachments_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/export_attachment_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/get_email_content_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_email_important_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_email_read_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/move_to_mailbox_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_star_email_interactor.dart';
import 'package:tmail_ui_user/features/email/presentation/email_controller.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';
@@ -37,6 +37,7 @@ class EmailBindings extends Bindings {
Get.find<CredentialRepository>()));
Get.lazyPut(() => MoveToMailboxInteractor(Get.find<EmailRepository>()));
Get.lazyPut(() => MarkAsEmailImportantInteractor(Get.find<EmailRepository>()));
Get.lazyPut(() => MarkAsStarEmailInteractor(Get.find<EmailRepository>()));
Get.put(EmailController(
Get.find<GetEmailContentInteractor>(),
Get.find<MarkAsEmailReadInteractor>(),
@@ -46,6 +47,6 @@ class EmailBindings extends Bindings {
Get.find<ExportAttachmentInteractor>(),
Get.find<MoveToMailboxInteractor>()));
Get.find<ExportAttachmentInteractor>(),
Get.find<MarkAsEmailImportantInteractor>()));
Get.find<MarkAsStarEmailInteractor>()));
}
}
@@ -18,11 +18,12 @@ import 'package:tmail_ui_user/features/email/domain/state/export_attachment_stat
import 'package:tmail_ui_user/features/email/domain/state/get_email_content_state.dart';
import 'package:tmail_ui_user/features/email/domain/state/move_to_mailbox_state.dart';
import 'package:tmail_ui_user/features/email/domain/state/mark_as_email_important_state.dart';
import 'package:tmail_ui_user/features/email/domain/state/mark_as_email_star_state.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/download_attachments_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/state/mark_as_email_read_state.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/export_attachment_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/get_email_content_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_email_important_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_star_email_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_email_read_interactor.dart';
import 'package:tmail_ui_user/features/email/domain/usecases/move_to_mailbox_interactor.dart';
import 'package:tmail_ui_user/features/email/presentation/model/composer_arguments.dart';
@@ -45,6 +46,7 @@ class EmailController extends BaseController {
final ExportAttachmentInteractor _exportAttachmentInteractor;
final MoveToMailboxInteractor _moveToMailboxInteractor;
final MarkAsEmailImportantInteractor _markAsEmailImportantInteractor;
final MarkAsStarEmailInteractor _markAsStarEmailInteractor;
final emailAddressExpandMode = ExpandMode.COLLAPSE.obs;
final attachmentsExpandMode = ExpandMode.COLLAPSE.obs;
@@ -60,6 +62,7 @@ class EmailController extends BaseController {
this._exportAttachmentInteractor,
this._moveToMailboxInteractor,
this._markAsEmailImportantInteractor,
this._markAsStarEmailInteractor,
);
@override
@@ -118,6 +121,8 @@ class EmailController extends BaseController {
_moveToMailboxSuccess(success);
} else if (success is MarkAsEmailImportantSuccess) {
_markAsEmailImportantSuccess(success);
} else if (success is MarkAsStarEmailSuccess) {
_markAsEmailStarSuccess(success);
}
});
}
@@ -309,15 +314,19 @@ class EmailController extends BaseController {
}
void markAsEmailImportant(PresentationEmail presentationEmail) async {
void markAsStarEmail(PresentationEmail presentationEmail) async {
final accountId = mailboxDashBoardController.accountId.value;
final mailboxCurrent = mailboxDashBoardController.selectedMailbox.value;
if (accountId != null && mailboxCurrent != null) {
final importantAction = presentationEmail.isFlaggedEmail() ? ImportantAction.unMarkStar : ImportantAction.markStar;
consumeState(_markAsEmailImportantInteractor.execute(accountId, presentationEmail.id, importantAction));
final markStarAction = presentationEmail.isFlaggedEmail() ? MarkStarAction.unMarkStar : MarkStarAction.markStar;
consumeState(_markAsStarEmailInteractor.execute(accountId, presentationEmail.toEmail(), markStarAction));
}
}
void _markAsEmailImportantSuccess(Success success) {
void _markAsEmailStarSuccess(Success success) {
if (success is MarkAsStarEmailSuccess) {
mailboxDashBoardController.setSelectedEmail(success.updatedEmail.toPresentationEmail(selectMode: SelectMode.ACTIVE));
}
mailboxDashBoardController.dispatchState(Right(success));
}
@@ -62,6 +62,8 @@ class EmailView extends GetView {
emailController.markAsEmailRead(presentationEmail, ReadActions.markAsUnread))
..addOnMarkAsImportantActionClick((presentationEmail) =>
emailController.markAsEmailImportant(presentationEmail)))
..addOnMarkAsStarActionClick((presentationEmail) =>
emailController.markAsStarEmail(presentationEmail)))
.build()));
}
@@ -10,12 +10,14 @@ typedef OnBackActionClick = void Function();
typedef OnUnreadEmailActionClick = void Function(PresentationEmail presentationEmail);
typedef OnMoveToMailboxActionClick = void Function(PresentationEmail presentationEmail);
typedef OnMarkAsImportantActionClick = void Function(PresentationEmail presentationEmail);
typedef OnMarkAsStarActionClick = void Function(PresentationEmail presentationEmail);
class AppBarMailWidgetBuilder {
OnBackActionClick? _onBackActionClick;
OnUnreadEmailActionClick? _onUnreadEmailActionClick;
OnMoveToMailboxActionClick? _onMoveToMailboxActionClick;
OnMarkAsImportantActionClick? _onMarkAsImportantActionClick;
OnMarkAsStarActionClick? _markAsStarActionClick;
final BuildContext _context;
final ImagePaths _imagePaths;
@@ -43,6 +45,8 @@ class AppBarMailWidgetBuilder {
void addOnMarkAsImportantActionClick(OnMarkAsImportantActionClick onMarkAsImportantActionClick) {
_onMarkAsImportantActionClick = onMarkAsImportantActionClick;
void addOnMarkAsStarActionClick(OnMarkAsStarActionClick onMarkAsStarActionClick) {
_markAsStarActionClick = onMarkAsStarActionClick;
}
Widget build() {
@@ -99,8 +103,8 @@ class AppBarMailWidgetBuilder {
: _imagePaths.icFlag)
.key(Key('button_mark_as_email_important'))
.onPressActionClick(() {
if (_onMarkAsImportantActionClick != null && _presentationEmail != null) {
_onMarkAsImportantActionClick!(_presentationEmail!);
if (_markAsStarActionClick != null && _presentationEmail != null) {
_markAsStarActionClick!(_presentationEmail!);
}})
.build(),
SizedBox(width: 10),