Merge pull request #3099 from TechnologyEnhancedLearning/Develop/Fixes/TD-5292-Issuewith'Goback'linkwhencreatingacourseandadding'Coursecontent'on'Coursesetup'screen

rshrirohit · web-flow · commit ec0e37c45c4b · 2025-02-11T09:19:45.000Z
TD-5292 Issue with 'Go back' link when creating a course and adding 'Course content' on 'Course set up' screen
diff --git a/DigitalLearningSolutions.Web/Controllers/TrackingSystem/CourseSetup/CourseSetupController.cs b/DigitalLearningSolutions.Web/Controllers/TrackingSystem/CourseSetup/CourseSetupController.cs
@@ -365,7 +365,8 @@ public IActionResult SetCourseOptions(EditCourseOptionsFormData model)
             data!.CourseOptionsData = model.ToCourseOptionsTempData();
             multiPageFormService.SetMultiPageFormData(data, MultiPageFormDataFeature.AddNewCourse, TempData);
 
-            return RedirectToAction("SetCourseContent", false);
+            bool editCourseContent = data?.EditCourseContent ?? false;
+            return RedirectToAction("SetCourseContent", editCourseContent);
         }
 
         [HttpGet("AddCourse/SetCourseContent")]
@@ -381,7 +382,7 @@ public IActionResult SetCourseContent(bool editCourseContent)
             {
                 return RedirectToAction("Summary");
             }
-            data.EditCourseContent = editCourseContent;
+            data.EditCourseContent = data.EditCourseContent || editCourseContent;
             multiPageFormService.SetMultiPageFormData(data, MultiPageFormDataFeature.AddNewCourse, TempData);
 
             var model = data!.CourseContentData != null

Original file line number	Diff line number	Diff line change
`@@ -365,7 +365,8 @@ public IActionResult SetCourseOptions(EditCourseOptionsFormData model)`
`365`	`365`	`data!.CourseOptionsData = model.ToCourseOptionsTempData();`
`366`	`366`	`multiPageFormService.SetMultiPageFormData(data, MultiPageFormDataFeature.AddNewCourse, TempData);`
`367`	`367`
`368`		`- return RedirectToAction("SetCourseContent", false);`
	`368`	`+ bool editCourseContent = data?.EditCourseContent ?? false;`
	`369`	`+ return RedirectToAction("SetCourseContent", editCourseContent);`
`369`	`370`	`}`
`370`	`371`
`371`	`372`	`[HttpGet("AddCourse/SetCourseContent")]`
`@@ -381,7 +382,7 @@ public IActionResult SetCourseContent(bool editCourseContent)`
`381`	`382`	`{`
`382`	`383`	`return RedirectToAction("Summary");`
`383`	`384`	`}`
`384`		`- data.EditCourseContent = editCourseContent;`
	`385`	`+ data.EditCourseContent = data.EditCourseContent \|\| editCourseContent;`
`385`	`386`	`multiPageFormService.SetMultiPageFormData(data, MultiPageFormDataFeature.AddNewCourse, TempData);`
`386`	`387`
`387`	`388`	`var model = data!.CourseContentData != null`