Fixing redirection see BT#8207 and #6768

chamilo · Jun 4, 2014 · 82df5fb · 82df5fb
1 parent 9b1ddc0
commit 82df5fb
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/main/inc/local.inc.php b/main/inc/local.inc.php
@@ -1239,22 +1239,25 @@
     Database::query($sql);
 }
 
-
 // direct login to course
-if ((isset($cas_login) && $cas_login && exist_firstpage_parameter())
-    || ($logging_in && exist_firstpage_parameter())) {
+if ((isset($cas_login) && $cas_login && exist_firstpage_parameter()) ||
+    ($logging_in && exist_firstpage_parameter())
+) {
     $redirectCourseDir = api_get_firstpage_parameter();
     api_delete_firstpage_parameter();    // delete the cookie
-    if (CourseManager::get_course_id_from_path($redirectCourseDir)) {
-        $_SESSION['noredirection'] = false;
-        $_SESSION['request_uri'] = api_get_path(WEB_COURSE_PATH).$redirectCourseDir;
+
+    if (!isset($_SESSION['request_uri'])) {
+        if (CourseManager::get_course_id_from_path($redirectCourseDir)) {
+            $_SESSION['noredirection'] = false;
+            $_SESSION['request_uri'] = api_get_path(WEB_COURSE_PATH) . $redirectCourseDir;
+        }
     }
 } elseif (api_user_is_login() && exist_firstpage_parameter()) {
     $redirectCourseDir = api_get_firstpage_parameter();
-    api_delete_firstpage_parameter();    // delete the cookie
+    api_delete_firstpage_parameter(); // delete the cookie
     if (CourseManager::get_course_id_from_path($redirectCourseDir)) {
         $_SESSION['noredirection'] = false;
-        $_SESSION['request_uri'] = api_get_path(WEB_COURSE_PATH).$redirectCourseDir;
+        $_SESSION['request_uri'] = api_get_path(WEB_COURSE_PATH) . $redirectCourseDir;
     }
 }