Skip to content

Commit 89ead46

Browse files
gnpricechrisbobbe
authored andcommitted
api [nfc]: Make all remaining constructor arguments required
This completes the sweep for the point of style we added to the README a few commits ago.
1 parent 8552608 commit 89ead46

File tree

5 files changed

+26
-26
lines changed

5 files changed

+26
-26
lines changed

lib/api/model/initial_snapshot.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,11 +73,11 @@ class InitialSnapshot {
7373
}
7474

7575
InitialSnapshot({
76-
this.queueId,
76+
required this.queueId,
7777
required this.lastEventId,
7878
required this.zulipFeatureLevel,
7979
required this.zulipVersion,
80-
this.zulipMergeBase,
80+
required this.zulipMergeBase,
8181
required this.alertWords,
8282
required this.customProfileFields,
8383
required this.recentPrivateConversations,

lib/api/model/model.dart

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ class User {
179179
required this.timezone,
180180
required this.avatarUrl,
181181
required this.avatarVersion,
182-
this.profileData,
182+
required this.profileData,
183183
required this.isSystemBot,
184184
});
185185

@@ -328,22 +328,22 @@ class Subscription {
328328
required this.renderedDescription,
329329
required this.dateCreated,
330330
required this.inviteOnly,
331-
this.desktopNotifications,
332-
this.emailNotifications,
333-
this.wildcardMentionsNotify,
334-
this.pushNotifications,
335-
this.audibleNotifications,
331+
required this.desktopNotifications,
332+
required this.emailNotifications,
333+
required this.wildcardMentionsNotify,
334+
required this.pushNotifications,
335+
required this.audibleNotifications,
336336
required this.pinToTop,
337337
required this.emailAddress,
338338
required this.isMuted,
339-
this.isWebPublic,
339+
required this.isWebPublic,
340340
required this.color,
341341
required this.streamPostPolicy,
342-
this.messageRetentionDays,
342+
required this.messageRetentionDays,
343343
required this.historyPublicToSubscribers,
344-
this.firstMessageId,
345-
this.streamWeeklyTraffic,
346-
this.canRemoveSubscribersGroupId,
344+
required this.firstMessageId,
345+
required this.streamWeeklyTraffic,
346+
required this.canRemoveSubscribersGroupId,
347347
});
348348

349349
factory Subscription.fromJson(Map<String, dynamic> json) =>
@@ -406,7 +406,7 @@ sealed class Message {
406406
required this.contentType,
407407
required this.id,
408408
required this.isMeMessage,
409-
this.lastEditTimestamp,
409+
required this.lastEditTimestamp,
410410
required this.reactions,
411411
required this.recipientId,
412412
required this.senderEmail,
@@ -416,8 +416,8 @@ sealed class Message {
416416
required this.subject,
417417
required this.timestamp,
418418
required this.flags,
419-
this.matchContent,
420-
this.matchSubject,
419+
required this.matchContent,
420+
required this.matchSubject,
421421
});
422422

423423
factory Message.fromJson(Map<String, dynamic> json) {
@@ -471,7 +471,7 @@ class StreamMessage extends Message {
471471
required super.contentType,
472472
required super.id,
473473
required super.isMeMessage,
474-
super.lastEditTimestamp,
474+
required super.lastEditTimestamp,
475475
required super.reactions,
476476
required super.recipientId,
477477
required super.senderEmail,
@@ -481,8 +481,8 @@ class StreamMessage extends Message {
481481
required super.subject,
482482
required super.timestamp,
483483
required super.flags,
484-
super.matchContent,
485-
super.matchSubject,
484+
required super.matchContent,
485+
required super.matchSubject,
486486
required this.displayRecipient,
487487
required this.streamId,
488488
});
@@ -573,7 +573,7 @@ class DmMessage extends Message {
573573
required super.contentType,
574574
required super.id,
575575
required super.isMeMessage,
576-
super.lastEditTimestamp,
576+
required super.lastEditTimestamp,
577577
required super.reactions,
578578
required super.recipientId,
579579
required super.senderEmail,
@@ -583,8 +583,8 @@ class DmMessage extends Message {
583583
required super.subject,
584584
required super.timestamp,
585585
required super.flags,
586-
super.matchContent,
587-
super.matchSubject,
586+
required super.matchContent,
587+
required super.matchSubject,
588588
required this.displayRecipient,
589589
});
590590

lib/api/route/account.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ class FetchApiKeyResult {
3232
FetchApiKeyResult({
3333
required this.apiKey,
3434
required this.email,
35-
this.userId,
35+
required this.userId,
3636
});
3737

3838
factory FetchApiKeyResult.fromJson(Map<String, dynamic> json) =>

lib/api/route/events.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ class GetEventsResult {
4141

4242
GetEventsResult({
4343
required this.events,
44-
this.queueId,
44+
required this.queueId,
4545
});
4646

4747
factory GetEventsResult.fromJson(Map<String, dynamic> json) =>

lib/api/route/realm.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ class GetServerSettingsResult {
5252
required this.authenticationMethods,
5353
required this.zulipFeatureLevel,
5454
required this.zulipVersion,
55-
this.zulipMergeBase,
55+
required this.zulipMergeBase,
5656
required this.pushNotificationsEnabled,
5757
required this.isIncompatible,
5858
required this.emailAuthEnabled,
@@ -61,7 +61,7 @@ class GetServerSettingsResult {
6161
required this.realmName,
6262
required this.realmIcon,
6363
required this.realmDescription,
64-
this.realmWebPublicAccessEnabled,
64+
required this.realmWebPublicAccessEnabled,
6565
});
6666

6767
factory GetServerSettingsResult.fromJson(Map<String, dynamic> json) =>

0 commit comments

Comments
 (0)