fixup! TF-2302 Fix delete folder

Signed-off-by: dab246 <[email protected]>
linagora · Nov 8, 2023 · e3be9cb · e3be9cb
1 parent 503ccd1
commit e3be9cb
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 14 deletions.
diff --git a/lib/features/base/mixin/handle_error_mixin.dart b/lib/features/base/mixin/handle_error_mixin.dart
@@ -56,20 +56,20 @@ mixin HandleSetErrorMixin {
     }
   }
 
-  List<MapEntry<Id, SetError>> handleSetResponse(List<SetResponse?> listSetResponse) {
+  Map<Id, SetError> handleSetResponse(List<SetResponse?> listSetResponse) {
     final listSetResponseNotNull = listSetResponse.whereNotNull().toList();
     if (listSetResponseNotNull.isEmpty) {
-      return [];
+      return <Id, SetError>{};
     }
 
-    final List<MapEntry<Id, SetError>> remainedErrors = [];
+    final Map<Id, SetError> remainedErrors = <Id, SetError>{};
     for (var response in listSetResponseNotNull) {
       handleSetErrors(
         notDestroyedError: response.notDestroyed,
         notUpdatedError: response.notUpdated,
         notCreatedError: response.notCreated,
         unCatchErrorHandler: (setErrorEntry) {
-          remainedErrors.add(setErrorEntry);
+          remainedErrors.addEntries({setErrorEntry});
           return false;
         }
       );

diff --git a/lib/features/email/data/network/email_api.dart b/lib/features/email/data/network/email_api.dart
@@ -204,12 +204,11 @@ class EmailAPI with HandleSetErrorMixin {
     }
 
     final emailCreated = setEmailResponse?.created?[idCreateMethod];
-    final listEntriesErrors = handleSetResponse([
+    final mapErrors = handleSetResponse([
       setEmailResponse,
       setEmailSubmissionResponse,
       markAsAnsweredOrForwardedSetResponse
     ]);
-    final mapErrors = Map.fromEntries(listEntriesErrors);
 
     if (emailCreated != null && mapErrors.isEmpty) {
       return true;
@@ -482,8 +481,7 @@ class EmailAPI with HandleSetErrorMixin {
     );
 
     final emailCreated = setEmailResponse?.created?[idCreateMethod];
-    final listEntriesErrors = handleSetResponse([setEmailResponse]);
-    final mapErrors = Map.fromEntries(listEntriesErrors);
+    final mapErrors = handleSetResponse([setEmailResponse]);
 
     if (emailCreated != null && mapErrors.isEmpty) {
       return emailCreated;
@@ -549,8 +547,7 @@ class EmailAPI with HandleSetErrorMixin {
 
     final emailUpdated = setEmailResponse?.created?[idCreateMethod];
     final isEmailDeleted = setEmailResponse?.destroyed?.contains(oldEmailId.id);
-    final listEntriesErrors = handleSetResponse([setEmailResponse]);
-    final mapErrors = Map.fromEntries(listEntriesErrors);
+    final mapErrors = handleSetResponse([setEmailResponse]);
 
     if (emailUpdated != null && isEmailDeleted == true && mapErrors.isEmpty) {
       return emailUpdated;

diff --git a/lib/features/mailbox/data/network/mailbox_api.dart b/lib/features/mailbox/data/network/mailbox_api.dart
@@ -281,10 +281,7 @@ class MailboxAPI with HandleSetErrorMixin {
     if (setMailboxResponse?.updated?.containsKey(request.mailboxId.id) == true) {
       return true;
     } else {
-      final listEntriesErrors = handleSetResponse([
-        setMailboxResponse,
-      ]);
-      final mapErrors = Map.fromEntries(listEntriesErrors);
+      final mapErrors = handleSetResponse([setMailboxResponse]);
       throw SetMethodException(mapErrors);
     }
   }