Skip to content

Commit 27e131c

Browse files
authored
Merge pull request #24 from TechnologyEnhancedLearning/Develop/Fixes/TD-3744-FIXUnknownError
TD-3744 -Job role with slash in causes error during registration
2 parents cd47341 + 02ab8ad commit 27e131c

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

LearningHub.Nhs.UserApi/Controllers/JobRoleController.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,11 @@
55
using System.Linq;
66
using System.Threading.Tasks;
77
using elfhHub.Nhs.Models.Common;
8+
using LearningHub.Nhs.UserApi.Helpers;
89
using LearningHub.Nhs.UserApi.Services.Interface;
910
using Microsoft.AspNetCore.Authorization;
1011
using Microsoft.AspNetCore.Mvc;
12+
using Newtonsoft.Json.Linq;
1113

1214
/// <summary>
1315
/// The job role controller.
@@ -100,7 +102,7 @@ public async Task<IActionResult> GetFilteredWithStaffGroup(string filter)
100102
[Route("GetPagedFilteredWithStaffGroup/{filter}/{page}/{pageSize}")]
101103
public async Task<Tuple<int, List<JobRoleBasicViewModel>>> GetPagedFilteredWithStaffGroup(string filter, int page, int pageSize)
102104
{
103-
var list = await this.jobRoleService.GetFilteredWithStaffGroupAsync(filter);
105+
var list = await this.jobRoleService.GetFilteredWithStaffGroupAsync(filter.DecodeParameter());
104106
int total = list.Count;
105107
var pagedList = list.Skip((page - 1) * pageSize).Take(pageSize).ToList();
106108
return new Tuple<int, List<JobRoleBasicViewModel>>(total, pagedList);

0 commit comments

Comments
 (0)