Skip to content

Commit cc67ddf

Browse files
authored
Merge pull request #3377 from TechnologyEnhancedLearning/Develop/Fix/TD-6235-Completeby-date-once-set-cannot-cleared
TD-6235- MultiPageFormData value for CompleteByDate is set to null
2 parents abad383 + fbb35da commit cc67ddf

File tree

1 file changed

+6
-12
lines changed
  • DigitalLearningSolutions.Web/Controllers/SupervisorController

1 file changed

+6
-12
lines changed

DigitalLearningSolutions.Web/Controllers/SupervisorController/Supervisor.cs

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -910,19 +910,13 @@ public IActionResult EnrolDelegateSetCompleteBy(int supervisorDelegateId, int da
910910
{
911911
var validationResult = OldDateValidator.ValidateDate(day, month, year);
912912
if (!validationResult.DateValid)
913-
{
914913
return RedirectToAction("EnrolDelegateCompleteBy", new { supervisorDelegateId, day, month, year });
915-
}
916-
else
917-
{
918-
var completeByDate = new DateTime(year, month, day);
919-
sessionEnrolOnRoleProfile.CompleteByDate = completeByDate;
920-
multiPageFormService.SetMultiPageFormData(
921-
sessionEnrolOnRoleProfile,
922-
MultiPageFormDataFeature.EnrolDelegateOnProfileAssessment,
923-
TempData
924-
);
925-
}
914+
915+
sessionEnrolOnRoleProfile.CompleteByDate = new DateTime(year, month, day);
916+
}
917+
else
918+
{
919+
sessionEnrolOnRoleProfile.CompleteByDate = null;
926920
}
927921

928922
var supervisorRoles =

0 commit comments

Comments
 (0)