Merge pull request #2713 from TechnologyEnhancedLearning/Develop/Fix/TD-4222-Supervisor-has-different-email

TD-4222-Enrolled Supervisor has different email address

Author: rshrirohit

DigitalLearningSolutions.Data/DataServices/SupervisorService.cs

@@ -393,7 +393,7 @@ LEFT JOIN UserCentreDetails ucd
         public IEnumerable<SupervisorForEnrolDelegate> GetSupervisorForEnrolDelegate(int CustomisationID, int CentreID)
         {
             return connection.Query<SupervisorForEnrolDelegate>(
-                $@"SELECT AdminID, Forename + ' ' + Surname + ' (' + CentreName +')' AS Name, Email FROM AdminUsers AS au
+                $@"SELECT AdminID, Forename + ' ' + Surname + ' (' + Email +'),' + ' ' + CentreName AS Name, Email FROM AdminUsers AS au
                     WHERE (Supervisor = 1) AND (CentreID = @CentreID) AND (CategoryID = 0 OR
                          CategoryID = (SELECT au.CategoryID FROM Applications AS a INNER JOIN
                            Customisations AS c ON a.ApplicationID = c.ApplicationID

DigitalLearningSolutions.Data/Models/SessionData/Tracking/Delegate/Enrol/SessionEnrolDelegate.cs

@@ -13,6 +13,7 @@ public class SessionEnrolDelegate
         public string? SelfAssessmentSupervisorRoleName { get; set; }
         public int? SupervisorID { get; set; }
         public string? SupervisorName { get; set; }
+        public string? SupervisorEmail { get; set; }
         public bool IsSelfAssessment { get; set; }
         public int AssessmentVersion { get; set; }
     }

DigitalLearningSolutions.Web/Controllers/TrackingSystem/Delegates/EnrolController.cs

@@ -256,13 +256,14 @@ public IActionResult EnrolDelegateSupervisor(int delegateId, EnrolSupervisorView
             {
                 sessionEnrol.SupervisorName = supervisorList.FirstOrDefault(x => x.AdminId == model.SelectedSupervisor).Name;
                 sessionEnrol.SupervisorID = model.SelectedSupervisor;
+                sessionEnrol.SupervisorEmail = supervisorList.FirstOrDefault(x => x.AdminId == model.SelectedSupervisor).Email;
             }
             if (model.SelectedSupervisorRoleId.HasValue && model.SelectedSupervisorRoleId.Value > 0)
             {
                 sessionEnrol.SelfAssessmentSupervisorRoleName = roles.FirstOrDefault(x => x.ID == model.SelectedSupervisorRoleId).RoleName;
             }
             sessionEnrol.SelfAssessmentSupervisorRoleId = model.SelectedSupervisorRoleId;
-            if (roles.Count()==1 && !string.IsNullOrEmpty(sessionEnrol.SupervisorName))
+            if (roles.Count() == 1 && !string.IsNullOrEmpty(sessionEnrol.SupervisorName))
             {
                 sessionEnrol.SelfAssessmentSupervisorRoleName = roles.FirstOrDefault().RoleName;
                 sessionEnrol.SelfAssessmentSupervisorRoleId = roles.FirstOrDefault().ID;
@@ -292,6 +293,7 @@ public IActionResult EnrolDelegateSummary(int delegateId)
 
             var model = new EnrolSummaryViewModel();
             model.SupervisorName = sessionEnrol.SupervisorName;
+            model.SupervisorEmail = sessionEnrol.SupervisorEmail;
             model.ActivityName = sessionEnrol.AssessmentName;
             model.CompleteByDate = sessionEnrol.CompleteByDate;
             model.DelegateId = delegateId;
@@ -316,12 +318,11 @@ public IActionResult EnrolDelegateSummary()
             }
             else
             {
-                var adminEmail = User.GetUserPrimaryEmailKnownNotNull();
                 var selfAssessmentId = enrolService.EnrolOnActivitySelfAssessment(
                     sessionEnrol.AssessmentID.GetValueOrDefault(),
                     delegateId,
                     sessionEnrol.SupervisorID.GetValueOrDefault(),
-                    adminEmail,
+                    sessionEnrol.SupervisorEmail,
                     sessionEnrol.SelfAssessmentSupervisorRoleId.GetValueOrDefault(),
                     sessionEnrol.CompleteByDate,
                     (int)sessionEnrol.DelegateUserID,

DigitalLearningSolutions.Web/ViewModels/TrackingSystem/Delegates/Enrol/EnrolDelegateSummaryViewModel.cs

@@ -9,6 +9,7 @@ public class EnrolSummaryViewModel
         public string? ActivityName { get; set; }
         public DateTime? CompleteByDate { get; set; }
         public string? SupervisorName { get; set; }
+        public string? SupervisorEmail { get; set; }
         public string? SupervisorRoleName { get; set; }
         public bool? IsMandatory { get; set; }
         public string? ValidFor { get; set; }