Upgrade Flutter v3.24.5

tddang-linagora · hoangdat · commit 392f12f7a1f6 · 2025-02-10T01:10:24.000-08:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -3,7 +3,7 @@
 name: CI
 
 env:
-  FLUTTER_VERSION: 3.22.2
+  FLUTTER_VERSION: 3.24.5
 
 # Controls when the workflow will run
 on: [push, pull_request, workflow_dispatch]
diff --git a/lib/http/converter/email/email_body_value_converter.dart b/lib/http/converter/email/email_body_value_converter.dart
@@ -12,6 +12,6 @@ class EmailBodyValueConverter {
   }
 
   MapEntry<String, dynamic> toJson(PartId partId, EmailBodyValue value) {
-    return MapEntry(PartIdConverter().toJson(partId), value.toJson());
+    return MapEntry(const PartIdConverter().toJson(partId), value.toJson());
   }
 }
diff --git a/lib/http/converter/email/email_mailbox_ids_converter.dart b/lib/http/converter/email/email_mailbox_ids_converter.dart
@@ -6,6 +6,6 @@ class EmailMailboxIdsConverter {
   MapEntry<MailboxId, bool> parseEntry(String key, bool value) => MapEntry(MailboxId(Id(key)), value);
 
   MapEntry<String, bool> toJson(MailboxId mailboxId, bool value) {
-    return MapEntry(IdConverter().toJson(mailboxId.id), value);
+    return MapEntry(const IdConverter().toJson(mailboxId.id), value);
   }
 }
diff --git a/lib/http/converter/method_response_converter.dart b/lib/http/converter/method_response_converter.dart
@@ -7,7 +7,7 @@ class MethodResponseConverter implements JsonConverter<MethodResponse, dynamic>
 
   @override
   MethodResponse fromJson(dynamic json) {
-    return this.fromJson(json);
+    return fromJson(json);
   }
 
   @override
diff --git a/lib/jmap/core/method/request/get_method.dart b/lib/jmap/core/method/request/get_method.dart
@@ -20,9 +20,7 @@ mixin OptionalIds {
   Set<Id>? ids;
 
   void addIds(Set<Id> values) {
-    if (ids == null) {
-      ids = Set();
-    }
+    ids ??= <Id>{};
     ids?.addAll(values);
   }
 }
diff --git a/lib/jmap/core/method/request/query_method.dart b/lib/jmap/core/method/request/query_method.dart
@@ -54,9 +54,7 @@ mixin OptionalSort {
   Set<Comparator>? sort;
 
   void addSorts(Set<Comparator> value) {
-    if (sort == null) {
-      sort = Set();
-    }
+    sort ??= <Comparator>{};
     sort?.addAll(value);
   }
 }
diff --git a/lib/jmap/core/method/request/set_method.dart b/lib/jmap/core/method/request/set_method.dart
@@ -22,7 +22,7 @@ mixin OptionalIfInState {
   State? ifInState;
 
   void addIfInState(State? state) {
-    this.ifInState = state;
+    ifInState = state;
   }
 }
 
@@ -31,16 +31,12 @@ mixin OptionalCreate<T> {
   Map<Id, T>? create;
 
   void addCreates(Map<Id, T> creates) {
-    if (create == null) {
-      create = Map<Id, T>();
-    }
+    create ??= <Id, T>{};
     create?.addAll(creates);
   }
 
   void addCreate(Id id, T createItem) {
-    if (create == null) {
-      create = Map<Id, T>();
-    }
+    create ??= <Id, T>{};
     create?.addAll({id: createItem});
   }
 }
@@ -50,9 +46,7 @@ mixin OptionalUpdate {
   Map<Id, PatchObject>? update;
 
   void addUpdates(Map<Id, PatchObject> updates) {
-    if (update == null) {
-      update = Map<Id, PatchObject>();
-    }
+    update ??= <Id, PatchObject>{};
     update?.addAll(updates);
   }
 }
@@ -62,9 +56,7 @@ mixin OptionalDestroy {
   Set<Id>? destroy;
 
   void addDestroy(Set<Id> values) {
-    if (destroy == null) {
-      destroy = Set();
-    }
+    destroy ??= <Id>{};
     destroy?.addAll(values);
   }
 }
@@ -74,9 +66,7 @@ mixin OptionalUpdateSingleton<T> {
   Map<Id, T>? updateSingleton;
 
   void addUpdatesSingleton(Map<Id, T> updates) {
-    if (updateSingleton == null) {
-      updateSingleton = Map<Id, T>();
-    }
+    updateSingleton ??= <Id, T>{};
     updateSingleton?.addAll(updates);
   }
 }
diff --git a/lib/jmap/core/patch_object.dart b/lib/jmap/core/patch_object.dart
@@ -2,8 +2,8 @@ import 'package:equatable/equatable.dart';
 
 class PatchObject with EquatableMixin {
 
-  static final mailboxIdsProperty = 'mailboxIds';
-  static final keywordsProperty = 'keywords';
+  static const mailboxIdsProperty = 'mailboxIds';
+  static const keywordsProperty = 'keywords';
   static const identityIdsProperty = 'identityIds';
 
   PatchObject(this.patches);
diff --git a/lib/jmap/core/properties/properties.dart b/lib/jmap/core/properties/properties.dart
@@ -5,7 +5,7 @@ class Properties with EquatableMixin {
 
   Properties(this.value);
 
-  static Properties empty() => Properties(Set());
+  static Properties empty() => Properties(<String>{});
 
   Properties union(Properties other) => Properties(value.union(other.value));
 
diff --git a/lib/jmap/core/response/response_object.dart b/lib/jmap/core/response/response_object.dart
@@ -27,7 +27,7 @@ class ResponseObject with EquatableMixin {
 
   Map<String, dynamic> toJson() => _$ResponseObjectToJson(this);
 
-  T? parse<T extends MethodResponse>(MethodCallId methodCallId, T fromJson(Map<String, dynamic> o), {MethodName? methodName}) {
+  T? parse<T extends MethodResponse>(MethodCallId methodCallId, T Function(Map<String, dynamic> o) fromJson, {MethodName? methodName}) {
     final matchedResponse = methodResponses.firstWhere((method) => methodName == null
         ? method.methodCallId == methodCallId
         : (method.methodCallId == methodCallId && _validMethodResponseName(method, methodName)));
diff --git a/lib/jmap/core/session/session.dart b/lib/jmap/core/session/session.dart
@@ -36,14 +36,12 @@ class Session with EquatableMixin {
   );
 
   factory Session.fromJson(Map<String, dynamic> json, {CapabilitiesConverter? converter}) {
-    if (converter == null) {
-      converter = CapabilitiesConverter.defaultConverter;
-    }
+    converter ??= CapabilitiesConverter.defaultConverter;
     return Session(
       (json['capabilities'] as Map<String, dynamic>)
         .map((key, value) => converter!.convert(key, value)),
       (json['accounts'] as Map<String, dynamic>)
-        .map((key, value) => AccountConverter().convert(key, value, converter!)),
+        .map((key, value) => const AccountConverter().convert(key, value, converter!)),
       (json['primaryAccounts'] as Map<String, dynamic>)
         .map((key, value) => MapEntry(
           CapabilityIdentifier(Uri.parse(key)),
diff --git a/lib/jmap/identities/set/set_identity_response.dart b/lib/jmap/identities/set/set_identity_response.dart
@@ -34,29 +34,29 @@ class SetIdentityResponse extends SetResponse<Identity> {
   static SetIdentityResponse deserialize(Map<String, dynamic> json) {
     return SetIdentityResponse(
       const AccountIdConverter().fromJson(json['accountId'] as String),
-      newState: StateNullableConverter().fromJson(json['newState'] as String?),
-      oldState: StateNullableConverter().fromJson(json['oldState'] as String?),
+      newState: const StateNullableConverter().fromJson(json['newState'] as String?),
+      oldState: const StateNullableConverter().fromJson(json['oldState'] as String?),
       created: (json['created'] as Map<String, dynamic>?)
         ?.map((key, value) => MapEntry(
-            IdConverter().fromJson(key),
+            const IdConverter().fromJson(key),
             Identity.fromJson(value as Map<String, dynamic>))),
       updated: (json['updated'] as Map<String, dynamic>?)
         ?.map((key, value) => MapEntry(
-            IdConverter().fromJson(key),
+            const IdConverter().fromJson(key),
             value != null ? Identity.fromJson(value as Map<String, dynamic>) : null)),
       destroyed: (json['destroyed'] as List<dynamic>?)
-        ?.map((id) => IdConverter().fromJson(id)).toSet(),
+        ?.map((id) => const IdConverter().fromJson(id)).toSet(),
       notCreated: (json['notCreated'] as Map<String, dynamic>?)
         ?.map((key, value) => MapEntry(
-            IdConverter().fromJson(key),
+            const IdConverter().fromJson(key),
             SetError.fromJson(value))),
       notUpdated: (json['notUpdated'] as Map<String, dynamic>?)
         ?.map((key, value) => MapEntry(
-            IdConverter().fromJson(key),
+            const IdConverter().fromJson(key),
             SetError.fromJson(value))),
       notDestroyed: (json['notDestroyed'] as Map<String, dynamic>?)
         ?.map((key, value) => MapEntry(
-            IdConverter().fromJson(key),
+            const IdConverter().fromJson(key),
             SetError.fromJson(value))),
     );
   }
diff --git a/lib/jmap/mail/email/get/get_email_method.dart b/lib/jmap/mail/email/get/get_email_method.dart
@@ -37,6 +37,7 @@ class GetEmailMethod extends GetMethod with OptionalEmailBodyProperties, Optiona
 
   factory GetEmailMethod.fromJson(Map<String, dynamic> json) => _$GetEmailMethodFromJson(json);
 
+  @override
   Map<String, dynamic> toJson() => _$GetEmailMethodToJson(this);
 }
 
diff --git a/lib/jmap/mail/email/query/query_email_response.dart b/lib/jmap/mail/email/query/query_email_response.dart
@@ -2,7 +2,6 @@ import 'package:jmap_dart_client/http/converter/account_id_converter.dart';
 import 'package:jmap_dart_client/http/converter/id_converter.dart';
 import 'package:jmap_dart_client/http/converter/state_converter.dart';
 import 'package:jmap_dart_client/http/converter/unsigned_int_converter.dart';
-import 'package:jmap_dart_client/http/converter/unsigned_int_nullable_converter.dart';
 import 'package:jmap_dart_client/jmap/account_id.dart';
 import 'package:jmap_dart_client/jmap/core/id.dart';
 import 'package:jmap_dart_client/jmap/core/method/response/query_response.dart';
diff --git a/lib/jmap/mail/email/query/query_email_response.g.dart b/lib/jmap/mail/email/query/query_email_response.g.dart
diff --git a/lib/jmap/mail/email/set/set_email_response.dart b/lib/jmap/mail/email/set/set_email_response.dart
@@ -34,29 +34,29 @@ class SetEmailResponse extends SetResponse<Email> {
   static SetEmailResponse deserialize(Map<String, dynamic> json) {
     return SetEmailResponse(
       const AccountIdConverter().fromJson(json['accountId'] as String),
-      newState: StateNullableConverter().fromJson(json['newState'] as String?),
-      oldState: StateNullableConverter().fromJson(json['oldState'] as String?),
+      newState: const StateNullableConverter().fromJson(json['newState'] as String?),
+      oldState: const StateNullableConverter().fromJson(json['oldState'] as String?),
       created: (json['created'] as Map<String, dynamic>?)
         ?.map((key, value) => MapEntry(
-            IdConverter().fromJson(key),
+            const IdConverter().fromJson(key),
             Email.fromJson(value as Map<String, dynamic>))),
       updated: (json['updated'] as Map<String, dynamic>?)
         ?.map((key, value) => MapEntry(
-            IdConverter().fromJson(key),
+            const IdConverter().fromJson(key),
             value != null ? Email.fromJson(value as Map<String, dynamic>) : null)),
       destroyed: (json['destroyed'] as List<dynamic>?)
-        ?.map((id) => IdConverter().fromJson(id)).toSet(),
+        ?.map((id) => const IdConverter().fromJson(id)).toSet(),
       notCreated: (json['notCreated'] as Map<String, dynamic>?)
         ?.map((key, value) => MapEntry(
-            IdConverter().fromJson(key),
+            const IdConverter().fromJson(key),
             SetError.fromJson(value))),
       notUpdated: (json['notUpdated'] as Map<String, dynamic>?)
         ?.map((key, value) => MapEntry(
-            IdConverter().fromJson(key),
+            const IdConverter().fromJson(key),
             SetError.fromJson(value))),
       notDestroyed: (json['notDestroyed'] as Map<String, dynamic>?)
         ?.map((key, value) => MapEntry(
-            IdConverter().fromJson(key),
+            const IdConverter().fromJson(key),
             SetError.fromJson(value))),
     );
   }
diff --git a/lib/jmap/mail/email/submission/email_submission.dart b/lib/jmap/mail/email/submission/email_submission.dart
@@ -53,9 +53,9 @@ class EmailSubmission with EquatableMixin {
       deliveryStatus: (json['deliveryStatus'] as Map<String, dynamic>?)
           ?.map((key, value) => DeliveryStatusConverter().parseEntry(key, value)),
       dsnBlobIds: (json['dsnBlobIds'] as List<dynamic>?)
-          ?.map((json) => IdConverter().fromJson(json)).toSet(),
+          ?.map((json) => const IdConverter().fromJson(json)).toSet(),
       mdnBlobIds: (json['mdnBlobIds'] as List<dynamic>?)
-          ?.map((json) => IdConverter().fromJson(json)).toSet()
+          ?.map((json) => const IdConverter().fromJson(json)).toSet()
     );
   }
 
diff --git a/lib/jmap/mail/email/submission/set/set_email_submission_response.dart b/lib/jmap/mail/email/submission/set/set_email_submission_response.dart
@@ -36,22 +36,22 @@ class SetEmailSubmissionResponse extends SetResponse<EmailSubmission> {
   static SetEmailSubmissionResponse deserialize(Map<String, dynamic> json) {
     return SetEmailSubmissionResponse(
       const AccountIdConverter().fromJson(json['accountId'] as String),
-      newState: StateNullableConverter().fromJson(json['newState'] as String?),
-      oldState: StateNullableConverter().fromJson(json['oldState'] as String?),
+      newState: const StateNullableConverter().fromJson(json['newState'] as String?),
+      oldState: const StateNullableConverter().fromJson(json['oldState'] as String?),
       created: (json['created'] as Map<String, dynamic>?)
-        ?.map((key, value) => MapEntry(IdConverter().fromJson(key), EmailSubmission.fromJson(value as Map<String, dynamic>))),
+        ?.map((key, value) => MapEntry(const IdConverter().fromJson(key), EmailSubmission.fromJson(value as Map<String, dynamic>))),
       updated: (json['updated'] as Map<String, dynamic>?)
         ?.map((key, value) => MapEntry(
-            IdConverter().fromJson(key),
+            const IdConverter().fromJson(key),
             value != null ? EmailSubmission.fromJson(value as Map<String, dynamic>) : null)),
       destroyed: (json['destroyed'] as List<dynamic>?)
-        ?.map((id) => IdConverter().fromJson(id)).toSet(),
+        ?.map((id) => const IdConverter().fromJson(id)).toSet(),
       notCreated: (json['notCreated'] as Map<String, dynamic>?)
-        ?.map((key, value) => MapEntry(IdConverter().fromJson(key), SetError.fromJson(value))),
+        ?.map((key, value) => MapEntry(const IdConverter().fromJson(key), SetError.fromJson(value))),
       notUpdated: (json['notUpdated'] as Map<String, dynamic>?)
-        ?.map((key, value) => MapEntry(IdConverter().fromJson(key), SetError.fromJson(value))),
+        ?.map((key, value) => MapEntry(const IdConverter().fromJson(key), SetError.fromJson(value))),
       notDestroyed: (json['notDestroyed'] as Map<String, dynamic>?)
-        ?.map((key, value) => MapEntry(IdConverter().fromJson(key), SetError.fromJson(value))),
+        ?.map((key, value) => MapEntry(const IdConverter().fromJson(key), SetError.fromJson(value))),
     );
   }
 
diff --git a/lib/jmap/mail/mailbox/query/query_mailbox_response.dart b/lib/jmap/mail/mailbox/query/query_mailbox_response.dart
@@ -2,7 +2,6 @@ import 'package:jmap_dart_client/http/converter/account_id_converter.dart';
 import 'package:jmap_dart_client/http/converter/id_converter.dart';
 import 'package:jmap_dart_client/http/converter/state_converter.dart';
 import 'package:jmap_dart_client/http/converter/unsigned_int_converter.dart';
-import 'package:jmap_dart_client/http/converter/unsigned_int_nullable_converter.dart';
 import 'package:jmap_dart_client/jmap/account_id.dart';
 import 'package:jmap_dart_client/jmap/core/id.dart';
 import 'package:jmap_dart_client/jmap/core/method/response/query_response.dart';
diff --git a/lib/jmap/mail/mailbox/query/query_mailbox_response.g.dart b/lib/jmap/mail/mailbox/query/query_mailbox_response.g.dart
diff --git a/lib/jmap/mail/mailbox/set/set_mailbox_response.dart b/lib/jmap/mail/mailbox/set/set_mailbox_response.dart
diff --git a/lib/jmap/mail/vacation/set/set_vacation_response.dart b/lib/jmap/mail/vacation/set/set_vacation_response.dart
diff --git a/pubspec.yaml b/pubspec.yaml
diff --git a/test/jmap/core/utc_date_converter_test.dart b/test/jmap/core/utc_date_converter_test.dart

Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,6 @@ class EmailBodyValueConverter {`
`12`	`12`	`}`
`13`	`13`
`14`	`14`	`MapEntry<String, dynamic> toJson(PartId partId, EmailBodyValue value) {`
`15`		`- return MapEntry(PartIdConverter().toJson(partId), value.toJson());`
	`15`	`+ return MapEntry(const PartIdConverter().toJson(partId), value.toJson());`
`16`	`16`	`}`
`17`	`17`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,6 @@ class EmailMailboxIdsConverter {`
`6`	`6`	`MapEntry<MailboxId, bool> parseEntry(String key, bool value) => MapEntry(MailboxId(Id(key)), value);`
`7`	`7`
`8`	`8`	`MapEntry<String, bool> toJson(MailboxId mailboxId, bool value) {`
`9`		`- return MapEntry(IdConverter().toJson(mailboxId.id), value);`
	`9`	`+ return MapEntry(const IdConverter().toJson(mailboxId.id), value);`
`10`	`10`	`}`
`11`	`11`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ class MethodResponseConverter implements JsonConverter<MethodResponse, dynamic>`
`7`	`7`
`8`	`8`	`@override`
`9`	`9`	`MethodResponse fromJson(dynamic json) {`
`10`		`- return this.fromJson(json);`
	`10`	`+ return fromJson(json);`
`11`	`11`	`}`
`12`	`12`
`13`	`13`	`@override`
Original file line number	Diff line number	Diff line change
`@@ -20,9 +20,7 @@ mixin OptionalIds {`
`20`	`20`	`Set<Id>? ids;`
`21`	`21`
`22`	`22`	`void addIds(Set<Id> values) {`
`23`		`- if (ids == null) {`
`24`		`- ids = Set();`
`25`		`- }`
	`23`	`+ ids ??= <Id>{};`
`26`	`24`	`ids?.addAll(values);`
`27`	`25`	`}`
`28`	`26`	`}`
Original file line number	Diff line number	Diff line change
`@@ -54,9 +54,7 @@ mixin OptionalSort {`
`54`	`54`	`Set<Comparator>? sort;`
`55`	`55`
`56`	`56`	`void addSorts(Set<Comparator> value) {`
`57`		`- if (sort == null) {`
`58`		`- sort = Set();`
`59`		`- }`
	`57`	`+ sort ??= <Comparator>{};`
`60`	`58`	`sort?.addAll(value);`
`61`	`59`	`}`
`62`	`60`	`}`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ mixin OptionalIfInState {`
`22`	`22`	`State? ifInState;`
`23`	`23`
`24`	`24`	`void addIfInState(State? state) {`
`25`		`- this.ifInState = state;`
	`25`	`+ ifInState = state;`
`26`	`26`	`}`
`27`	`27`	`}`
`28`	`28`
`@@ -31,16 +31,12 @@ mixin OptionalCreate<T> {`
`31`	`31`	`Map<Id, T>? create;`
`32`	`32`
`33`	`33`	`void addCreates(Map<Id, T> creates) {`
`34`		`- if (create == null) {`
`35`		`- create = Map<Id, T>();`
`36`		`- }`
	`34`	`+ create ??= <Id, T>{};`
`37`	`35`	`create?.addAll(creates);`
`38`	`36`	`}`
`39`	`37`
`40`	`38`	`void addCreate(Id id, T createItem) {`
`41`		`- if (create == null) {`
`42`		`- create = Map<Id, T>();`
`43`		`- }`
	`39`	`+ create ??= <Id, T>{};`
`44`	`40`	`create?.addAll({id: createItem});`
`45`	`41`	`}`
`46`	`42`	`}`
`@@ -50,9 +46,7 @@ mixin OptionalUpdate {`
`50`	`46`	`Map<Id, PatchObject>? update;`
`51`	`47`
`52`	`48`	`void addUpdates(Map<Id, PatchObject> updates) {`
`53`		`- if (update == null) {`
`54`		`- update = Map<Id, PatchObject>();`
`55`		`- }`
	`49`	`+ update ??= <Id, PatchObject>{};`
`56`	`50`	`update?.addAll(updates);`
`57`	`51`	`}`
`58`	`52`	`}`
`@@ -62,9 +56,7 @@ mixin OptionalDestroy {`
`62`	`56`	`Set<Id>? destroy;`
`63`	`57`
`64`	`58`	`void addDestroy(Set<Id> values) {`
`65`		`- if (destroy == null) {`
`66`		`- destroy = Set();`
`67`		`- }`
	`59`	`+ destroy ??= <Id>{};`
`68`	`60`	`destroy?.addAll(values);`
`69`	`61`	`}`
`70`	`62`	`}`
`@@ -74,9 +66,7 @@ mixin OptionalUpdateSingleton<T> {`
`74`	`66`	`Map<Id, T>? updateSingleton;`
`75`	`67`
`76`	`68`	`void addUpdatesSingleton(Map<Id, T> updates) {`
`77`		`- if (updateSingleton == null) {`
`78`		`- updateSingleton = Map<Id, T>();`
`79`		`- }`
	`69`	`+ updateSingleton ??= <Id, T>{};`
`80`	`70`	`updateSingleton?.addAll(updates);`
`81`	`71`	`}`
`82`	`72`	`}`
Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,7 @@ class GetEmailMethod extends GetMethod with OptionalEmailBodyProperties, Optiona`
`37`	`37`
`38`	`38`	`factory GetEmailMethod.fromJson(Map<String, dynamic> json) => _$GetEmailMethodFromJson(json);`
`39`	`39`
	`40`	`+ @override`
`40`	`41`	`Map<String, dynamic> toJson() => _$GetEmailMethodToJson(this);`
`41`	`42`	`}`
`42`	`43`