From be5ffdd4b3b53c7111592233025ab493c2133f4b Mon Sep 17 00:00:00 2001
From: Malte Berg <malte.berg@capgemini.com>
Date: Fri, 6 Oct 2023 14:29:15 +0200
Subject: [PATCH] add groupIds

---
 controllers/courses.js | 24 +++++++++++++++++++++---
 1 file changed, 21 insertions(+), 3 deletions(-)

diff --git a/controllers/courses.js b/controllers/courses.js
index b11d7cf18a..feb11b4143 100644
--- a/controllers/courses.js
+++ b/controllers/courses.js
@@ -108,6 +108,7 @@ const deleteEventsForCourse = (req, res, courseId) => {
 	return Promise.resolve(true);
 };
 
+let groups = [];
 const editCourseHandler = (req, res, next) => {
 	let coursePromise;
 	let action;
@@ -167,9 +168,16 @@ const editCourseHandler = (req, res, next) => {
 		scopePermissions,
 	]).then(([course, _classes, _teachers, _students, _scopePermissions]) => {
 		// these 3 might not change anything because hooks allow just ownSchool results by now, but to be sure:
-		const classes = FEATURE_GROUPS_IN_COURSE_ENABLED ? _classes.data : _classes.filter(
-			(c) => c.schoolId === res.locals.currentSchool,
-		).sort();
+		let classes = [];
+		if (FEATURE_GROUPS_IN_COURSE_ENABLED) {
+			classes = _classes.data;
+			groups = _classes.data.filter((c) => c.externalSourceName !== undefined);
+		} else {
+			classes = _classes.filter(
+				(c) => c.schoolId === res.locals.currentSchool,
+			).sort();
+		}
+
 		const teachers = _teachers.filter(
 			(t) => t.schoolId === res.locals.currentSchool,
 		);
@@ -746,6 +754,9 @@ router.patch('/:courseId', async (req, res, next) => {
 		if (!req.body.classIds) {
 			req.body.classIds = [];
 		}
+		if (!req.body.groupIds) {
+			req.body.groupIds = [];
+		}
 		if (!req.body.userIds) {
 			req.body.userIds = [];
 		}
@@ -753,6 +764,13 @@ router.patch('/:courseId', async (req, res, next) => {
 			req.body.substitutionIds = [];
 		}
 
+		if (FEATURE_GROUPS_IN_COURSE_ENABLED) {
+			req.body.groupIds = req.body.classIds
+				.filter((id) => groups.some((group) => group.id === id));
+			req.body.classIds = req.body.classIds
+				.filter((id) => groups.some((group) => group.id !== id));
+		}
+
 		const startDate = timesHelper.dateStringToMoment(req.body.startDate);
 		const untilDate = timesHelper.dateStringToMoment(req.body.untilDate);