remove useless check for unset user_is_logged_in, since that is
[squirrelmail.git] / src / redirect.php
index bfa88bef29fb2eb8058864169c7c7cc524b0b729..cc2dde99ef55e55997ca827cb81ed9ebf2b0bbd3 100644 (file)
@@ -10,7 +10,9 @@
  * @version $Id$
  * @package squirrelmail
  */
-$sInitLocation = 'redirect';
+
+/** This is the redirect page */
+define('PAGE_NAME', 'redirect');
 
 /**
  * Include the SquirrelMail initialization file.
@@ -51,69 +53,68 @@ if (!isset($login_username)) {
     exit;
 }
 
-if (!sqsession_is_registered('user_is_logged_in')) {
-    do_hook('login_before', $null);
-
-    $onetimepad = OneTimePadCreate(strlen($secretkey));
-    $key = OneTimePadEncrypt($secretkey, $onetimepad);
+do_hook('login_before', $null);
 
-    /* remove redundant spaces */
-    $login_username = trim($login_username);
-
-    /* Verify that username and password are correct. */
-    if ($force_username_lowercase) {
-        $login_username = strtolower($login_username);
-    }
+$onetimepad = OneTimePadCreate(strlen($secretkey));
+$key = OneTimePadEncrypt($secretkey, $onetimepad);
 
-    $imapConnection = sqimap_login($login_username, $key, $imapServerAddress, $imapPort, 0);
-    /* From now on we are logged it. If the login failed then sqimap_login handles it */
-
-    /* regenerate the session id to avoid session hyijacking */
-    sqsession_destroy();
-    @sqsession_is_active();
-    session_regenerate_id();
-    /**
-     * The cookie part. session_start and session_regenerate_session normally set
-     * their own cookie. SquirrelMail sets another cookie which overwites the
-     * php cookies. The sqsetcookie function sets the cookie by using the header
-     * function which gives us full control how the cookie is set. We do that
-     * to add the HttpOnly cookie attribute which blocks javascript access on
-     * IE6 SP1.
-     */
-    sqsetcookie(session_name(),session_id(),false,$base_uri);
-    sqsetcookie('key', $key, false, $base_uri);
+/* remove redundant spaces */
+$login_username = trim($login_username);
 
-    sqsession_register($onetimepad, 'onetimepad');
+/* Case-normalise username if so desired */
+if ($force_username_lowercase) {
+    $login_username = strtolower($login_username);
+}
 
-    $sqimap_capabilities = sqimap_capability($imapConnection);
+/* Verify that username and password are correct. */
+$imapConnection = sqimap_login($login_username, $key, $imapServerAddress, $imapPort, 0);
+/* From now on we are logged it. If the login failed then sqimap_login handles it */
 
-    /* Server side sorting control */
-    if (isset($sqimap_capabilities['SORT']) && $sqimap_capabilities['SORT'] == true &&
-        isset($disable_server_sort) && $disable_server_sort) {
-        unset($sqimap_capabilities['SORT']);
-    }
+/* regenerate the session id to avoid session hyijacking */
+sqsession_destroy();
+@sqsession_is_active();
+session_regenerate_id();
+/**
+* The cookie part. session_start and session_regenerate_session normally set
+* their own cookie. SquirrelMail sets another cookie which overwites the
+* php cookies. The sqsetcookie function sets the cookie by using the header
+* function which gives us full control how the cookie is set. We do that
+* to add the HttpOnly cookie attribute which blocks javascript access on
+* IE6 SP1.
+*/
+sqsetcookie(session_name(),session_id(),false,$base_uri);
+sqsetcookie('key', $key, false, $base_uri);
+
+sqsession_register($onetimepad, 'onetimepad');
+
+$sqimap_capabilities = sqimap_capability($imapConnection);
+
+/* Server side sorting control */
+if (isset($sqimap_capabilities['SORT']) && $sqimap_capabilities['SORT'] == true &&
+    isset($disable_server_sort) && $disable_server_sort) {
+    unset($sqimap_capabilities['SORT']);
+}
 
-    /* Thread sort control */
-    if (isset($sqimap_capabilities['THREAD']) && $sqimap_capabilities['THREAD'] == true &&
-        isset($disable_thread_sort) && $disable_thread_sort) {
-        unset($sqimap_capabilities['THREAD']);
-    }
+/* Thread sort control */
+if (isset($sqimap_capabilities['THREAD']) && $sqimap_capabilities['THREAD'] == true &&
+    isset($disable_thread_sort) && $disable_thread_sort) {
+    unset($sqimap_capabilities['THREAD']);
+}
 
-    sqsession_register($sqimap_capabilities, 'sqimap_capabilities');
-    $delimiter = sqimap_get_delimiter ($imapConnection);
+sqsession_register($sqimap_capabilities, 'sqimap_capabilities');
+$delimiter = sqimap_get_delimiter ($imapConnection);
 
-    if (isset($sqimap_capabilities['NAMESPACE']) && $sqimap_capabilities['NAMESPACE'] == true) {
-        $namespace = sqimap_get_namespace($imapConnection);
-        sqsession_register($namespace, 'sqimap_namespace');
-    }
+if (isset($sqimap_capabilities['NAMESPACE']) && $sqimap_capabilities['NAMESPACE'] == true) {
+    $namespace = sqimap_get_namespace($imapConnection);
+    sqsession_register($namespace, 'sqimap_namespace');
+}
 
-    sqimap_logout($imapConnection);
-    sqsession_register($delimiter, 'delimiter');
+sqimap_logout($imapConnection);
+sqsession_register($delimiter, 'delimiter');
 
-    $username = $login_username;
-    sqsession_register ($username, 'username');
-    do_hook('login_verified', $null);
-}
+$username = $login_username;
+sqsession_register ($username, 'username');
+do_hook('login_verified', $null);
 
 /* Set the login variables. */
 $user_is_logged_in = true;
@@ -142,14 +143,14 @@ $redirect_url = $location . '/webmail.php';
 
 if ( sqgetGlobalVar('session_expired_location', $session_expired_location, SQ_SESSION) ) {
     sqsession_unregister('session_expired_location');
-    if ( strpos($session_expired_location, 'compose.php') !== FALSE ) {
+    if ( $session_expired_location == 'compose' ) {
         $compose_new_win = getPref($data_dir, $username, 'compose_new_win', 0);
         if ($compose_new_win) {
-            // do not prefix $location here because $session_expired_location is set to PHP_SELF
+            // do not prefix $location here because $session_expired_location is set to the PAGE_NAME
             // of the last page
-            $redirect_url = $session_expired_location;
+            $redirect_url = $session_expired_location.'.php';
         } else {
-            $redirect_url = $location.'/webmail.php?right_frame='.urlencode($session_expired_location);
+            $redirect_url = $location.'/webmail.php?right_frame='.urlencode($session_expired_location).'php';
         }
     }
     unset($session_expired_location);
@@ -181,9 +182,9 @@ function attachment_common_parse($str) {
 
     foreach ($types as $val) {
         // Ignore the ";q=1.0" stuff
-        if (strpos($val, ';') !== false)
+        if (strpos($val, ';') !== false) {
             $val = substr($val, 0, strpos($val, ';'));
-
+        }
         if (! isset($attachment_common_types[$val])) {
             $attachment_common_types[$val] = true;
         }