TD-4223-LastAccessDateSetToNullForNewCourses

Author: rshrirohit

DigitalLearningSolutions.Data.Migrations/202406140805_UpdateProgressMarkSubmittedTimeAsNullable.cs

@@ -0,0 +1,23 @@
+namespace DigitalLearningSolutions.Data.Migrations
+{
+    using FluentMigrator;
+
+    [Migration(202406140805)]
+    public class UpdateProgressMarkSubmittedTimeAsNullable : Migration
+    {
+        public override void Up()
+        {
+            Alter.Table("Progress")
+                .AlterColumn("SubmittedTime")
+                .AsDateTime()
+                .Nullable();
+        }
+        public override void Down()
+        {
+            Alter.Table("Progress")
+               .AlterColumn("SubmittedTime")
+               .AsDateTime()
+               .NotNullable();
+        }
+    }
+}

DigitalLearningSolutions.Data/DataServices/CourseDataService.cs

@@ -864,7 +864,7 @@ public IEnumerable<CourseStatistics> GetNonArchivedCourseStatisticsAtCentreFilte
 
         public IEnumerable<DelegateCourseInfo> GetDelegateCoursesInfo(int delegateId)
         {
-            return connection.Query<DelegateCourseInfo>(
+                return connection.Query<DelegateCourseInfo>(
                 $@"{selectDelegateCourseInfoQuery}
                     WHERE pr.CandidateID = @delegateId
                         AND pr.RemovedDate IS NULL
@@ -910,7 +910,7 @@ AND ap.DefaultContentTypeID <> 4
                         da.CentreID,
                         ap.ArchivedDate",
                 new { delegateId }
-            );
+            );            
         }
 
         public DelegateCourseInfo? GetDelegateCourseInfoByProgressId(int progressId)

DigitalLearningSolutions.Data/DataServices/ProgressDataService.cs

@@ -20,7 +20,7 @@ int CreateNewDelegateProgress(
             int delegateId,
             int customisationId,
             int customisationVersion,
-            DateTime submittedTime,
+            DateTime? submittedTime,
             int enrollmentMethodId,
             int? enrolledByAdminId,
             DateTime? completeByDate,
@@ -163,7 +163,7 @@ public int CreateNewDelegateProgress(
             int delegateId,
             int customisationId,
             int customisationVersion,
-            DateTime submittedTime,
+            DateTime? submittedTime,
             int enrollmentMethodId,
             int? enrolledByAdminId,
             DateTime? completeByDate,

DigitalLearningSolutions.Data/Models/Courses/DelegateCourseInfo.cs

@@ -68,7 +68,7 @@ public DelegateCourseInfo(
             string? supervisorSurname,
             bool? supervisorAdminActive,
             DateTime enrolled,
-            DateTime lastUpdated,
+            DateTime? lastUpdated,
             DateTime? completeBy,
             DateTime? completed,
             DateTime? evaluated,
@@ -145,7 +145,7 @@ bool isDelegateActive
         public bool AllCentresCourse { get; set; }
         public int ProgressId { get; set; }
         public bool IsProgressLocked { get; set; }
-        public DateTime LastUpdated { get; set; }
+        public DateTime? LastUpdated { get; set; }
         public DateTime? CompleteBy { get; set; }
         public DateTime? RemovedDate { get; set; }
         public DateTime? Completed { get; set; }

DigitalLearningSolutions.Web/Services/CourseDelegatesDownloadFileService.cs

@@ -547,7 +547,7 @@ DataRow row
             }
             row[DelegateId] = courseDelegate.CandidateNumber;
             row[Enrolled] = courseDelegate.Enrolled.Date;
-            row[LastAccessed] = courseDelegate.LastUpdated.Date;
+            row[LastAccessed] = courseDelegate.LastUpdated?.Date;
             row[CompleteBy] = courseDelegate.CompleteBy?.Date;
             row[CompletedDate] = courseDelegate.Completed?.Date;
             row[Logins] = courseDelegate.LoginCount;

DigitalLearningSolutions.Web/Services/EnrolService.cs

@@ -114,7 +114,7 @@ public void EnrolDelegateOnCourse(int delegateId, int customisationId, int custo
                     delegateId,
                     customisationId,
                     customisationVersion,
-                    clockUtility.UtcNow,
+                    null,
                     2,
                     enrolledByAdminId,
                 completeByDate,

DigitalLearningSolutions.Web/ViewModels/TrackingSystem/Delegates/Shared/DelegateCourseInfoViewModel.cs

@@ -64,7 +64,7 @@ private DelegateCourseInfoViewModel(DelegateCourseInfo info)
                 : "None";
 
             CompleteBy = info.CompleteBy?.ToString(DateHelper.StandardDateAndTimeFormat);
-            LastAccessed = info.LastUpdated.ToString(DateHelper.StandardDateAndTimeFormat);
+            LastAccessed = info.LastUpdated?.ToString(DateHelper.StandardDateAndTimeFormat);
             Completed = info.Completed?.ToString(DateHelper.StandardDateAndTimeFormat);
             Evaluated = info.Evaluated?.ToString(DateHelper.StandardDateAndTimeFormat);
             RemovedDate = info.RemovedDate?.ToString(DateHelper.StandardDateAndTimeFormat);

DigitalLearningSolutions.Web/Views/TrackingSystem/Delegates/Shared/_DelegateCourseProgressDetailsWithStatusTags.cshtml

@@ -79,7 +79,7 @@
             Last access
         </dt>
         <dd class="nhsuk-summary-list__value" data-name-for-sorting="last-accessed">
-            @Model.LastAccessed
+            @(Model.LastAccessed ?? "-")
         </dd>
         <dd class="nhsuk-summary-list__actions"></dd>
     </div>