From d224551cfbf5903505f41bc9e51ad9840119942f Mon Sep 17 00:00:00 2001 From: thomascube <thomas@roundcube.net> Date: Thu, 31 Jul 2008 11:23:06 -0400 Subject: [PATCH] Fix read_when_deleted behavior + javascript codestyle --- program/include/rcmail.php | 41 +++++++++++++++++++++++++++++++---------- 1 files changed, 31 insertions(+), 10 deletions(-) diff --git a/program/include/rcmail.php b/program/include/rcmail.php index 057cabd..f3b3970 100644 --- a/program/include/rcmail.php +++ b/program/include/rcmail.php @@ -233,13 +233,17 @@ * Init output object for GUI and add common scripts. * This will instantiate a rcmail_template object and set * environment vars according to the current session and configuration + * + * @param boolean True if this request is loaded in a (i)frame + * @return object rcube_template Reference to HTML output object */ public function load_gui($framed = false) { // init output page - $this->output = new rcube_template($this->task, $framed); + if (!($this->output instanceof rcube_template)) + $this->output = new rcube_template($this->task, $framed); - foreach (array('flag_for_deletion') as $js_config_var) { + foreach (array('flag_for_deletion','read_when_deleted') as $js_config_var) { $this->output->set_env($js_config_var, $this->config->get($js_config_var)); } @@ -262,10 +266,13 @@ /** * Create an output object for JSON responses + * + * @return object rcube_json_output Reference to JSON output object */ public function init_json() { - $this->output = new rcube_json_output($this->task); + if (!($this->output instanceof rcube_json_output)) + $this->output = new rcube_json_output($this->task); return $this->output; } @@ -308,7 +315,7 @@ { $conn = false; - if ($_SESSION['imap_host']) { + if ($_SESSION['imap_host'] && !$this->imap->conn) { if (!($conn = $this->imap->connect($_SESSION['imap_host'], $_SESSION['username'], $this->decrypt_passwd($_SESSION['password']), $_SESSION['imap_port'], $_SESSION['imap_ssl'], rcmail::get_instance()->config->get('imap_auth_type', 'check')))) { if ($this->output) $this->output->show_message($this->imap->error_code == -1 ? 'imaperror' : 'sessionerror', 'error'); @@ -317,7 +324,7 @@ $this->set_imap_prop(); } - return $conn; + return $conn; } @@ -474,12 +481,20 @@ public function autoselect_host() { $default_host = $this->config->get('default_host'); - $host = empty($default_host) ? get_input_value('_host', RCUBE_INPUT_POST) : $default_host; + $host = null; - if (is_array($host)) { + if (is_array($default_host)) { + $post_host = get_input_value('_host', RCUBE_INPUT_POST); + + // direct match in default_host array + if ($default_host[$post_host] || in_array($post_host, array_values($default_host))) { + $host = $post_host; + } + + // try to select host by mail domain list($user, $domain) = explode('@', get_input_value('_user', RCUBE_INPUT_POST)); if (!empty($domain)) { - foreach ($host as $imap_host => $mail_domains) { + foreach ($default_host as $imap_host => $mail_domains) { if (is_array($mail_domains) && in_array($domain, $mail_domains)) { $host = $imap_host; break; @@ -488,9 +503,15 @@ } // take the first entry if $host is still an array - if (is_array($host)) - $host = array_shift($host); + if (empty($host)) { + $host = array_shift($default_host); + } } + else if (empty($default_host)) { + $host = get_input_value('_host', RCUBE_INPUT_POST); + } + else + $host = $default_host; return $host; } -- Gitblit v1.9.1