From 7dfb1fba5001299300736e6b5d95d9400575e3e7 Mon Sep 17 00:00:00 2001 From: thomascube <thomas@roundcube.net> Date: Thu, 18 Sep 2008 14:59:02 -0400 Subject: [PATCH] Set the right number of arguments for setcookie() --- program/steps/mail/get.inc | 117 +++++++++++++++++++++++++--------------------------------- 1 files changed, 50 insertions(+), 67 deletions(-) diff --git a/program/steps/mail/get.inc b/program/steps/mail/get.inc index f05bbbd..fc3ee83 100644 --- a/program/steps/mail/get.inc +++ b/program/steps/mail/get.inc @@ -5,7 +5,7 @@ | program/steps/mail/get.inc | | | | This file is part of the RoundCube Webmail client | - | Copyright (C) 2005, RoundCube Dev. - Switzerland | + | Copyright (C) 2005-2008, RoundCube Dev. - Switzerland | | Licensed under the GNU GPL | | | | PURPOSE: | @@ -23,107 +23,90 @@ // show loading page -if ($_GET['_preload']) - { +if (!empty($_GET['_preload'])) { $url = str_replace('&_preload=1', '', $_SERVER['REQUEST_URI']); $message = rcube_label('loadingdata'); print "<html>\n<head>\n" . - '<meta http-equiv="refresh" content="0; url='.$url.'">' . + '<meta http-equiv="refresh" content="0; url='.Q($url).'">' . "\n</head>\n<body>" . $message . "\n</body>\n</html>"; exit; - } +} // similar code as in program/steps/mail/show.inc -if ($_GET['_uid']) - { - $MESSAGE = array('UID' => get_input_value('_uid', RCUBE_INPUT_GET)); - $MESSAGE['structure'] = $IMAP->get_structure($MESSAGE['UID']); - $MESSAGE['parts'] = $IMAP->get_mime_numbers($MESSAGE['structure']); - } +if (!empty($_GET['_uid'])) { + $RCMAIL->config->set('prefer_html', true); + $MESSAGE = new rcube_message(get_input_value('_uid', RCUBE_INPUT_GET)); +} // show part page -if ($_GET['_frame']) - { - parse_template('messagepart'); +if (!empty($_GET['_frame'])) { + $OUTPUT->send('messagepart'); exit; - } +} -else if ($pid = get_input_value('_part', RCUBE_INPUT_GET)) - { - if ($part = $MESSAGE['parts'][$pid]); - { +else if ($pid = get_input_value('_part', RCUBE_INPUT_GET)) { + if ($part = $MESSAGE->mime_parts[$pid]) { $ctype_primary = strtolower($part->ctype_primary); $ctype_secondary = strtolower($part->ctype_secondary); - $mimetype = sprintf('%s/%s', $ctype_primary, $ctype_secondary); - $filename = $part->d_parameters['filename'] ? $part->d_parameters['filename'] : $part->ctype_parameters['name']; + + $browser = new rcube_browser; - // send correct headers for content type and length - if ($_GET['_download']) - { - // send download headers + send_nocacheing_headers(); + + // send download headers + if ($_GET['_download']) { header("Content-Type: application/octet-stream"); - header(sprintf('Content-Disposition: attachment; filename="%s"', - $filename ? rcube_imap::decode_mime_string($filename) : "roundcube.$ctype_secondary")); - } - else - { + if ($browser->ie) + header("Content-Type: application/force-download"); + } + else if ($ctype_primary == 'text') { + header("Content-Type: text/$ctype_secondary; charset=" . ($part->charset ? $part->charset : RCMAIL_CHARSET)); + } + else { header("Content-Type: $mimetype"); - header(sprintf('Content-Disposition: inline; filename="%s"', rcube_imap::decode_mime_string($filename))); - } - - // We need to set the following headers to make downloads work using IE in HTTPS mode. - if (isset($_SERVER['HTTPS'])) - { - header('Pragma: '); - header('Cache-Control: '); - } + header("Content-Transfer-Encoding: binary"); + } // deliver part content - if ($ctype_primary=='text' && $ctype_secondary=='html') - { + if ($ctype_primary == 'text' && $ctype_secondary == 'html') { // get part body if not available if (!$part->body) - $part->body = $IMAP->get_message_part($MESSAGE['UID'], $part->mime_id, $part); - - list($MESSAGE['parts']) = rcmail_parse_message($part, - array('safe' => (bool)$_GET['_safe'], - 'prefer_html' => TRUE, - 'get_url' => $GET_URL.'&_part=%s')); + $part->body = $MESSAGE->get_part_content($part->mime_id); $OUTPUT = new rcube_html_page(); - $OUTPUT->write(rcmail_print_body($MESSAGE['parts'][0], (bool)$_GET['_safe'])); + $OUTPUT->write(rcmail_print_body($part, array('safe' => $MESSAGE->is_safe, 'inline_html' => false))); + } + else { + // don't kill the connection if download takes more than 30 sec. + if (!ini_get('safe_mode')) { + set_time_limit(0); } - else - { + + $filename = $part->filename ? $part->filename : ($MESSAGE->subject ? $MESSAGE->subject : 'roundcube') . '.'.$ctype_secondary; + $filename = abbreviate_string($part->filename, 55); + $filename = $browser->ie ? rawurlencode($filename) : addslashes($filename); + $disposition = !empty($_GET['_download']) ? 'attachment' : 'inline'; + + header("Content-Disposition: $disposition; filename=\"$filename\""); + // turn off output buffering and print part content - //@ob_end_clean(); - $IMAP->get_message_part($MESSAGE['UID'], $part->mime_id, $part->encoding, true); - } + $IMAP->get_message_part($MESSAGE->uid, $part->mime_id, $part, true); + } exit; - } } +} // print message -else - { - $ctype_primary = strtolower($MESSAGE['structure']->ctype_primary); - $ctype_secondary = strtolower($MESSAGE['structure']->ctype_secondary); - $mimetype = sprintf('%s/%s', $ctype_primary, $ctype_secondary); - +else { // send correct headers for content type header("Content-Type: text/html"); - - $cont = ''; - list($MESSAGE['parts']) = rcmail_parse_message($MESSAGE['structure'], - array('safe' => (bool)$_GET['_safe'], - 'get_url' => $GET_URL.'&_part=%s')); $cont = "<html>\n<head><title></title>\n</head>\n<body>"; $cont .= rcmail_message_body(array()); @@ -133,11 +116,11 @@ $OUTPUT->write($cont); exit; - } +} // if we arrive here, the requested part was not found header('HTTP/1.1 404 Not Found'); exit; -?> \ No newline at end of file +?> -- Gitblit v1.9.1