From 3bd05718e0bc9ca2cc35112a3d3afceb4ad294a7 Mon Sep 17 00:00:00 2001 From: Andrew Horth Date: Thu, 14 Sep 2023 17:45:22 +0100 Subject: [PATCH 1/3] Added ability to specify sort order for person/contact search results --- .../Dqt/Models/ContactSearchSortByOption.cs | 19 +++++ .../Queries/GetContactsByDateOfBirthQuery.cs | 2 +- .../Dqt/Queries/GetContactsByNameQuery.cs | 2 +- .../GetContactsByDateOfBirthHandler.cs | 23 ++++++ .../QueryHandlers/GetContactsByNameHandler.cs | 26 ++++++- .../Pages/Persons/Index.cshtml | 24 ++++-- .../Pages/Persons/Index.cshtml.cs | 12 +-- .../Pages/Persons/PersonDetail/Index.cshtml | 8 +- .../Persons/PersonDetail/Index.cshtml.cs | 3 + .../TrsLinkGenerator.cs | 8 +- .../GetContactsByDateOfBirthTests.cs | 73 +++++++++++++++++- .../QueryTests/GetContactsByNameTests.cs | 76 +++++++++++++++++-- 12 files changed, 245 insertions(+), 31 deletions(-) create mode 100644 TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Models/ContactSearchSortByOption.cs diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Models/ContactSearchSortByOption.cs b/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Models/ContactSearchSortByOption.cs new file mode 100644 index 000000000..d31e28ed4 --- /dev/null +++ b/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Models/ContactSearchSortByOption.cs @@ -0,0 +1,19 @@ +using System.ComponentModel.DataAnnotations; + +namespace TeachingRecordSystem.Core.Dqt.Models; + +public enum ContactSearchSortByOption +{ + [Display(Name = "Last name (A-Z)")] + LastNameAscending, + [Display(Name = "Last name (Z-A)")] + LastNameDescending, + [Display(Name = "First name (A-Z)")] + FirstNameAscending, + [Display(Name = "First name (Z-A)")] + FirstNameDescending, + [Display(Name = "Date of birth (ascending)")] + DateOfBirthAscending, + [Display(Name = "Date of birth (descending)")] + DateOfBirthDescending +} diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Queries/GetContactsByDateOfBirthQuery.cs b/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Queries/GetContactsByDateOfBirthQuery.cs index c88df54ff..d1cf1fd70 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Queries/GetContactsByDateOfBirthQuery.cs +++ b/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Queries/GetContactsByDateOfBirthQuery.cs @@ -2,4 +2,4 @@ namespace TeachingRecordSystem.Core.Dqt.Queries; -public record GetContactsByDateOfBirthQuery(DateOnly DateOfBirth, int MaxRecordCount, ColumnSet ColumnSet) : ICrmQuery; +public record GetContactsByDateOfBirthQuery(DateOnly DateOfBirth, ContactSearchSortByOption SortBy, int MaxRecordCount, ColumnSet ColumnSet) : ICrmQuery; diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Queries/GetContactsByNameQuery.cs b/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Queries/GetContactsByNameQuery.cs index e7f972525..a16570906 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Queries/GetContactsByNameQuery.cs +++ b/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/Queries/GetContactsByNameQuery.cs @@ -2,4 +2,4 @@ namespace TeachingRecordSystem.Core.Dqt.Queries; -public record GetContactsByNameQuery(string Name, int MaxRecordCount, ColumnSet ColumnSet) : ICrmQuery; +public record GetContactsByNameQuery(string Name, ContactSearchSortByOption SortBy, int MaxRecordCount, ColumnSet ColumnSet) : ICrmQuery; diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/QueryHandlers/GetContactsByDateOfBirthHandler.cs b/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/QueryHandlers/GetContactsByDateOfBirthHandler.cs index 7c13c08b5..8f03be99c 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/QueryHandlers/GetContactsByDateOfBirthHandler.cs +++ b/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/QueryHandlers/GetContactsByDateOfBirthHandler.cs @@ -16,6 +16,29 @@ public async Task Execute(GetContactsByDateOfBirthQuery query, IOrgan }; queryByAttribute.AddAttributeValue(Contact.Fields.StateCode, (int)ContactState.Active); queryByAttribute.AddAttributeValue(Contact.Fields.BirthDate, query.DateOfBirth.ToDateTime()); + switch (query.SortBy) + { + case ContactSearchSortByOption.LastNameAscending: + queryByAttribute.AddOrder(Contact.Fields.LastName, OrderType.Ascending); + break; + case ContactSearchSortByOption.LastNameDescending: + queryByAttribute.AddOrder(Contact.Fields.LastName, OrderType.Descending); + break; + case ContactSearchSortByOption.FirstNameAscending: + queryByAttribute.AddOrder(Contact.Fields.FirstName, OrderType.Ascending); + break; + case ContactSearchSortByOption.FirstNameDescending: + queryByAttribute.AddOrder(Contact.Fields.FirstName, OrderType.Descending); + break; + case ContactSearchSortByOption.DateOfBirthAscending: + queryByAttribute.AddOrder(Contact.Fields.BirthDate, OrderType.Ascending); + break; + case ContactSearchSortByOption.DateOfBirthDescending: + queryByAttribute.AddOrder(Contact.Fields.BirthDate, OrderType.Descending); + break; + default: + break; + } var response = await organizationService.RetrieveMultipleAsync(queryByAttribute); return response.Entities.Select(e => e.ToEntity()).ToArray(); diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/QueryHandlers/GetContactsByNameHandler.cs b/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/QueryHandlers/GetContactsByNameHandler.cs index 91dc23206..d336527a7 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/QueryHandlers/GetContactsByNameHandler.cs +++ b/TeachingRecordSystem/src/TeachingRecordSystem.Core/Dqt/QueryHandlers/GetContactsByNameHandler.cs @@ -18,9 +18,33 @@ public async Task Execute(GetContactsByNameQuery query, IOrganization { ColumnSet = query.ColumnSet, Criteria = filter, - TopCount = query.MaxRecordCount + TopCount = query.MaxRecordCount, }; + switch (query.SortBy) + { + case ContactSearchSortByOption.LastNameAscending: + queryExpression.AddOrder(Contact.Fields.LastName, OrderType.Ascending); + break; + case ContactSearchSortByOption.LastNameDescending: + queryExpression.AddOrder(Contact.Fields.LastName, OrderType.Descending); + break; + case ContactSearchSortByOption.FirstNameAscending: + queryExpression.AddOrder(Contact.Fields.FirstName, OrderType.Ascending); + break; + case ContactSearchSortByOption.FirstNameDescending: + queryExpression.AddOrder(Contact.Fields.FirstName, OrderType.Descending); + break; + case ContactSearchSortByOption.DateOfBirthAscending: + queryExpression.AddOrder(Contact.Fields.BirthDate, OrderType.Ascending); + break; + case ContactSearchSortByOption.DateOfBirthDescending: + queryExpression.AddOrder(Contact.Fields.BirthDate, OrderType.Descending); + break; + default: + break; + } + var response = await organizationService.RetrieveMultipleAsync(queryExpression); return response.Entities.Select(e => e.ToEntity()).ToArray(); } diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml index 8961141ac..166932c84 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml +++ b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml @@ -1,4 +1,5 @@ @page "/persons" +@using TeachingRecordSystem.Core.Dqt.Models; @model TeachingRecordSystem.SupportUi.Pages.Persons.IndexModel @{ ViewBag.Title = "All records"; @@ -10,7 +11,7 @@
-
+
Search by: Name, TRN or date of birth (DD/MM/YYYY) Search -
+
+ + Sort by + @ContactSearchSortByOption.LastNameAscending.GetDisplayName() + @ContactSearchSortByOption.LastNameDescending.GetDisplayName() + @ContactSearchSortByOption.FirstNameAscending.GetDisplayName() + @ContactSearchSortByOption.FirstNameDescending.GetDisplayName() + @ContactSearchSortByOption.DateOfBirthAscending.GetDisplayName() + @ContactSearchSortByOption.DateOfBirthDescending.GetDisplayName() +
@@ -33,12 +43,12 @@ @if (Model.PreviousPage.HasValue) { - + } @if (Model.NextPage.HasValue) { - + } } @@ -64,7 +74,7 @@ @foreach (var personInfo in Model.SearchResults!) { - @personInfo.Name + @personInfo.Name @(personInfo.DateOfBirth.HasValue ? personInfo.DateOfBirth.Value.ToString("dd/MM/yyyy") : "-") @(!string.IsNullOrEmpty(personInfo.Trn) ? personInfo.Trn : "-") @(!string.IsNullOrEmpty(personInfo.NationalInsuranceNumber) ? personInfo.NationalInsuranceNumber : "-") @@ -79,12 +89,12 @@ @if (Model.PreviousPage.HasValue) { - + } @if (Model.NextPage.HasValue) { - + } } diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs index 2e25eec96..7f996caa8 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs +++ b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs @@ -31,6 +31,9 @@ public IndexModel( [Display(Name = "Search")] public string? Search { get; set; } + [BindProperty(SupportsGet = true)] + public ContactSearchSortByOption SortBy { get; set; } + [BindProperty(SupportsGet = true)] public int? PageNumber { get; set; } @@ -72,15 +75,12 @@ private async Task PerformSearch() Contact.Fields.dfeta_StatedFirstName, Contact.Fields.dfeta_StatedMiddleName, Contact.Fields.dfeta_StatedLastName, - Contact.Fields.EMailAddress1, - Contact.Fields.MobilePhone, - Contact.Fields.dfeta_NINumber, - Contact.Fields.dfeta_ActiveSanctions); + Contact.Fields.dfeta_NINumber); // Check if the search string is a date of birth, TRN or one or more names if (DateOnly.TryParse(Search, out var dateOfBirth)) { - contacts = await _crmQueryDispatcher.ExecuteQuery(new GetContactsByDateOfBirthQuery(dateOfBirth, MaxSearchResultCount, columnSet)); + contacts = await _crmQueryDispatcher.ExecuteQuery(new GetContactsByDateOfBirthQuery(dateOfBirth, SortBy, MaxSearchResultCount, columnSet)); } else if (TrnRegex().IsMatch(Search!)) { @@ -92,7 +92,7 @@ private async Task PerformSearch() } else { - contacts = await _crmQueryDispatcher.ExecuteQuery(new GetContactsByNameQuery(Search!, MaxSearchResultCount, columnSet)); + contacts = await _crmQueryDispatcher.ExecuteQuery(new GetContactsByNameQuery(Search!, SortBy, MaxSearchResultCount, columnSet)); } TotalKnownPages = Math.Max((int)Math.Ceiling((decimal)contacts!.Length / PageSize), 1); diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/PersonDetail/Index.cshtml b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/PersonDetail/Index.cshtml index 40528ab1c..984202b15 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/PersonDetail/Index.cshtml +++ b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/PersonDetail/Index.cshtml @@ -5,7 +5,7 @@ } @section BeforeContent { - + }

@ViewBag.Title

@@ -16,15 +16,15 @@ diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/PersonDetail/Index.cshtml.cs b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/PersonDetail/Index.cshtml.cs index e4d56e205..bd8883772 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/PersonDetail/Index.cshtml.cs +++ b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/PersonDetail/Index.cshtml.cs @@ -24,6 +24,9 @@ public IndexModel(ICrmQueryDispatcher crmQueryDispatcher) [FromQuery] public int? PageNumber { get; set; } + [FromQuery] + public ContactSearchSortByOption SortBy { get; set; } + [FromQuery] public PersonSubNavigationTab? SelectedTab { get; set; } diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/TrsLinkGenerator.cs b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/TrsLinkGenerator.cs index 35487799d..675cba515 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/TrsLinkGenerator.cs +++ b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/TrsLinkGenerator.cs @@ -1,3 +1,4 @@ +using TeachingRecordSystem.Core.Dqt.Models; using static TeachingRecordSystem.SupportUi.Pages.Persons.PersonDetail.IndexModel; namespace TeachingRecordSystem.SupportUi; @@ -27,10 +28,11 @@ public TrsLinkGenerator(LinkGenerator linkGenerator) public string RejectCase(string ticketNumber) => GetRequiredPathByPage("/Cases/EditCase/Reject", routeValues: new { ticketNumber }); - public string Persons(string? search = null, int? pageNumber = null) => GetRequiredPathByPage("/Persons/Index", routeValues: new { search, pageNumber }); + public string Persons(string? search = null, ContactSearchSortByOption? sortBy = null, int? pageNumber = null) => + GetRequiredPathByPage("/Persons/Index", routeValues: new { search, sortBy, pageNumber }); - public string PersonDetail(Guid personId, PersonSubNavigationTab? selectedTab = null, string? search = null, int? pageNumber = null) => - GetRequiredPathByPage("/Persons/PersonDetail/Index", routeValues: new { personId, selectedTab, search, pageNumber }); + public string PersonDetail(Guid personId, PersonSubNavigationTab? selectedTab = null, string? search = null, ContactSearchSortByOption? sortBy = null, int? pageNumber = null) => + GetRequiredPathByPage("/Persons/PersonDetail/Index", routeValues: new { personId, selectedTab, search, sortBy, pageNumber }); public string Users() => GetRequiredPathByPage("/Users/Index"); diff --git a/TeachingRecordSystem/tests/TeachingRecordSystem.Core.Dqt.Tests/QueryTests/GetContactsByDateOfBirthTests.cs b/TeachingRecordSystem/tests/TeachingRecordSystem.Core.Dqt.Tests/QueryTests/GetContactsByDateOfBirthTests.cs index dcae18c4e..46316ef69 100644 --- a/TeachingRecordSystem/tests/TeachingRecordSystem.Core.Dqt.Tests/QueryTests/GetContactsByDateOfBirthTests.cs +++ b/TeachingRecordSystem/tests/TeachingRecordSystem.Core.Dqt.Tests/QueryTests/GetContactsByDateOfBirthTests.cs @@ -17,22 +17,89 @@ public GetContactsByDateOfBirthTests(CrmClientFixture crmClientFixture) public async Task DisposeAsync() => await _dataScope.DisposeAsync(); - [Fact] - public async Task ReturnsMatchingContactsFromCrm() + public static TheoryData GetContactSearchSortScenarioData() + { + return new TheoryData + { + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.LastNameAscending, + Selector = (Contact c) => c.LastName, + IsAscending = true + }, + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.LastNameDescending, + Selector = (Contact c) => c.LastName, + IsAscending = false + }, + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.FirstNameAscending, + Selector = (Contact c) => c.FirstName, + IsAscending = true + }, + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.FirstNameDescending, + Selector = (Contact c) => c.FirstName, + IsAscending = false + }, + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.DateOfBirthAscending, + Selector = (Contact c) => c.BirthDate is null ? string.Empty : c.BirthDate.Value.ToString("yyyy-MM-dd"), + IsAscending = true + }, + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.DateOfBirthDescending, + Selector = (Contact c) => c.BirthDate is null ? string.Empty : c.BirthDate.Value.ToString("yyyy-MM-dd"), + IsAscending = false + } + }; + } + + [Theory] + [MemberData(nameof(GetContactSearchSortScenarioData))] + public async Task ReturnsMatchingContactsFromCrmInExpectedSortOrder(ContactSearchSortScenarioData testScenarioData) { // Arrange var dateOfBirth = new DateOnly(1990, 1, 1); var maxRecordCount = 3; + var columnSet = new ColumnSet( + Contact.Fields.dfeta_TRN, + Contact.Fields.BirthDate, + Contact.Fields.FirstName, + Contact.Fields.MiddleName, + Contact.Fields.LastName, + Contact.Fields.FullName); + var person1 = await _dataScope.TestData.CreatePerson(b => b.WithDateOfBirth(dateOfBirth)); var person2 = await _dataScope.TestData.CreatePerson(b => b.WithDateOfBirth(dateOfBirth)); var person3 = await _dataScope.TestData.CreatePerson(b => b.WithDateOfBirth(dateOfBirth)); // Act - var results = await _crmQueryDispatcher.ExecuteQuery(new GetContactsByDateOfBirthQuery(dateOfBirth, maxRecordCount, new ColumnSet())); + var results = await _crmQueryDispatcher.ExecuteQuery(new GetContactsByDateOfBirthQuery(dateOfBirth, testScenarioData.SortBy, maxRecordCount, columnSet)); // Assert Assert.NotNull(results); Assert.Equal(maxRecordCount, results.Length); + if (testScenarioData.IsAscending) + { + Assert.Equal(results.Select(testScenarioData.Selector).OrderBy(x => x).ToArray(), results.Select(testScenarioData.Selector).ToArray()); + } + else + { + Assert.Equal(results.Select(testScenarioData.Selector).OrderByDescending(x => x).ToArray(), results.Select(testScenarioData.Selector).ToArray()); + } + } + + public class ContactSearchSortScenarioData + { + public required ContactSearchSortByOption SortBy { get; init; } + public required Func Selector { get; init; } + public required bool IsAscending { get; init; } } } diff --git a/TeachingRecordSystem/tests/TeachingRecordSystem.Core.Dqt.Tests/QueryTests/GetContactsByNameTests.cs b/TeachingRecordSystem/tests/TeachingRecordSystem.Core.Dqt.Tests/QueryTests/GetContactsByNameTests.cs index 519d6a3db..1c5bcdf89 100644 --- a/TeachingRecordSystem/tests/TeachingRecordSystem.Core.Dqt.Tests/QueryTests/GetContactsByNameTests.cs +++ b/TeachingRecordSystem/tests/TeachingRecordSystem.Core.Dqt.Tests/QueryTests/GetContactsByNameTests.cs @@ -17,18 +17,84 @@ public GetContactsByNameTests(CrmClientFixture crmClientFixture) public async Task DisposeAsync() => await _dataScope.DisposeAsync(); - [Fact] - public async Task ReturnsMatchingContactsFromCrm() + public static TheoryData GetContactSearchSortScenarioData() + { + return new TheoryData + { + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.LastNameAscending, + Selector = (Contact c) => c.LastName, + IsAscending = true + }, + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.LastNameDescending, + Selector = (Contact c) => c.LastName, + IsAscending = false + }, + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.FirstNameAscending, + Selector = (Contact c) => c.FirstName, + IsAscending = true + }, + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.FirstNameDescending, + Selector = (Contact c) => c.FirstName, + IsAscending = false + }, + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.DateOfBirthAscending, + Selector = (Contact c) => c.BirthDate is null ? string.Empty : c.BirthDate.Value.ToString("yyyy-MM-dd"), + IsAscending = true + }, + new ContactSearchSortScenarioData + { + SortBy = ContactSearchSortByOption.DateOfBirthDescending, + Selector = (Contact c) => c.BirthDate is null ? string.Empty : c.BirthDate.Value.ToString("yyyy-MM-dd"), + IsAscending = false + } + }; + } + + [Theory] + [MemberData(nameof(GetContactSearchSortScenarioData))] + public async Task ReturnsMatchingContactsFromCrmInExpectedSortOrder(ContactSearchSortScenarioData testScenarioData) { // Arrange - var name = "smith"; - var maxRecordCount = 4; // pretty safe bet there will always be at least 4 smiths in the dev CRM database + var name = "andrew"; + var maxRecordCount = 4; // pretty safe bet there will always be at least 4 andrew names in the dev CRM database + var columnSet = new ColumnSet( + Contact.Fields.dfeta_TRN, + Contact.Fields.BirthDate, + Contact.Fields.FirstName, + Contact.Fields.MiddleName, + Contact.Fields.LastName, + Contact.Fields.FullName); // Act - var results = await _crmQueryDispatcher.ExecuteQuery(new GetContactsByNameQuery(name, maxRecordCount, new ColumnSet())); + var results = await _crmQueryDispatcher.ExecuteQuery(new GetContactsByNameQuery(name, testScenarioData.SortBy, maxRecordCount, columnSet)); // Assert Assert.NotNull(results); Assert.Equal(maxRecordCount, results.Length); + if (testScenarioData.IsAscending) + { + Assert.Equal(results.Select(testScenarioData.Selector).OrderBy(x => x).ToArray(), results.Select(testScenarioData.Selector).ToArray()); + } + else + { + Assert.Equal(results.Select(testScenarioData.Selector).OrderByDescending(x => x).ToArray(), results.Select(testScenarioData.Selector).ToArray()); + } + } + + public class ContactSearchSortScenarioData + { + public required ContactSearchSortByOption SortBy { get; init; } + public required Func Selector { get; init; } + public required bool IsAscending { get; init; } } } From 0d06e72ec409fdedc0788e6abd873dd1740dc17f Mon Sep 17 00:00:00 2001 From: Andrew Horth Date: Fri, 15 Sep 2023 14:05:56 +0100 Subject: [PATCH 2/3] Amended to show pages numbers where we know we have all possible search results returned --- .../Pages/Persons/Index.cshtml | 31 +++++++++---------- .../Pages/Persons/Index.cshtml.cs | 19 +++++++++++- 2 files changed, 33 insertions(+), 17 deletions(-) diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml index 166932c84..52d35da1a 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml +++ b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml @@ -22,7 +22,6 @@ Search - Sort by @ContactSearchSortByOption.LastNameAscending.GetDisplayName() @ContactSearchSortByOption.LastNameDescending.GetDisplayName() @ContactSearchSortByOption.FirstNameAscending.GetDisplayName() @@ -38,21 +37,6 @@ {
- @if (Model.TotalKnownPages > 1) - { - - @if (Model.PreviousPage.HasValue) - { - - } - - @if (Model.NextPage.HasValue) - { - - } - - } - @@ -92,6 +76,21 @@ } + @if (Model.DisplayPageNumbers) + { + @for (int i = 0; i < Model.PaginationPages!.Length; i++) + { + var item = Model.PaginationPages[i]; + + if (i > 0 && Model.PaginationPages[i - 1] != item - 1) + { + + } + + @item + } + } + @if (Model.NextPage.HasValue) { diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs index 7f996caa8..50ea5d783 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs +++ b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs @@ -32,19 +32,24 @@ public IndexModel( public string? Search { get; set; } [BindProperty(SupportsGet = true)] + [Display(Name = "Sort by")] public ContactSearchSortByOption SortBy { get; set; } [BindProperty(SupportsGet = true)] public int? PageNumber { get; set; } - public PersonInfo[]? SearchResults { get; set; } + public int[]? PaginationPages { get; set; } public int TotalKnownPages { get; set; } + public bool DisplayPageNumbers { get; set; } + public int? PreviousPage { get; set; } public int? NextPage { get; set; } + public PersonInfo[]? SearchResults { get; set; } + public async Task OnGet() { PageNumber ??= 1; @@ -100,6 +105,18 @@ private async Task PerformSearch() PreviousPage = PageNumber > 1 ? PageNumber - 1 : null; NextPage = PageNumber < TotalKnownPages ? PageNumber + 1 : null; + if (contacts.Length < MaxSearchResultCount) + { + DisplayPageNumbers = true; + PaginationPages = Enumerable.Range(-2, 5).Select(offset => PageNumber!.Value + offset) + .Append(1) + .Append(TotalKnownPages) + .Where(page => page <= TotalKnownPages && page >= 1) + .Distinct() + .Order() + .ToArray(); + } + SearchResults = contacts .Skip((PageNumber!.Value - 1) * PageSize) .Take(PageSize) From 12d1fdf491fce03602dfd4e6e5e2a92e6e57ca9b Mon Sep 17 00:00:00 2001 From: Andrew Horth Date: Mon, 18 Sep 2023 11:18:10 +0100 Subject: [PATCH 3/3] Update TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs Co-authored-by: James Gunn --- .../Pages/Persons/Index.cshtml.cs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs index 50ea5d783..8403d39af 100644 --- a/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs +++ b/TeachingRecordSystem/src/TeachingRecordSystem.SupportUi/Pages/Persons/Index.cshtml.cs @@ -109,12 +109,12 @@ private async Task PerformSearch() { DisplayPageNumbers = true; PaginationPages = Enumerable.Range(-2, 5).Select(offset => PageNumber!.Value + offset) - .Append(1) - .Append(TotalKnownPages) - .Where(page => page <= TotalKnownPages && page >= 1) - .Distinct() - .Order() - .ToArray(); + .Append(1) + .Append(TotalKnownPages) + .Where(page => page <= TotalKnownPages && page >= 1) + .Distinct() + .Order() + .ToArray(); } SearchResults = contacts