TF-3042 Clean up public assets after identity is saved
This commit is contained in:
@@ -10,12 +10,12 @@ class CreateNewDefaultIdentityRequest extends CreateNewIdentityRequest {
|
||||
super.creationId,
|
||||
super.newIdentity,
|
||||
{
|
||||
this.oldDefaultIdentityIds
|
||||
this.oldDefaultIdentityIds,
|
||||
super.publicAssetsInIdentityArguments,
|
||||
});
|
||||
|
||||
@override
|
||||
List<Object?> get props => [
|
||||
super.creationId,
|
||||
super.newIdentity,
|
||||
...super.props,
|
||||
oldDefaultIdentityIds];
|
||||
}
|
||||
@@ -2,20 +2,23 @@
|
||||
import 'package:equatable/equatable.dart';
|
||||
import 'package:jmap_dart_client/jmap/core/id.dart';
|
||||
import 'package:jmap_dart_client/jmap/identities/identity.dart';
|
||||
import 'package:tmail_ui_user/features/public_asset/domain/model/public_assets_in_identity_arguments.dart';
|
||||
|
||||
class CreateNewIdentityRequest with EquatableMixin {
|
||||
|
||||
final Identity newIdentity;
|
||||
final Id creationId;
|
||||
final bool isDefaultIdentity;
|
||||
final PublicAssetsInIdentityArguments? publicAssetsInIdentityArguments;
|
||||
|
||||
CreateNewIdentityRequest(
|
||||
this.creationId,
|
||||
this.newIdentity,
|
||||
{
|
||||
this.isDefaultIdentity = false
|
||||
this.isDefaultIdentity = false,
|
||||
this.publicAssetsInIdentityArguments,
|
||||
});
|
||||
|
||||
@override
|
||||
List<Object?> get props => [creationId, newIdentity, isDefaultIdentity];
|
||||
List<Object?> get props => [creationId, newIdentity, isDefaultIdentity, publicAssetsInIdentityArguments];
|
||||
}
|
||||
@@ -10,14 +10,13 @@ class EditDefaultIdentityRequest extends EditIdentityRequest {
|
||||
required super.identityId,
|
||||
required super.identityRequest,
|
||||
required super.isDefaultIdentity,
|
||||
this.oldDefaultIdentityIds
|
||||
this.oldDefaultIdentityIds,
|
||||
super.publicAssetsInIdentityArguments
|
||||
});
|
||||
|
||||
@override
|
||||
List<Object?> get props => [
|
||||
super.identityId,
|
||||
super.identityRequest,
|
||||
super.isDefaultIdentity,
|
||||
...super.props,
|
||||
oldDefaultIdentityIds
|
||||
];
|
||||
}
|
||||
@@ -2,23 +2,27 @@
|
||||
import 'package:equatable/equatable.dart';
|
||||
import 'package:jmap_dart_client/jmap/identities/identity.dart';
|
||||
import 'package:model/model.dart';
|
||||
import 'package:tmail_ui_user/features/public_asset/domain/model/public_assets_in_identity_arguments.dart';
|
||||
|
||||
class EditIdentityRequest with EquatableMixin {
|
||||
|
||||
final IdentityRequestDto identityRequest;
|
||||
final IdentityId identityId;
|
||||
final bool isDefaultIdentity;
|
||||
final PublicAssetsInIdentityArguments? publicAssetsInIdentityArguments;
|
||||
|
||||
EditIdentityRequest({
|
||||
required this.identityId,
|
||||
required this.identityRequest,
|
||||
this.isDefaultIdentity = false
|
||||
this.isDefaultIdentity = false,
|
||||
this.publicAssetsInIdentityArguments
|
||||
});
|
||||
|
||||
@override
|
||||
List<Object?> get props => [
|
||||
identityId,
|
||||
identityRequest,
|
||||
isDefaultIdentity
|
||||
isDefaultIdentity,
|
||||
publicAssetsInIdentityArguments
|
||||
];
|
||||
}
|
||||
@@ -1,16 +1,20 @@
|
||||
import 'package:core/core.dart';
|
||||
import 'package:jmap_dart_client/jmap/identities/identity.dart';
|
||||
import 'package:tmail_ui_user/features/public_asset/domain/model/public_assets_in_identity_arguments.dart';
|
||||
|
||||
class CreateNewDefaultIdentityLoading extends UIState {}
|
||||
|
||||
class CreateNewDefaultIdentitySuccess extends UIState {
|
||||
|
||||
final Identity newIdentity;
|
||||
final PublicAssetsInIdentityArguments? publicAssetsInIdentityArguments;
|
||||
|
||||
CreateNewDefaultIdentitySuccess(this.newIdentity);
|
||||
CreateNewDefaultIdentitySuccess(
|
||||
this.newIdentity,
|
||||
{this.publicAssetsInIdentityArguments});
|
||||
|
||||
@override
|
||||
List<Object?> get props => [newIdentity];
|
||||
List<Object?> get props => [newIdentity, publicAssetsInIdentityArguments];
|
||||
}
|
||||
|
||||
class CreateNewDefaultIdentityFailure extends FeatureFailure {
|
||||
|
||||
@@ -1,16 +1,20 @@
|
||||
import 'package:core/core.dart';
|
||||
import 'package:jmap_dart_client/jmap/identities/identity.dart';
|
||||
import 'package:tmail_ui_user/features/public_asset/domain/model/public_assets_in_identity_arguments.dart';
|
||||
|
||||
class CreateNewIdentityLoading extends UIState {}
|
||||
|
||||
class CreateNewIdentitySuccess extends UIState {
|
||||
|
||||
final Identity newIdentity;
|
||||
final PublicAssetsInIdentityArguments? publicAssetsInIdentityArguments;
|
||||
|
||||
CreateNewIdentitySuccess(this.newIdentity);
|
||||
CreateNewIdentitySuccess(
|
||||
this.newIdentity,
|
||||
{this.publicAssetsInIdentityArguments});
|
||||
|
||||
@override
|
||||
List<Object?> get props => [newIdentity];
|
||||
List<Object?> get props => [newIdentity, publicAssetsInIdentityArguments];
|
||||
}
|
||||
|
||||
class CreateNewIdentityFailure extends FeatureFailure {
|
||||
|
||||
@@ -4,7 +4,11 @@ import 'package:tmail_ui_user/features/manage_account/domain/state/edit_identity
|
||||
|
||||
class EditDefaultIdentityLoading extends UIState {}
|
||||
|
||||
class EditDefaultIdentitySuccess extends EditIdentitySuccess {}
|
||||
class EditDefaultIdentitySuccess extends EditIdentitySuccess {
|
||||
EditDefaultIdentitySuccess(
|
||||
super.identityId,
|
||||
{super.publicAssetsInIdentityArguments});
|
||||
}
|
||||
|
||||
class EditDefaultIdentityFailure extends FeatureFailure {
|
||||
|
||||
|
||||
@@ -1,9 +1,21 @@
|
||||
import 'package:core/presentation/state/failure.dart';
|
||||
import 'package:core/presentation/state/success.dart';
|
||||
import 'package:jmap_dart_client/jmap/identities/identity.dart';
|
||||
import 'package:tmail_ui_user/features/public_asset/domain/model/public_assets_in_identity_arguments.dart';
|
||||
|
||||
class EditIdentityLoading extends UIState {}
|
||||
|
||||
class EditIdentitySuccess extends UIState {}
|
||||
class EditIdentitySuccess extends UIState {
|
||||
final IdentityId identityId;
|
||||
final PublicAssetsInIdentityArguments? publicAssetsInIdentityArguments;
|
||||
|
||||
EditIdentitySuccess(
|
||||
this.identityId,
|
||||
{this.publicAssetsInIdentityArguments});
|
||||
|
||||
@override
|
||||
List<Object?> get props => [identityId, publicAssetsInIdentityArguments];
|
||||
}
|
||||
|
||||
class EditIdentityFailure extends FeatureFailure {
|
||||
|
||||
|
||||
+3
-1
@@ -33,7 +33,9 @@ class CreateNewDefaultIdentityInteractor {
|
||||
final defaultRequest = _createNewIdentityDefault(identityRequest, listDefaultIdentities);
|
||||
|
||||
final newIdentity = await _identityRepository.createNewIdentity(session, accountId, defaultRequest);
|
||||
yield Right(CreateNewDefaultIdentitySuccess(newIdentity));
|
||||
yield Right(CreateNewDefaultIdentitySuccess(
|
||||
newIdentity,
|
||||
publicAssetsInIdentityArguments: identityRequest.publicAssetsInIdentityArguments));
|
||||
} catch (exception) {
|
||||
yield Left(CreateNewDefaultIdentityFailure(exception));
|
||||
}
|
||||
|
||||
@@ -21,7 +21,9 @@ class CreateNewIdentityInteractor {
|
||||
try {
|
||||
yield Right(CreateNewIdentityLoading());
|
||||
final newIdentity = await _identityRepository.createNewIdentity(session, accountId, identityRequest);
|
||||
yield Right(CreateNewIdentitySuccess(newIdentity));
|
||||
yield Right(CreateNewIdentitySuccess(
|
||||
newIdentity,
|
||||
publicAssetsInIdentityArguments: identityRequest.publicAssetsInIdentityArguments));
|
||||
} catch (exception) {
|
||||
yield Left(CreateNewIdentityFailure(exception));
|
||||
}
|
||||
|
||||
@@ -41,7 +41,11 @@ class EditDefaultIdentityInteractor {
|
||||
.toList());
|
||||
|
||||
final result = await _identityRepository.editIdentity(session, accountId, editDefaultRequest);
|
||||
yield result ? Right(EditDefaultIdentitySuccess()) : Left(EditDefaultIdentityFailure(null));
|
||||
yield result
|
||||
? Right(EditDefaultIdentitySuccess(
|
||||
editDefaultRequest.identityId,
|
||||
publicAssetsInIdentityArguments: editDefaultRequest.publicAssetsInIdentityArguments))
|
||||
: Left(EditDefaultIdentityFailure(null));
|
||||
} catch (exception) {
|
||||
yield Left(EditDefaultIdentityFailure(exception));
|
||||
}
|
||||
|
||||
@@ -22,7 +22,11 @@ class EditIdentityInteractor {
|
||||
try {
|
||||
yield Right(EditIdentityLoading());
|
||||
final result = await _identityRepository.editIdentity(session, accountId, editIdentityRequest);
|
||||
yield result ? Right(EditIdentitySuccess()) : Left(EditIdentityFailure(null));
|
||||
yield result
|
||||
? Right(EditIdentitySuccess(
|
||||
editIdentityRequest.identityId,
|
||||
publicAssetsInIdentityArguments: editIdentityRequest.publicAssetsInIdentityArguments))
|
||||
: Left(EditIdentityFailure(null));
|
||||
} catch (exception) {
|
||||
yield Left(EditIdentityFailure(exception));
|
||||
}
|
||||
|
||||
+62
@@ -3,11 +3,13 @@ import 'package:core/presentation/extensions/color_extension.dart';
|
||||
import 'package:core/presentation/state/failure.dart';
|
||||
import 'package:core/presentation/state/success.dart';
|
||||
import 'package:core/presentation/views/dialog/confirmation_dialog_builder.dart';
|
||||
import 'package:core/utils/app_logger.dart';
|
||||
import 'package:core/utils/platform_info.dart';
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:flutter_svg/flutter_svg.dart';
|
||||
import 'package:get/get.dart';
|
||||
import 'package:jmap_dart_client/jmap/account_id.dart';
|
||||
import 'package:jmap_dart_client/jmap/core/capability/capability_identifier.dart';
|
||||
import 'package:jmap_dart_client/jmap/core/session/session.dart';
|
||||
import 'package:jmap_dart_client/jmap/identities/identity.dart';
|
||||
import 'package:tmail_ui_user/features/base/base_controller.dart';
|
||||
@@ -31,6 +33,11 @@ import 'package:tmail_ui_user/features/manage_account/presentation/extensions/id
|
||||
import 'package:tmail_ui_user/features/manage_account/presentation/manage_account_dashboard_controller.dart';
|
||||
import 'package:tmail_ui_user/features/manage_account/presentation/model/identity_action_type.dart';
|
||||
import 'package:tmail_ui_user/features/manage_account/presentation/profiles/identities/widgets/delete_identity_dialog_builder.dart';
|
||||
import 'package:tmail_ui_user/features/public_asset/domain/model/public_assets_in_identity_arguments.dart';
|
||||
import 'package:tmail_ui_user/features/public_asset/domain/usecase/clean_up_public_assets_interactor.dart';
|
||||
import 'package:tmail_ui_user/features/public_asset/presentation/clean_up_public_assets_interactor_bindings.dart';
|
||||
import 'package:tmail_ui_user/main/bindings/network/binding_tag.dart';
|
||||
import 'package:tmail_ui_user/main/error/capability_validator.dart';
|
||||
import 'package:tmail_ui_user/main/localizations/app_localizations.dart';
|
||||
import 'package:tmail_ui_user/main/routes/app_routes.dart';
|
||||
import 'package:tmail_ui_user/main/routes/dialog_router.dart';
|
||||
@@ -99,6 +106,7 @@ class IdentitiesController extends BaseController {
|
||||
final session = accountDashBoardController.sessionCurrent;
|
||||
if (accountId != null && session != null) {
|
||||
_getAllIdentities(session, accountId);
|
||||
_injectCleanUpPublicAssetsInteractorBindings(session, accountId);
|
||||
}
|
||||
});
|
||||
}
|
||||
@@ -177,6 +185,11 @@ class IdentitiesController extends BaseController {
|
||||
AppLocalizations.of(currentContext!).you_have_created_a_new_identity);
|
||||
}
|
||||
|
||||
_cleanUpPublicAssets(
|
||||
success.newIdentity.id,
|
||||
IdentityActionType.create,
|
||||
success.publicAssetsInIdentityArguments);
|
||||
|
||||
_refreshAllIdentities();
|
||||
}
|
||||
|
||||
@@ -187,6 +200,11 @@ class IdentitiesController extends BaseController {
|
||||
AppLocalizations.of(currentContext!).you_have_created_a_new_default_identity);
|
||||
}
|
||||
|
||||
_cleanUpPublicAssets(
|
||||
success.newIdentity.id,
|
||||
IdentityActionType.create,
|
||||
success.publicAssetsInIdentityArguments);
|
||||
|
||||
_refreshAllIdentities();
|
||||
}
|
||||
|
||||
@@ -287,8 +305,52 @@ class IdentitiesController extends BaseController {
|
||||
AppLocalizations.of(currentContext!).you_are_changed_your_identity_successfully);
|
||||
}
|
||||
|
||||
_cleanUpPublicAssets(
|
||||
success.identityId,
|
||||
IdentityActionType.edit,
|
||||
success.publicAssetsInIdentityArguments);
|
||||
|
||||
_refreshAllIdentities();
|
||||
}
|
||||
|
||||
bool get isSignatureShow => identitySelected.value != null;
|
||||
|
||||
void _injectCleanUpPublicAssetsInteractorBindings(Session? session, AccountId? accountId) {
|
||||
try {
|
||||
requireCapability(session!, accountId!, [CapabilityIdentifier.jmapPublicAsset]);
|
||||
CleanUpPublicAssetsInteractorBindings().dependencies();
|
||||
} catch(e) {
|
||||
logError('$runtimeType::injectCleanUpPublicAssetsInteractorBindings(): exception: $e');
|
||||
}
|
||||
}
|
||||
|
||||
void _cleanUpPublicAssets(
|
||||
IdentityId? identityId,
|
||||
IdentityActionType? identityActionType,
|
||||
PublicAssetsInIdentityArguments? publicAssetsInIdentityArguments) {
|
||||
final session = accountDashBoardController.sessionCurrent;
|
||||
final accountId = accountDashBoardController.accountId.value;
|
||||
|
||||
if (session == null
|
||||
|| accountId == null
|
||||
|| identityId == null
|
||||
|| identityActionType == null
|
||||
|| publicAssetsInIdentityArguments == null) return;
|
||||
|
||||
final cleanUpPublicAssetsInteractor = getBinding<CleanUpPublicAssetsInteractor>(
|
||||
tag: BindingTag.cleanUpPublicAssetsInteractorBindingsTag);
|
||||
if (cleanUpPublicAssetsInteractor == null) return;
|
||||
consumeState(cleanUpPublicAssetsInteractor.execute(
|
||||
session,
|
||||
accountId,
|
||||
identityId: identityId,
|
||||
identityActionType: identityActionType,
|
||||
publicAssetsInIdentityArguments: publicAssetsInIdentityArguments));
|
||||
}
|
||||
|
||||
@override
|
||||
void onClose() {
|
||||
CleanUpPublicAssetsInteractorBindings().close();
|
||||
super.onClose();
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user