X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=src%2Fread_body.php;h=2e45408826e51ca3e0d1e05d3565da258c4c655a;hb=7fc4b14fdb798a7b1c1d003163afa5865cce01f4;hp=c3b739031767b93ef9a5c61a8182faed411cbeb9;hpb=3c621ba1116356a3a8e2616ef6d267f5edb3ea06;p=squirrelmail.git diff --git a/src/read_body.php b/src/read_body.php index c3b73903..2e454088 100644 --- a/src/read_body.php +++ b/src/read_body.php @@ -1,13 +1,13 @@ $value) { - if ($passed_id == $value) { - if ($key == $count) { - break; - } - $result = $uidset[$key + 1]; - break; - } + $result = sqm_array_get_value_by_offset($uidset,$passed_id,1); + if ($result === false) { + return -1; + } else { + return $result; } - return $result; } /** @@ -70,17 +65,12 @@ function findPreviousMessage($uidset, $passed_id) { if (!is_array($uidset)) { return -1; } - $result = -1; - foreach($uidset as $key=>$value) { - if ($passed_id == $value) { - if ($key != 0) { - $result = $uidset[$key - 1]; - } - break; - } + $result = sqm_array_get_value_by_offset($uidset,$passed_id,-1); + if ($result === false) { + return -1; + } else { + return $result; } - - return $result; } /** @@ -90,11 +80,19 @@ function findPreviousMessage($uidset, $passed_id) { * @param int $passed_id */ function printer_friendly_link($mailbox, $passed_id, $passed_ent_id) { - global $javascript_on, $color; + global $javascript_on, $show_html_default; + /* hackydiehack */ + if( !sqgetGlobalVar('view_unsafe_images', $view_unsafe_images, SQ_GET) ) { + $view_unsafe_images = false; + } else { + $view_unsafe_images = true; + } $params = '?passed_ent_id=' . urlencode($passed_ent_id) . '&mailbox=' . urlencode($mailbox) . - '&passed_id=' . urlencode($passed_id); + '&passed_id=' . urlencode($passed_id) . + '&view_unsafe_images='. (bool) $view_unsafe_images . + '&show_html_default=' . $show_html_default; $print_text = _("View Printable Version"); @@ -124,15 +122,13 @@ function ServerMDNSupport($aFlags) { } function SendMDN ( $mailbox, $passed_id, $sender, $message, $imapConnection) { - global $username, $attachment_dir, - $version, $attachments, $squirrelmail_language, $default_charset, - $languages, $useSendmail, $domain, $sent_folder, - $popuser, $data_dir, $username; + global $username, $attachment_dir, $popuser, $username, $color, + $version, $squirrelmail_language, $default_charset, + $languages, $useSendmail, $domain, $sent_folder; sqgetGlobalVar('SERVER_NAME', $SERVER_NAME, SQ_SERVER); $header = $message->rfc822_header; - $hashed_attachment_dir = getHashedDir($username, $attachment_dir); $rfc822_header = new Rfc822Header(); $content_type = new ContentType('multipart/report'); @@ -184,9 +180,9 @@ function SendMDN ( $mailbox, $passed_id, $sender, $message, $imapConnection) { $now = getLongDateString( time() ); set_my_charset(); $body = _("Your message") . "\r\n\r\n" . - "\t" . _("To:") . ' ' . decodeHeader($to,false,false) . "\r\n" . - "\t" . _("Subject:") . ' ' . decodeHeader($header->subject,false,false) . "\r\n" . - "\t" . _("Sent:") . ' ' . $senton . "\r\n" . + "\t" . _("To") . ': ' . decodeHeader($to,false,false) . "\r\n" . + "\t" . _("Subject") . ': ' . decodeHeader($header->subject,false,false) . "\r\n" . + "\t" . _("Sent") . ': ' . $senton . "\r\n" . "\r\n" . sprintf( _("Was displayed on %s"), $now ); @@ -202,6 +198,8 @@ function SendMDN ( $mailbox, $passed_id, $sender, $message, $imapConnection) { $special_encoding = '7bit'; } } + } elseif (sq_is8bit($body)) { + $special_encoding = '8bit'; } $part1 = new Message(); $part1->setBody($body); @@ -253,7 +251,7 @@ function SendMDN ( $mailbox, $passed_id, $sender, $message, $imapConnection) { } else { require_once(SM_PATH . 'class/deliver/Deliver_SMTP.class.php'); $deliver = new Deliver_SMTP(); - global $smtpServerAddress, $smtpPort, $smtp_auth_mech, $pop_before_smtp; + global $smtpServerAddress, $smtpPort, $pop_before_smtp; $authPop = (isset($pop_before_smtp) && $pop_before_smtp) ? true : false; get_smtp_user($user, $pass); $stream = $deliver->initStream($composeMessage,$domain,0, @@ -265,9 +263,12 @@ function SendMDN ( $mailbox, $passed_id, $sender, $message, $imapConnection) { $success = $deliver->finalizeStream($stream); } if (!$success) { - $msg = $deliver->dlv_msg . '
' . - _("Server replied: ") . $deliver->dlv_ret_nr . ' '. - $deliver->dlv_server_msg; + $msg = $deliver->dlv_msg; + if (! empty($deliver->dlv_server_msg)) { + $msg.= '
' . + _("Server replied:") . ' ' . $deliver->dlv_ret_nr . ' ' . + nl2br(htmlspecialchars($deliver->dlv_server_msg)); + } require_once(SM_PATH . 'functions/display_messages.php'); plain_error_message($msg, $color); } else { @@ -291,27 +292,6 @@ function ToggleMDNflag ($set ,$imapConnection, $mailbox, $passed_id) { $readmessage, TRUE); } -function ClearAttachments() { - global $username, $attachments, $attachment_dir; - - $hashed_attachment_dir = getHashedDir($username, $attachment_dir); - - $rem_attachments = array(); - if (isset($attachments)) { - foreach ($attachments as $info) { - if ($info['session'] == -1) { - $attached_file = "$hashed_attachment_dir/$info[localfilename]"; - if (file_exists($attached_file)) { - unlink($attached_file); - } - } else { - $rem_attachments[] = $info; - } - } - } - $attachments = $rem_attachments; -} - function formatRecipientString($recipients, $item ) { global $show_more_cc, $show_more, $show_more_bcc, $PHP_SELF; @@ -367,11 +347,11 @@ function formatRecipientString($recipients, $item ) { function formatEnvheader($aMailbox, $passed_id, $passed_ent_id, $message, $color, $FirstTimeSee) { - global $msn_user_support, $default_use_mdn, $default_use_priority, + global $default_use_mdn, $default_use_priority, $show_xmailer_default, $mdn_user_support, $PHP_SELF, $javascript_on, - $squirrelmail_language; + $squirrelmail_language; - $mailbox = $aMailbox['NAME'] ; + $mailbox = $aMailbox['NAME']; $header = $message->rfc822_header; $env = array(); @@ -460,8 +440,8 @@ function formatMenubar($aMailbox, $passed_id, $passed_ent_id, $message, $removed global $base_uri, $draft_folder, $where, $what, $color, $sort, $startMessage, $PHP_SELF, $save_as_draft, $enable_forward_as_attachment, $imapConnection, $lastTargetMailbox, - $data_dir, $username, $delete_prev_next_display, - $compose_new_win, $javascript_on; + $username, $delete_prev_next_display, + $compose_new_win, $javascript_on, $compose_width, $compose_height; //FIXME cleanup argument list, use $aMailbox where possible $mailbox = $aMailbox['NAME']; @@ -493,7 +473,7 @@ function formatMenubar($aMailbox, $passed_id, $passed_ent_id, $message, $removed } $prev_link = _("Previous"); - if($entities[$passed_ent_id] > 1) { + if(isset($entities[$passed_ent_id]) && $entities[$passed_ent_id] > 1) { $prev_ent_id = $entity_count[$entities[$passed_ent_id] - 1]; $prev_link = ''; + $menu_row = ''; $comp_uri = $base_uri.'src/compose.php' . '?passed_id=' . $passed_id . '&mailbox=' . $urlMailbox . @@ -607,17 +581,26 @@ function formatMenubar($aMailbox, $passed_id, $passed_ent_id, $message, $removed $target = ''; $on_click=''; $method='method="post" '; + $onsubmit=''; if ($compose_new_win == '1') { + if (!preg_match("/^[0-9]{3,4}$/", $compose_width)) { + $compose_width = '640'; + } + if (!preg_match("/^[0-9]{3,4}$/", $compose_height)) { + $compose_height = '550'; + } if ( $javascript_on ) { - $on_click=' onclick="comp_in_new_form(\''.$comp_uri.'\', this, this.form)"'; + $on_click=' onclick="comp_in_new_form(\''.$comp_uri.'\', this, this.form,'. $compose_width .',' . $compose_height .')"'; $comp_uri = 'javascript:void(0)'; $method='method="get" '; + $onsubmit = 'onsubmit="return false" '; } else { $target = 'target="_blank"'; } } - $menu_row .= "\n".'
'."\n"; + $menu_row .= "\n".''."\n"; // If Draft folder - create Resume link if (($mailbox == $draft_folder) && ($save_as_draft)) { @@ -630,13 +613,13 @@ function formatMenubar($aMailbox, $passed_id, $passed_ent_id, $message, $removed } // Show Alt URI for Draft/Sent if (isset($comp_alt_string)) - $menu_row .= getButton('SUBMIT', $new_button, $comp_alt_string, $on_click) . "\n"; + $menu_row .= getButton('submit', $new_button, $comp_alt_string, $on_click) . "\n"; - $menu_row .= getButton('SUBMIT', 'smaction_reply', _("Reply"), $on_click) . "\n"; - $menu_row .= getButton('SUBMIT', 'smaction_reply_all', _("Reply All"), $on_click) ."\n"; - $menu_row .= getButton('SUBMIT', 'smaction_forward', _("Forward"), $on_click); + $menu_row .= getButton('submit', 'smaction_reply', _("Reply"), $on_click) . "\n"; + $menu_row .= getButton('submit', 'smaction_reply_all', _("Reply All"), $on_click) ."\n"; + $menu_row .= getButton('submit', 'smaction_forward', _("Forward"), $on_click); if ($enable_forward_as_attachment) - $menu_row .= '' . _("As Attachment") .'  '."\n"; + $menu_row .= '' . _("As Attachment") .'  '."\n"; $menu_row .= '
 '; @@ -648,10 +631,11 @@ function formatMenubar($aMailbox, $passed_id, $passed_ent_id, $message, $removed if (!(isset($passed_ent_id) && $passed_ent_id)) { $menu_row .= addHidden('mailbox', $aMailbox['NAME']); $menu_row .= addHidden('msg[0]', $passed_id); - $menu_row .= getButton('SUBMIT', 'delete', _("Delete")); - $menu_row .= '' . _("Bypass Trash"); + $menu_row .= addHidden('startMessage', $startMessage); + $menu_row .= getButton('submit', 'delete', _("Delete")); + $menu_row .= '' . _("Bypass Trash"); } else { - $menu_row .= getButton('SUBMIT', 'delete', _("Delete"), '', FALSE) . "\n"; // delete button is disabled + $menu_row .= getButton('submit', 'delete', _("Delete"), '', FALSE) . "\n"; // delete button is disabled } $menu_row .= ''; @@ -676,7 +660,7 @@ function formatMenubar($aMailbox, $passed_id, $passed_ent_id, $message, $removed } $menu_row .= ' '; - $menu_row .= getButton('SUBMIT', 'moveButton',_("Move")) . "\n" . ''; + $menu_row .= getButton('submit', 'moveButton',_("Move")) . "\n" . ''; } $menu_row .= ''; @@ -731,6 +715,27 @@ function formatToolbar($mailbox, $passed_id, $passed_ent_id, $message, $color) { } +/** + * Creates button + * + * @deprecated see form functions available in 1.5.1 and 1.4.3. + * @param string $type + * @param string $name + * @param string $value + * @param string $js + * @param bool $enabled + */ +function getButton($type, $name, $value, $js = '', $enabled = TRUE) { + $disabled = ( $enabled ? '' : 'disabled ' ); + $js = ( $js ? $js.' ' : '' ); + return ''; +} + + /***************************/ /* Main of read_body.php */ /***************************/ @@ -752,6 +757,10 @@ sqgetGlobalVar('sendreceipt', $sendreceipt, SQ_GET); if (!sqgetGlobalVar('where', $where, SQ_GET) ) { $where = 'right_main.php'; } +/* + * Used as entry key to the list of uid's cached in the mailbox cache + * we use the cached uid's to get the next and prev message. + */ if (!sqgetGlobalVar('what', $what, SQ_GET) ){ $what = 0; } @@ -768,6 +777,12 @@ if ( sqgetGlobalVar('view_hdr', $temp, SQ_GET) ) { $view_hdr = (int) $temp; } +if ( sqgetGlobalVar('account', $temp, SQ_GET) ) { + $iAccount = (int) $temp; +} else { + $iAccount = 0; +} + /** GET/POST VARS */ sqgetGlobalVar('passed_ent_id', $passed_ent_id); sqgetGlobalVar('mailbox', $mailbox); @@ -792,7 +807,65 @@ sqgetGlobalVar('mailbox_cache',$mailbox_cache,SQ_SESSION); global $sqimap_capabilities, $lastTargetMailbox; $imapConnection = sqimap_login($username, $key, $imapServerAddress, $imapPort, 0); -$aMailbox = sqm_api_mailbox_select($imapConnection, $mailbox,array('setindex' => $what),array()); +$aMailbox = sqm_api_mailbox_select($imapConnection, $iAccount, $mailbox,array('setindex' => $what, 'offset' => $startMessage),array()); + + +/** + Start code to set the columns to fetch in case of hitting the next/prev link + The reason for this is the fact that the cache can be invalidated which means that the headers + to fetch aren't there anymore. Before they got calculated when the messagelist was shown. + + Todo, better central handling of setting the mailbox options so we do not need to do the stuff below +*/ + +/** + * Replace From => To in case it concerns a draft or sent folder + */ +if (($mailbox == $sent_folder || $mailbox == $draft_folder) && + !in_array(SQM_COL_TO,$index_order)) { + $aNewOrder = array(); // nice var name ;) + foreach($index_order as $iCol) { + if ($iCol == SQM_COL_FROM) { + $iCol = SQM_COL_TO; + } + $aNewOrder[] = $iCol; + } + $aColumns = $aNewOrder; +} else { + $aColumns = $index_order; +} + +$aProps = array( + 'columns' => $aColumns, // columns bound settings + 'config' => array( + 'highlight_list' => $message_highlight_list, // row highlighting rules + 'trash_folder' => $trash_folder, + 'sent_folder' => $sent_folder, + 'draft_folder' => $draft_folder)); + +calcFetchColumns($aMailbox,$aProps); + +/** + End code to set the columns to fetch in case of hitting the next/prev link +*/ + + + +/** + * Check if cache is still valid, $what contains the key + * which gives us acces to the array with uid's. At this moment + * 0 is used for a normal message list and search uses 1 as key. This can be + * changed / extended in the future. + * If on a select of a mailbox we detect that the cache should be invalidated due to + * the delete of messages or due to new messages we empty the list with uid's and + * that's what we detect below. + */ +if (!is_array($aMailbox['UIDSET'][$what])) { + fetchMessageHeaders($imapConnection, $aMailbox); +} + +$iSetIndex = $aMailbox['SETINDEX']; +$aMailbox['CURRENT_MSG'][$iSetIndex] = $passed_id; /** * Update the seen state @@ -808,8 +881,6 @@ if (isset($aMailbox['MSG_HEADERS'][$passed_id]['FLAGS'])) { */ if ( sqgetGlobalVar('delete_id', $delete_id, SQ_GET) ) { handleMessageListForm($imapConnection,$aMailbox,$sButton='setDeleted', array($delete_id)); -// sqimap_messages_delete($imapConnection, $delete_id, $delete_id, $mailbox); -// sqimap_mailbox_expunge_dmn($imapConnection,$aMailbox,$delete_id); } /** @@ -836,11 +907,18 @@ if (isset($passed_ent_id) && $passed_ent_id) { $rfc822_header = new Rfc822Header(); $rfc822_header->parseHeader($read); $message->rfc822_header = $rfc822_header; +} else if ($message->type0 == 'message' && $message->type1 == 'rfc822' && isset($message->entities[0])) { + $read = sqimap_run_command ($imapConnection, "FETCH $passed_id BODY[1.HEADER]", true, $response, $msg, TRUE); + $rfc822_header = new Rfc822Header(); + $rfc822_header->parseHeader($read); + $message->rfc822_header = $rfc822_header; } else { $passed_ent_id = 0; } $header = $message->header; +$header = $message->header; + /****************************************/ /* Block for handling incoming url vars */ @@ -849,7 +927,7 @@ $header = $message->header; if (isset($sendreceipt)) { if ( !$message->is_mdnsent ) { $final_recipient = ''; - if ((isset($identity)) && ($identity != 0)) //Main identity + if ((isset($identity)) && ($identity != 0)) //Main identity $final_recipient = trim(getPref($data_dir, $username, 'email_address' . $identity, '' )); if ($final_recipient == '' ) $final_recipient = trim(getPref($data_dir, $username, 'email_address', '' )); @@ -859,7 +937,6 @@ if (isset($sendreceipt)) { $message->is_mdnsent = true; $aMailbox['MSG_HEADERS'][$passed_id]['MESSAGE_OBJECT'] = $message; } - ClearAttachments(); } } /***********************************************/ @@ -879,11 +956,11 @@ $cnt = count($ent_ar); for ($i = 0; $i < $cnt; $i++) { $messagebody .= formatBody($imapConnection, $message, $color, $wrap_at, $ent_ar[$i], $passed_id, $mailbox); if ($i != $cnt-1) { - $messagebody .= '
'; + $messagebody .= '
'; } } -displayPageHeader($color, $mailbox); +displayPageHeader($color, $mailbox,'',''); formatMenuBar($aMailbox, $passed_id, $passed_ent_id, $message,false); formatEnvheader($aMailbox, $passed_id, $passed_ent_id, $message, $color, $FirstTimeSee); echo ''; @@ -893,7 +970,7 @@ echo '
'; echo ' '; echo '
'; // echo ' '; -echo html_tag( 'table' ,'' , 'left', '', 'cellpadding="1" cellspacing="5" border="0"' ); +echo html_tag( 'table' ,'' , 'left', '', 'width="100%" cellpadding="1" cellspacing="5" border="0"' ); echo ' ' . html_tag( 'td', '
'. $messagebody."\n", 'left') . ''; echo '
'; @@ -949,11 +1026,11 @@ formatMenuBar($aMailbox, $passed_id, $passed_ent_id, $message, false, FALSE); do_hook('read_body_bottom'); sqimap_logout($imapConnection); -/* sessions are written at the end of the script. it's better to register - them at the end so we avoid double session_register calls */ -/* add the mailbox to the cache */ -$mailbox_cache[$aMailbox['NAME']] = $aMailbox; + +/** + * Write mailbox with updated seen flag information back to cache. + */ +$mailbox_cache[$iAccount.'_'.$aMailbox['NAME']] = $aMailbox; sqsession_register($mailbox_cache,'mailbox_cache'); +$oTemplate->display('footer.tpl'); ?> - -