Skip to content

Commit 20ffea7

Browse files
committed
Not nullable should be empty list
1 parent bc43f74 commit 20ffea7

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

OpenAPI/LearningHub.Nhs.OpenApi.Models/ViewModels/ResourceMetadataViewModel.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public ResourceMetadataViewModel(
3434
string resourceType,
3535
int? majorVersion,
3636
decimal rating,
37-
List<MajorVersionIdActivityStatusDescription>? userSummaryActivityStatuses)
37+
List<MajorVersionIdActivityStatusDescription> userSummaryActivityStatuses)
3838
{
3939
this.ResourceId = resourceId;
4040
this.Title = title;
@@ -85,6 +85,6 @@ public ResourceMetadataViewModel(
8585
/// <summary>
8686
/// Gets or sets <see cref="UserSummaryActivityStatuses"/>.
8787
/// </summary>
88-
public List<MajorVersionIdActivityStatusDescription>? UserSummaryActivityStatuses { get; set; }
88+
public List<MajorVersionIdActivityStatusDescription> UserSummaryActivityStatuses { get; set; }
8989
}
9090
}

OpenAPI/LearningHub.Nhs.OpenApi.Models/ViewModels/ResourceReferenceWithResourceDetailsViewModel.cs

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,11 @@ public class ResourceReferenceWithResourceDetailsViewModel
1717
/// <param name="description"><see cref="Description"/>.</param>
1818
/// <param name="catalogueViewModel"><see cref="Catalogue"/>.</param>
1919
/// <param name="resourceType"><see cref="ResourceType"/>.</param>
20+
/// <param name="majorVersion"></param>
2021
/// <param name="rating"><see cref="Rating"/>.</param>
2122
/// <param name="link"><see cref="Link"/>.</param>
22-
public ResourceReferenceWithResourceDetailsViewModel( // qqqq may need major version id adding
23+
/// <param name="userSummaryActivityStatuses"></param>
24+
public ResourceReferenceWithResourceDetailsViewModel(
2325
int resourceId,
2426
int refId,
2527
string title,
@@ -29,7 +31,7 @@ public ResourceReferenceWithResourceDetailsViewModel( // qqqq may need major ver
2931
int? majorVersion,
3032
decimal rating,
3133
string link,
32-
List<MajorVersionIdActivityStatusDescription>? userSummaryActivityStatuses)
34+
List<MajorVersionIdActivityStatusDescription> userSummaryActivityStatuses)
3335
{
3436
this.ResourceId = resourceId;
3537
this.RefId = refId;
@@ -94,6 +96,6 @@ public ResourceReferenceWithResourceDetailsViewModel( // qqqq may need major ver
9496
/// <summary>
9597
/// Gets <see cref="UserSummaryActivityStatuses"/>.
9698
/// </summary>
97-
public List<MajorVersionIdActivityStatusDescription>? UserSummaryActivityStatuses { get; }
99+
public List<MajorVersionIdActivityStatusDescription> UserSummaryActivityStatuses { get; }
98100
}
99101
}

0 commit comments

Comments
 (0)