Merge remote-tracking branch 'origin/N21-2136-moin-schule-logout-in-s…

…vs' into N21-2136-moin-schule-logout-in-svs
hpi-schul-cloud · Nov 11, 2024 · 26a6469 · 26a6469
2 parents 3768753 + 2bc23fa
commit 26a6469
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 4 deletions.
diff --git a/controllers/courses.js b/controllers/courses.js
@@ -80,8 +80,8 @@ const getSyncedElements = 	(
 		startDate,
 		untilDate,
 		syncedWithGroup,
+		excludeFromSync: course.excludeFromSync?.join(','),
 	};
-
 	return selectedElements;
 };
 
@@ -305,7 +305,17 @@ const editCourseHandler = (req, res, next) => {
 
 		if (syncedGroupId && group) {
 			course.name = group.name;
-			course.teacherIds = getUserIdsByRole(group.users, 'teacher');
+
+			const teacherIds = getUserIdsByRole(group.users, 'teacher');
+			const isTeacherInGroup = teacherIds.some((tid) => tid === res.locals.currentUser._id);
+			const isTeacher = res.locals.currentUser.roles.map((role) => role.name).includes('teacher');
+			if (!isTeacherInGroup && isTeacher) {
+				course.excludeFromSync = ['teachers'];
+				course.teacherIds = [res.locals.currentUser._id];
+			} else {
+				course.teacherIds = teacherIds;
+			}
+
 			course.userIds = getUserIdsByRole(group.users, 'student');
 
 			if (group.validPeriod) {
@@ -578,7 +588,7 @@ router.post('/', (req, res, next) => {
 		req.body.untilDate = untilDate.toDate();
 	}
 
-	const keys = ['teacherIds', 'substitutionIds', 'classIds', 'userIds'];
+	const keys = ['teacherIds', 'substitutionIds', 'classIds', 'userIds', 'excludeFromSync'];
 	req.body = strToPropsArray(req.body, keys);
 
 	req.body.features = [];
@@ -843,7 +853,7 @@ router.patch('/:courseId', async (req, res, next) => {
 			req.body.substitutionIds = [];
 		}
 
-		const keys = ['teacherIds', 'substitutionIds', 'classIds', 'userIds'];
+		const keys = ['teacherIds', 'substitutionIds', 'classIds', 'userIds', 'excludeFromSync'];
 		req.body = strToPropsArray(req.body, keys);
 
 		const startDate = timesHelper.dateTimeStringToMoment(req.body.startDate).utc();

diff --git a/views/courses/create-course.hbs b/views/courses/create-course.hbs
@@ -188,6 +188,10 @@
               </div>
             </div>
 
+              {{#if syncedWithGroup}}
+                      <input type="hidden" id="excludeFromSync" name="excludeFromSync" value="{{excludeFromSync}}">
+              {{/if}}
+
             </section>
 
             <section data-panel="section-2" data-testid="section-2-area" class="submit-page course-submit-page">

diff --git a/views/courses/edit-course.hbs b/views/courses/edit-course.hbs
@@ -300,6 +300,9 @@
 					</button>
 				</div>
 
+                {{#if course.syncedWithGroup}}
+                    <input type="hidden" id="excludeFromSync" name="excludeFromSync" value="{{excludeFromSync}}">
+                {{/if}}
 
 				{{#unless @root.course.isArchived}}
 					<div class="modal-footer">