Skip to content

Commit bee6cdb

Browse files
author
Binon
committed
Added moodle invalid user check
1 parent 98e285d commit bee6cdb

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

LearningHub.Nhs.WebUI/Controllers/HomeController.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -217,6 +217,7 @@ public async Task<IActionResult> Index(string myLearningDashboard = "my-in-progr
217217
var enrolledCoursesTask = Task.FromResult(new List<MoodleCourseResponseViewModel>());
218218
var enableMoodle = Task.Run(() => this.featureManager.IsEnabledAsync(FeatureFlags.EnableMoodle)).Result;
219219
this.ViewBag.EnableMoodle = enableMoodle;
220+
this.ViewBag.ValidMoodleUser = this.CurrentMoodleUserId > 0;
220221
if (enableMoodle && myLearningDashboard == "my-enrolled-courses")
221222
{
222223
enrolledCoursesTask = this.dashboardService.GetEnrolledCoursesFromMoodleAsync(this.CurrentMoodleUserId, 1);

LearningHub.Nhs.WebUI/Views/Home/_MyAccessedLearningTray.cshtml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,8 @@
3232
TotalCount = Model.MyLearnings.TotalCount
3333
};
3434

35-
var EnableMoodle = this.ViewBag.EnableMoodle;
35+
var enableMoodle = this.ViewBag.EnableMoodle;
36+
var validMoodleUser = this.ViewBag.ValidMoodleUser;
3637
}
3738
<div class="dashboardTray">
3839
<h2>My accessed learning</h2>
@@ -64,7 +65,7 @@
6465
<a tabindex="0" class="subnavwhite-link text-nowrap"
6566
asp-controller="Home" asp-action="Index" asp-route-myLearningDashboard="my-catalogues" asp-route-resourceDashboard="@(Model.Resources.Type)" asp-route-catalogueDashboard="@Model.Catalogues.Type" asp-fragment="my-learning">Catalogues</a>
6667
</li>
67-
@if (EnableMoodle)
68+
@if (enableMoodle && validMoodleUser)
6869
{
6970
<li class="subnavwhite-item nhsuk-bg-grey @(Model.EnrolledCourses?.Count() > 0 ? "active" : string.Empty)">
7071
<a tabindex="0" class="subnavwhite-link text-nowrap"
@@ -103,7 +104,7 @@
103104
}
104105
</ul>
105106
}
106-
else if (EnableMoodle && Model.EnrolledCourses?.Count() > 0)
107+
else if (enableMoodle && validMoodleUser && Model.EnrolledCourses?.Count() > 0)
107108
{
108109
<ul class="nhsuk-grid-row nhsuk-card-group nhsuk-card-group--centred">
109110

0 commit comments

Comments
 (0)