Bug #5661 - mymoodle redirect problem

moodle · Jun 1, 2006 · ec7418d · ec7418d
1 parent 6d895ae
commit ec7418d
Showing 1 changed file with 11 additions and 4 deletions.
diff --git a/login/index.php b/login/index.php
@@ -163,6 +163,7 @@
             }
 
 
+          /// Prepare redirection
             if (user_not_fully_set_up($USER)) {
                 $urltogo = $CFG->wwwroot.'/user/edit.php?id='.$USER->id.'&amp;course='.SITEID;
                 // We don't delete $SESSION->wantsurl yet, so we get there later
@@ -172,13 +173,19 @@
                 unset($SESSION->wantsurl);
 
             } else {
-                $urltogo = $CFG->wwwroot.'/my';      /// Go to the standard home page
-                if (isadmin() || empty($CFG->mymoodleredirect) || isguest()) {
-                    $urltogo = $CFG->wwwroot;       /// not needed by admins or guests or when it's turned off
+                // no wantsurl stored or external - go to homepage
+                $urltogo = $CFG->wwwroot;
+                unset($SESSION->wantsurl);
+            }
+
+          /// Go to my-moodle page instead of homepage if mymoodleredirect enabled
+            if (!isadmin() and !empty($CFG->mymoodleredirect) and !isguest()) {
+                if ($urltogo == $CFG->wwwroot or $urltogo == $CFG->wwwroot.'/' or $urltogo == $CFG->wwwroot.'/index.php') {
+                    $urltogo = $CFG->wwwroot.'/my/';
                 }
-                unset($SESSION->wantsurl);         /// Just in case
             }
 
+
             // check if user password has expired
             // Currently supported only for ldap-authentication module
             if (isset($CFG->ldap_expiration) && $CFG->ldap_expiration == 1 ) {