BC-7281 - fix redirection to a non-existent page (#3448)

hpi-schul-cloud · May 8, 2024 · 4ee7141 · 4ee7141
1 parent 9b14893
commit 4ee7141
Showing 1 changed file with 15 additions and 0 deletions.
diff --git a/controllers/courses.js b/controllers/courses.js
@@ -631,6 +631,21 @@ router.get('/:courseId/usersJson', (req, res, next) => {
 // EDITOR
 
 router.get('/:courseId/', async (req, res, next) => {
+	const allowedActiveTabs = ['tools', 'groups'];
+	const { activeTab } = req.query;
+
+	if (!allowedActiveTabs.includes(activeTab)) {
+		res.redirect(getDefaultRedirectUrl(req.params.courseId));
+		return;
+	}
+
+	const FEATURE_CTL_TOOLS_TAB_ENABLED = Configuration.get('FEATURE_CTL_TOOLS_TAB_ENABLED');
+
+	if (FEATURE_CTL_TOOLS_TAB_ENABLED && activeTab === 'tools') {
+		res.redirect(`/rooms/${req.params.courseId}?tab=tools`);
+		return;
+	}
+
 	const promises = [
 		api(req).get(`/courses/${req.params.courseId}`),
 		api(req).get('/lessons/', {