refactor: Move router rule for exercises status managment inside the …

…switch case
CPNV-ES · Oct 8, 2024 · 49ef6af · 49ef6af
1 parent 2168b14
commit 49ef6af
Showing 1 changed file with 7 additions and 8 deletions.
diff --git a/public/index.php b/public/index.php
@@ -30,14 +30,6 @@
 
 function getRedirection($redirect_uri)
 {
-	if (preg_match('/^\/exercises\/([0-9]+)\?exercise%5Bstatus%5D=(answering|closed)$/A', $_SERVER['REQUEST_URI'], $output_array)) {
-		if ($output_array[2] == 'answering') {
-			setExerciseAsAnswering($output_array[1]);
-		} elseif ($output_array[2] == 'closed') {
-			setExerciseAsClosed($output_array[1]);
-		}
-		return;
-	}
 	switch ($redirect_uri) {
 		case '/':
 			home();
@@ -51,6 +43,13 @@ function getRedirection($redirect_uri)
 		case '/exercises/new':
 			createAnExercises();
 			break;
+		case (preg_match('/^\/exercises\/([0-9]+)\?exercise%5Bstatus%5D=(answering|closed)$/A', $_SERVER['REQUEST_URI'], $output_array) ? true : false):
+			if ($output_array[2] == 'answering') {
+				setExerciseAsAnswering($output_array[1]);
+			} elseif ($output_array[2] == 'closed') {
+				setExerciseAsClosed($output_array[1]);
+			}
+			break;
 		default:
 			lost();
 	}