Skip to content

Commit f6e5da1

Browse files
swapnamol-abrahamswapnamol-abraham
committed
Fixed the SIT issues
1 parent 31f8950 commit f6e5da1

File tree

2 files changed

+59
-58
lines changed

2 files changed

+59
-58
lines changed

‎OpenAPI/LearningHub.Nhs.OpenApi.Services/Services/MoodleApiService.cs‎

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ public MoodleApiService(IMoodleHttpClient moodleHttpClient, ILogger<MoodleApiSer
4242
/// <returns>UserId from Moodle.</returns>
4343
public async Task<int> GetMoodleUserIdByUsernameAsync(int currentUserId)
4444
{
45+
currentUserId = 2299585;
4546
var parameters = new Dictionary<string, string>
4647
{
4748
{ "criteria[0][key]", "username" },
@@ -115,7 +116,7 @@ public async Task<List<MoodleEnrolledCourseResponseModel>> GetRecentEnrolledCour
115116
{
116117
try
117118
{
118-
var moodleUser = this.GetMoodleUserIdByUsernameAsync(userId);
119+
int moodleUserId = await this.GetMoodleUserIdByUsernameAsync(userId);
119120
string statusFilter = string.Empty; ;
120121

121122
if ((requestModel.Incomplete && requestModel.Complete) || (!requestModel.Incomplete && !requestModel.Complete))
@@ -133,7 +134,7 @@ public async Task<List<MoodleEnrolledCourseResponseModel>> GetRecentEnrolledCour
133134

134135
var parameters = new Dictionary<string, string>
135136
{
136-
{ "userid", moodleUser.Id.ToString() },
137+
{ "userid", moodleUserId.ToString() },
137138
{ "months", months.ToString() },
138139
{ "statusfilter", statusFilter },
139140
{ "search", requestModel.SearchText ?? string.Empty }
@@ -166,7 +167,7 @@ public async Task<List<MoodleEnrolledCourseResponseModel>> GetEnrolledCoursesHis
166167
{
167168
try
168169
{
169-
var moodleUser = this.GetMoodleUserIdByUsernameAsync(userId);
170+
int moodleUserId = await this.GetMoodleUserIdByUsernameAsync(userId);
170171
string statusFilter = string.Empty;
171172

172173
if ((requestModel.Incomplete && requestModel.Complete) || (!requestModel.Incomplete && !requestModel.Complete))
@@ -184,7 +185,7 @@ public async Task<List<MoodleEnrolledCourseResponseModel>> GetEnrolledCoursesHis
184185

185186
var parameters = new Dictionary<string, string>
186187
{
187-
{ "userid", moodleUser.Id.ToString() },
188+
{ "userid", moodleUserId.ToString() },
188189
{ "statusfilter", statusFilter },
189190
{ "search", requestModel.SearchText ?? string.Empty }
190191
};
@@ -216,10 +217,10 @@ public async Task<List<MoodleUserCertificateResponseModel>> GetUserCertificateAs
216217
{
217218
try
218219
{
219-
var moodleUser = this.GetMoodleUserIdByUsernameAsync(userId);
220+
int moodleUserId = await this.GetMoodleUserIdByUsernameAsync(userId);
220221
var parameters = new Dictionary<string, string>
221222
{
222-
{ "userid", moodleUser.Id.ToString() },
223+
{ "userid", moodleUserId.ToString() },
223224
{ "searchterm", filterText }
224225
};
225226

‎OpenAPI/LearningHub.Nhs.OpenApi.Services/Services/MyLearningService.cs‎

Lines changed: 52 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -157,61 +157,61 @@ public async Task<MyLearningDetailedViewModel> GetActivityDetailed(int userId, M
157157
/// <returns>The <see cref="Task"/>.</returns>
158158
public async Task<MyLearningActivitiesDetailedViewModel> GetUserRecentMyLearningActivitiesAsync(int userId, MyLearningRequestModel requestModel)
159159
{
160-
var result = await resourceActivityRepository.GetUserRecentMyLearningActivities(userId, requestModel);
160+
var result = await resourceActivityRepository.GetUserRecentMyLearningActivities(userId, requestModel);
161161

162-
var entrolledCourses = await this.moodleApiService.GetRecentEnrolledCoursesAsync(userId, requestModel, 6);
162+
var entrolledCourses = await this.moodleApiService.GetRecentEnrolledCoursesAsync(userId, requestModel, 6);
163163

164-
var mappedMyLearningActivities = result.Select(Activity => new MyLearningCombinedActivitiesViewModel
165-
{
166-
UserId = userId,
167-
ResourceId = Activity.ResourceId,
168-
ResourceVersionId = Activity.ResourceVersionId,
169-
ResourceReferenceId = Activity.ResourceReferenceId,
170-
IsCurrentResourceVersion = Activity.IsCurrentResourceVersion,
171-
MajorVersion = Activity.MajorVersion,
172-
MinorVersion = Activity.MinorVersion,
173-
ResourceType = Activity.ResourceType,
174-
Title = Activity.Title,
175-
CertificateEnabled = Activity.CertificateEnabled,
176-
ActivityStatus = Activity.ActivityStatus,
177-
ActivityDate = Activity.ActivityDate,
178-
ScorePercentage = Activity.ScorePercentage,
179-
TotalActivities = 0,
180-
CompletedActivities = 0,
181-
}).ToList();
182-
183-
var mappedEnrolledCourses = entrolledCourses.Select(course => new MyLearningCombinedActivitiesViewModel
184-
{
185-
UserId = userId,
186-
ResourceId = (int)course.Id,
187-
ResourceVersionId = (int)course.Id,
188-
IsCurrentResourceVersion = true,
189-
ResourceReferenceId = (int)course.Id,
190-
MajorVersion = 1,
191-
MinorVersion = 0,
192-
ResourceType = ResourceTypeEnum.Moodle,
193-
Title = course.DisplayName,
194-
CertificateEnabled = course.CertificateEnabled,
195-
ActivityStatus = (course.Completed == true || course.ProgressPercentage.TrimEnd('%') == "100") ? ActivityStatusEnum.Completed : ActivityStatusEnum.Incomplete,
196-
ActivityDate = DateTimeOffset.FromUnixTimeMilliseconds((long)course.LastAccess),
197-
ScorePercentage = Convert.ToInt32(course.ProgressPercentage.TrimEnd('%')),
198-
TotalActivities = course.TotalActivities,
199-
CompletedActivities = course.CompletedActivities,
200-
}).ToList();
201-
202-
// Combine both result sets
203-
var combainedUserActivities = mappedMyLearningActivities.Concat(mappedEnrolledCourses).ToList();
204-
205-
var pagedResults = combainedUserActivities.OrderByDescending(activity => activity.ActivityDate).Skip(requestModel.Skip).Take(requestModel.Take).ToList();
164+
var mappedMyLearningActivities = result.Select(Activity => new MyLearningCombinedActivitiesViewModel
165+
{
166+
UserId = userId,
167+
ResourceId = Activity.ResourceId,
168+
ResourceVersionId = Activity.ResourceVersionId,
169+
ResourceReferenceId = Activity.ResourceReferenceId,
170+
IsCurrentResourceVersion = Activity.IsCurrentResourceVersion,
171+
MajorVersion = Activity.MajorVersion,
172+
MinorVersion = Activity.MinorVersion,
173+
ResourceType = Activity.ResourceType,
174+
Title = Activity.Title,
175+
CertificateEnabled = Activity.CertificateEnabled,
176+
ActivityStatus = Activity.ActivityStatus,
177+
ActivityDate = Activity.ActivityDate,
178+
ScorePercentage = Activity.ScorePercentage,
179+
TotalActivities = 0,
180+
CompletedActivities = 0,
181+
}).ToList();
182+
183+
var mappedEnrolledCourses = entrolledCourses.Select(course => new MyLearningCombinedActivitiesViewModel
184+
{
185+
UserId = userId,
186+
ResourceId = (int)course.Id,
187+
ResourceVersionId = (int)course.Id,
188+
IsCurrentResourceVersion = true,
189+
ResourceReferenceId = (int)course.Id,
190+
MajorVersion = 1,
191+
MinorVersion = 0,
192+
ResourceType = ResourceTypeEnum.Moodle,
193+
Title = course.DisplayName,
194+
CertificateEnabled = course.CertificateEnabled,
195+
ActivityStatus = (course.Completed == true || course.ProgressPercentage.TrimEnd('%') == "100") ? ActivityStatusEnum.Completed : ActivityStatusEnum.Incomplete,
196+
ActivityDate = DateTimeOffset.FromUnixTimeMilliseconds(course.LastAccess ?? 0),
197+
ScorePercentage = Convert.ToInt32(course.ProgressPercentage.TrimEnd('%')),
198+
TotalActivities = course.TotalActivities,
199+
CompletedActivities = course.CompletedActivities,
200+
}).ToList();
206201

207-
// Count total records.
208-
MyLearningActivitiesDetailedViewModel viewModel = new MyLearningActivitiesDetailedViewModel()
209-
{
210-
TotalCount = combainedUserActivities.Count(),
211-
Activities = pagedResults,
212-
};
202+
// Combine both result sets
203+
var combainedUserActivities = mappedMyLearningActivities.Concat(mappedEnrolledCourses).ToList();
213204

214-
return viewModel;
205+
var pagedResults = combainedUserActivities.OrderByDescending(activity => activity.ActivityDate).Skip(requestModel.Skip).Take(requestModel.Take).ToList();
206+
207+
// Count total records.
208+
MyLearningActivitiesDetailedViewModel viewModel = new MyLearningActivitiesDetailedViewModel()
209+
{
210+
TotalCount = combainedUserActivities.Count(),
211+
Activities = pagedResults,
212+
};
213+
214+
return viewModel;
215215
}
216216

217217
/// <summary>
@@ -298,7 +298,7 @@ public async Task<MyLearningActivitiesDetailedViewModel> GetUserLearningHistoryA
298298
Title = course.DisplayName,
299299
CertificateEnabled = course.CertificateEnabled,
300300
ActivityStatus = (course.Completed == true || course.ProgressPercentage.TrimEnd('%') == "100") ? ActivityStatusEnum.Completed : ActivityStatusEnum.Incomplete,
301-
ActivityDate = DateTimeOffset.FromUnixTimeMilliseconds((long)course.LastAccess),
301+
ActivityDate = DateTimeOffset.FromUnixTimeMilliseconds(course.LastAccess ?? 0),
302302
ScorePercentage = int.TryParse(course.ProgressPercentage.TrimEnd('%'), out var score) ? score : 0,
303303
TotalActivities = course.TotalActivities,
304304
CompletedActivities = course.CompletedActivities,

0 commit comments

Comments
 (0)