- Fix race conditions when changing mailbox (set some env variables only wh...

alecpl

2008-10-02 52851464e0267795ffd688e3c769d9161011dba8

 program/include/rcmail.php

@@ -170,7 +170,7 @@
    $_SESSION['language'] = $this->user->language = $this->language_prop($this->config->get('language', $_SESSION['language']));

    // set localization
    setlocale(LC_ALL, $_SESSION['language'] . '.utf8');
    setlocale(LC_ALL, $_SESSION['language'] . '.utf8', 'en_US.utf8');
  }
  
  
@@ -223,7 +223,7 @@
  /**
   * Get the current database connection
   *
   * @return object rcube_db  Database connection object
   * @return object rcube_mdb2  Database connection object
   */
  public function get_dbh()
  {
@@ -460,7 +460,7 @@
      raise_error(array(
        'code' => 600,
        'type' => 'php',
        'file' => "config/main.inc.php",
        'file' => RCMAIL_CONFIG_DIR."/main.inc.php",
        'message' => "Acces denied for new user $username. 'auto_create_user' is disabled"
        ), true, false);
    }
@@ -728,7 +728,7 @@
      if (!$valid || ($_SERVER['REQUEST_METHOD']!='POST' && $now - $_SESSION['auth_time'] > 300)) {
        $_SESSION['last_auth'] = $_SESSION['auth_time'];
        $_SESSION['auth_time'] = $now;
        setcookie('sessauth', $this->get_auth_hash(session_id(), $now));
        rcmail::setcookie('sessauth', $this->get_auth_hash(session_id(), $now), 0);
      }
    }
    else {
@@ -751,7 +751,7 @@
  public function kill_session()
  {
    $_SESSION = array('language' => $this->user->language, 'auth_time' => time(), 'temp' => true);
    setcookie('sessauth', '-del-', time() - 60);
    rcmail::setcookie('sessauth', '-del-', time() - 60);
    $this->user->reset();
  }

@@ -891,19 +891,39 @@
  {
    if (!is_array($p))
      $p = array('_action' => @func_get_arg(0));
    
    if ($p['task'] && in_array($p['task'], rcmail::$main_tasks))
      $url = './?_task='.$p['task'];
    else
      $url = $this->comm_path;
    

    if (!$p['task'] || !in_array($p['task'], rcmail::$main_tasks))
      $p['task'] = $this->task;

    $p['_task'] = $p['task'];
    unset($p['task']);
    foreach ($p as $par => $val)
      if (isset($val))
        $url .= '&'.urlencode($par).'='.urlencode($val);
    

    $url = './';
    $delm = '?';
    foreach (array_reverse($p) as $par => $val)
    {
      if (!empty($val)) {
        $url .= $delm.urlencode($par).'='.urlencode($val);
        $delm = '&';
      }
    }
    return $url;
  }


  /**
   * Helper method to set a cookie with the current path and host settings
   *
   * @param string Cookie name
   * @param string Cookie value
   * @param string Expiration time
   */
  public static function setcookie($name, $value, $exp = 0)
  {
    $cookie = session_get_cookie_params();
    setcookie($name, $value, $exp, $cookie['path'], $cookie['domain'],
      ($_SERVER['HTTPS'] && ($_SERVER['HTTPS'] != 'off')));
  }
}