eduvia/workavia-mail-front
11
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fixup! fixup! fixup! TF-4236 Misleading variable name: maxBatches represents batch size, not batch count.

Browse Source
This commit is contained in:
Dat H. Pham
2026-01-27 11:21:11 +07:00
committed by Dat H. Pham
parent 8d989c3caa
commit 6aab6f83eb
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/features/base/mixin/batch_set_email_processing_mixin.dart
+2 -2
View File
@@ -40,7 +40,7 @@ mixin BatchSetEmailProcessingMixin on HandleSetErrorMixin, MailAPIMixin {
final maxObjects = getMaxObjectsInSetMethod(session, accountId); final maxObjects = getMaxObjectsInSetMethod(session, accountId);
final totalEmails = emailIds.length; final totalEmails = emailIds.length;
final batchSize = min(totalEmails, maxObjects); final batchSize = max(1, min(totalEmails, maxObjects));
final List<EmailId> updatedEmailIds = List.empty(growable: true); final List<EmailId> updatedEmailIds = List.empty(growable: true);
final List<SetResponse> listSetResponse = List.empty(growable: true); final List<SetResponse> listSetResponse = List.empty(growable: true);
@@ -70,7 +70,7 @@ mixin BatchSetEmailProcessingMixin on HandleSetErrorMixin, MailAPIMixin {
} }
} catch (e) { } catch (e) {
logWarning( logWarning(
'BatchSetEmailProcessingMixin::$debugLabel: Error processing batch ${start + 1}-$end', 'BatchSetEmailProcessingMixin::$debugLabel: Error processing batch ${start + 1}-$end: $e',
); );
} }
} }