X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=src%2Fread_body.php;h=09c41347ee2a2b9e322615fe859f63a7295902b3;hb=f7cd8eb4e788b45f94c494836eae2ac63522a672;hp=e07ab9db62189953a5d8505d2de76ce365a8c7e9;hpb=e1728a7ada03ba597fc2f8a800773cb427a711ce;p=squirrelmail.git diff --git a/src/read_body.php b/src/read_body.php index e07ab9db..09c41347 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'); @@ -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, @@ -266,7 +264,7 @@ function SendMDN ( $mailbox, $passed_id, $sender, $message, $imapConnection) { } if (!$success) { $msg = $deliver->dlv_msg . '
' . - _("Server replied: ") . $deliver->dlv_ret_nr . ' '. + _("Server replied:") . ' ' . $deliver->dlv_ret_nr . ' ' . $deliver->dlv_server_msg; require_once(SM_PATH . 'functions/display_messages.php'); plain_error_message($msg, $color); @@ -291,27 +289,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 +344,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 +437,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 +470,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 +578,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)) { @@ -648,6 +628,7 @@ 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 .= addHidden('startMessage', $startMessage); $menu_row .= getButton('submit', 'delete', _("Delete")); $menu_row .= '' . _("Bypass Trash"); } else { @@ -731,6 +712,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 +754,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 +774,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 +804,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 +878,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 +904,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 +924,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 +934,6 @@ if (isset($sendreceipt)) { $message->is_mdnsent = true; $aMailbox['MSG_HEADERS'][$passed_id]['MESSAGE_OBJECT'] = $message; } - ClearAttachments(); } } /***********************************************/ @@ -879,11 +953,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 +967,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 +1023,12 @@ 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'); + ?> - - +