Merge pull request #1633 from TechnologyEnhancedLearning/Report_Deployment_Script

OluwatobiAwe · web-flow · commit eedeeb6f02df · 2026-01-07T16:17:48.000Z
bugfix for failing report update
diff --git a/LearningHub.Nhs.WebUI/Controllers/ReportsController.cs b/LearningHub.Nhs.WebUI/Controllers/ReportsController.cs
@@ -171,8 +171,7 @@ public async Task<IActionResult> CreateReportCourseSelection(ReportCreationCours
             }
 
             this.ModelState.AddModelError("Courses", CommonValidationErrorMessages.CourseRequired);
-            reportCreation.Courses = null;
-            await this.multiPageFormService.SetMultiPageFormData(reportCreation, MultiPageFormDataFeature.AddCustomWebForm("ReportWizardCWF"), this.TempData);
+
             courseSelection.BuildCourses(await this.GetCoursesAsync());
             courseSelection.Courses = reportCreation.Courses;
             this.ViewBag.ReturnUrl = returnUrl;

Original file line number	Diff line number	Diff line change
`@@ -171,8 +171,7 @@ public async Task<IActionResult> CreateReportCourseSelection(ReportCreationCours`
`171`	`171`	`}`
`172`	`172`
`173`	`173`	`this.ModelState.AddModelError("Courses", CommonValidationErrorMessages.CourseRequired);`
`174`		`- reportCreation.Courses = null;`
`175`		`- await this.multiPageFormService.SetMultiPageFormData(reportCreation, MultiPageFormDataFeature.AddCustomWebForm("ReportWizardCWF"), this.TempData);`
	`174`	`+`
`176`	`175`	`courseSelection.BuildCourses(await this.GetCoursesAsync());`
`177`	`176`	`courseSelection.Courses = reportCreation.Courses;`
`178`	`177`	`this.ViewBag.ReturnUrl = returnUrl;`