TD-5291 Issue on 'Course set up' screen when selected Few course content first time and changed to 'All' of them clicking 'change link'

sherif-olaboye · sherif-olaboye · commit 4c1d42f401cd · 2025-01-30T13:31:08.000Z
diff --git a/DigitalLearningSolutions.Web/Controllers/TrackingSystem/CourseSetup/CourseSetupController.cs b/DigitalLearningSolutions.Web/Controllers/TrackingSystem/CourseSetup/CourseSetupController.cs
@@ -662,10 +662,13 @@ private IActionResult SaveSectionAndRedirect(SetSectionContentViewModel model)
                     sectionsToRemove.Add(section);
                 }
             }
+            if (sectionsToRemove.Count > 0)
+            { 
             foreach (var section in sectionsToRemove)
             {
                 data.SectionContentData.Remove(section);
             }
+            }
             data!.SectionContentData!.Add(
             new SectionContentTempData(
                 model.Tutorials != null

Original file line number	Diff line number	Diff line change
`@@ -662,10 +662,13 @@ private IActionResult SaveSectionAndRedirect(SetSectionContentViewModel model)`
`662`	`662`	`sectionsToRemove.Add(section);`
`663`	`663`	`}`
`664`	`664`	`}`
	`665`	`+ if (sectionsToRemove.Count > 0)`
	`666`	`+ {`
`665`	`667`	`foreach (var section in sectionsToRemove)`
`666`	`668`	`{`
`667`	`669`	`data.SectionContentData.Remove(section);`
`668`	`670`	`}`
	`671`	`+ }`
`669`	`672`	`data!.SectionContentData!.Add(`
`670`	`673`	`new SectionContentTempData(`
`671`	`674`	`model.Tutorials != null`