Merge pull request #1368 from TechnologyEnhancedLearning/Develop/Fixes/Removed-hardcorded-UserId

swapnamol-abraham · web-flow · commit 1e2d421c90fd · 2025-09-02T09:12:22.000+01:00
Removed the hard coded user id from the code
diff --git a/OpenAPI/LearningHub.Nhs.OpenApi.Services/Services/MoodleApiService.cs b/OpenAPI/LearningHub.Nhs.OpenApi.Services/Services/MoodleApiService.cs
@@ -115,8 +115,6 @@ public async Task<List<MoodleEnrolledCourseResponseModel>> GetRecentEnrolledCour
         {
             try
             {
-                userId = 3;
-
                 string statusFilter = string.Empty; ;
 
                 if ((requestModel.Incomplete && requestModel.Complete)  || (!requestModel.Incomplete && !requestModel.Complete))
@@ -167,7 +165,6 @@ public async Task<List<MoodleEnrolledCourseResponseModel>> GetEnrolledCoursesHis
         {
             try
             {
-                userId = 3;
                 string statusFilter = string.Empty;
 
                 if ((requestModel.Incomplete && requestModel.Complete) || (!requestModel.Incomplete && !requestModel.Complete))
@@ -217,7 +214,6 @@ public async Task<List<MoodleUserCertificateResponseModel>> GetUserCertificateAs
         {
             try
             {
-                userId = 3;
                 var parameters = new Dictionary<string, string>
             {
                 { "userid", userId.ToString() },

Original file line number	Diff line number	Diff line change
`@@ -115,8 +115,6 @@ public async Task<List<MoodleEnrolledCourseResponseModel>> GetRecentEnrolledCour`
`115`	`115`	`{`
`116`	`116`	`try`
`117`	`117`	`{`
`118`		`- userId = 3;`
`119`		`-`
`120`	`118`	`string statusFilter = string.Empty; ;`
`121`	`119`
`122`	`120`	`if ((requestModel.Incomplete && requestModel.Complete) \|\| (!requestModel.Incomplete && !requestModel.Complete))`
`@@ -167,7 +165,6 @@ public async Task<List<MoodleEnrolledCourseResponseModel>> GetEnrolledCoursesHis`
`167`	`165`	`{`
`168`	`166`	`try`
`169`	`167`	`{`
`170`		`- userId = 3;`
`171`	`168`	`string statusFilter = string.Empty;`
`172`	`169`
`173`	`170`	`if ((requestModel.Incomplete && requestModel.Complete) \|\| (!requestModel.Incomplete && !requestModel.Complete))`
`@@ -217,7 +214,6 @@ public async Task<List<MoodleUserCertificateResponseModel>> GetUserCertificateAs`
`217`	`214`	`{`
`218`	`215`	`try`
`219`	`216`	`{`
`220`		`- userId = 3;`
`221`	`217`	`var parameters = new Dictionary<string, string>`
`222`	`218`	`{`
`223`	`219`	`{ "userid", userId.ToString() },`