X-Git-Url: https://vcs.fsf.org/?p=squirrelmail.git;a=blobdiff_plain;f=functions%2Fmailbox_display.php;h=97f1418bbac8958512b97d7e590d1ded1fd80ff0;hp=309a6f43b7e2c76020daf572a757a78288d67a95;hb=ce20ea9260adccbf62ca895c7e372947c3007f21;hpb=be2d549570f1501e50b89557351aa3b0372e2e36 diff --git a/functions/mailbox_display.php b/functions/mailbox_display.php index 309a6f43..97f1418b 100644 --- a/functions/mailbox_display.php +++ b/functions/mailbox_display.php @@ -3,7 +3,7 @@ /** * mailbox_display.php * - * Copyright (c) 1999-2002 The SquirrelMail Project Team + * Copyright (c) 1999-2003 The SquirrelMail Project Team * Licensed under the GNU GPL. For full terms see the file COPYING. * * This contains functions that display mailbox information, such as the @@ -20,10 +20,24 @@ require_once(SM_PATH . 'functions/imap_mailbox.php'); /* Default value for page_selector_max. */ define('PG_SEL_MAX', 10); +function elapsed($start) +{ + $end = microtime(); + list($start2, $start1) = explode(" ", $start); + list($end2, $end1) = explode(" ", $end); + $diff1 = $end1 - $start1; + $diff2 = $end2 - $start2; + if( $diff2 < 0 ){ + $diff1 -= 1; + $diff2 += 1.0; + } + return $diff2 + $diff1; +} + function printMessageInfo($imapConnection, $t, $not_last=true, $key, $mailbox, $start_msg, $where, $what) { global $checkall, - $color, $msgs, $msort, + $color, $msgs, $msort, $td_str, $msg, $default_use_priority, $message_highlight_list, $index_order, @@ -58,26 +72,32 @@ function printMessageInfo($imapConnection, $t, $not_last=true, $key, $mailbox, if (handleAsSent($mailbox)) { $msg['FROM'] = $msg['TO']; + } + $msg['FROM'] = parseAddress($msg['FROM']); + /* * This is done in case you're looking into Sent folders, * because you can have multiple receivers. */ - $senderNames = explode(',', $msg['FROM']); - $senderName = ''; - if (sizeof($senderNames)){ - foreach ($senderNames as $senderNames_part) { + + $senderNames = $msg['FROM']; + $senderName = ''; + if (sizeof($senderNames)){ + foreach ($senderNames as $senderNames_part) { if ($senderName != '') { $senderName .= ', '; } - $senderName .= sqimap_find_displayable_name($senderNames_part); - } - } - } else { - $senderName = sqimap_find_displayable_name($msg['FROM']); + if ($senderNames_part[1]) { + $senderName .= decodeHeader($senderNames_part[1]); + } else { + $senderName .= htmlspecialchars($senderNames_part[0]); + } + } } - $subject = processSubject($msg['SUBJECT']); - + $msg['SUBJECT'] = decodeHeader($msg['SUBJECT']); + $subject = processSubject($msg['SUBJECT'], $indent_array[$msg['ID']]); + echo html_tag( 'tr','','','','VALIGN="top"') . "\n"; if (isset($msg['FLAG_FLAGGED']) && ($msg['FLAG_FLAGGED'] == true)) { @@ -118,22 +138,56 @@ function printMessageInfo($imapConnection, $t, $not_last=true, $key, $mailbox, * AAAAH! Make my eyes stop bleeding! * Who wrote this?! */ - if (sizeof($message_highlight_list)){ + if (is_array($message_highlight_list) && count($message_highlight_list)) { + $msg['TO'] = parseAddress($msg['TO']); + $msg['CC'] = parseAddress($msg['CC']); foreach ($message_highlight_list as $message_highlight_list_part) { if (trim($message_highlight_list_part['value']) != '') { $high_val = strtolower($message_highlight_list_part['value']); $match_type = strtoupper($message_highlight_list_part['match_type']); - if ($match_type == 'TO_CC') { - if (strstr('^^' . strtolower($msg['TO']), $high_val) || - strstr('^^' . strtolower($msg['CC']), $high_val)) { - $hlt_color = $message_highlight_list_part['color']; - continue; - } - } else { - if (strstr('^^' . strtolower($msg[$match_type]), $high_val)) { - $hlt_color = $message_highlight_list_part['color']; - continue; - } + switch($match_type) { + case('TO'): + case('TO_CC'): + foreach ($msg['TO'] as $address) { + $address[0] = decodeHeader($address[0]); + $address[1] = decodeHeader($address[1]); + if (strstr('^^' . strtolower($address[0]), $high_val) || + strstr('^^' . strtolower($address[1]), $high_val)) { + $hlt_color = $message_highlight_list_part['color']; + continue; + } + } + if($match_type != 'TO_CC') { + break; + } + case('CC'): + foreach ($msg['CC'] as $address) { + $address[0] = decodeHeader($address[0]); + $address[1] = decodeHeader($address[1]); + if( strstr('^^' . strtolower($address[0]), $high_val) || + strstr('^^' . strtolower($address[1]), $high_val)) { + $hlt_color = $message_highlight_list_part['color']; + continue; + } + } + break; + case('FROM'): + foreach ($msg['FROM'] as $address) { + $address[0] = decodeHeader($address[0]); + $address[1] = decodeHeader($address[1]); + if( strstr('^^' . strtolower($address[0]), $high_val) || + strstr('^^' . strtolower($address[1]), $high_val)) { + $hlt_color = $message_highlight_list_part['color']; + continue; + } + } + break; + default: + if (strstr('^^' . strtolower($msg[$match_type]), $high_val)) { + $hlt_color = $message_highlight_list_part['color']; + continue; + } + break; } } } @@ -142,7 +196,7 @@ function printMessageInfo($imapConnection, $t, $not_last=true, $key, $mailbox, if (!isset($hlt_color)) { $hlt_color = $color_string; } - $checked = ($checkall == 1) ? true : false; + $checked = ($checkall == 1) ? ' CHECKED' : ''; $col = 0; if (sizeof($index_order)) { foreach ($index_order as $index_order_part) { @@ -175,14 +229,14 @@ function printMessageInfo($imapConnection, $t, $not_last=true, $key, $mailbox, case 4: /* subject */ $td_str = $bold; if ($thread_sort_messages == 1) { - if (isset($indent_array[$msg["ID"]])) { + if (isset($indent_array[$msg['ID']])) { $td_str .= str_repeat("    ",$indent_array[$msg['ID']]); } } $td_str .= ''; + $t = elapsed($start); + //echo("elapsed time = $t seconds\n"); } function calc_msort($msgs, $sort) { @@ -455,12 +521,19 @@ function calc_msort($msgs, $sort) { * 4 = Subject (up) * 5 = Subject (dn) */ + if (($sort == 0) || ($sort == 1)) { - $msort = array_cleave ($msgs, 'TIME_STAMP'); + foreach ($msgs as $item) { + $msort[] = $item['TIME_STAMP']; + } } elseif (($sort == 2) || ($sort == 3)) { - $msort = array_cleave ($msgs, 'FROM-SORT'); + foreach ($msgs as $item) { + $msort[] = $item['FROM-SORT']; + } } elseif (($sort == 4) || ($sort == 5)) { - $msort = array_cleave ($msgs, 'SUBJECT-SORT'); + foreach ($msgs as $item) { + $msort[] = $item['SUBJECT-SORT']; + } } else { $msort = $msgs; } @@ -475,67 +548,7 @@ function calc_msort($msgs, $sort) { } function fillMessageArray($imapConnection, $id, $count) { - $msgs_list = sqimap_get_small_header_list($imapConnection, $id); - $messages = array(); - if (sizeof($msgs_list)) { - foreach ($msgs_list as $hdr) { - $unique_id[] = $hdr->uid; - $from[] = $hdr->from; - $date[] = $hdr->date; - $subject[] = $hdr->subject; - $to[] = $hdr->to; - $priority[] = $hdr->priority; - $cc[] = $hdr->cc; - $size[] = $hdr->size; - $type[] = $hdr->type0; - $flag_deleted[] = $hdr->flag_deleted; - $flag_answered[] = $hdr->flag_answered; - $flag_seen[] = $hdr->flag_seen; - $flag_flagged[] = $hdr->flag_flagged; - } - } - - for($j = 0; $j < $count; ++$j) { - if (isset($date[$j])) { - $date[$j] = str_replace(' ', ' ', $date[$j]); - $tmpdate = explode(' ', trim($date[$j])); - } else { - $tmpdate = $date = array('', '', '', '', '', ''); - } - $messages[$j]['TIME_STAMP'] = getTimeStamp($tmpdate); - $messages[$j]['DATE_STRING'] = - getDateString($messages[$j]['TIME_STAMP']); - $messages[$j]['ID'] = $unique_id[$j]; - $messages[$j]['FROM'] = decodeHeader($from[$j]); - $messages[$j]['FROM-SORT'] = - strtolower(sqimap_find_displayable_name(decodeHeader($from[$j]))); - $messages[$j]['SUBJECT'] = decodeHeader($subject[$j]); - $messages[$j]['SUBJECT-SORT'] = strtolower(decodeHeader($subject[$j])); - $messages[$j]['TO'] = decodeHeader($to[$j]); - $messages[$j]['PRIORITY'] = $priority[$j]; - $messages[$j]['CC'] = $cc[$j]; - $messages[$j]['SIZE'] = $size[$j]; - $messages[$j]['TYPE0'] = $type[$j]; - $messages[$j]['FLAG_DELETED'] = $flag_deleted[$j]; - $messages[$j]['FLAG_ANSWERED'] = $flag_answered[$j]; - $messages[$j]['FLAG_SEEN'] = $flag_seen[$j]; - $messages[$j]['FLAG_FLAGGED'] = $flag_flagged[$j]; - - /* - * fix SUBJECT-SORT to remove Re: - * vedr|sv (Danish) - * re|aw (English) - * - * TODO: i18n should be incorporated here. E.g. we catch the ones - * we know about, but also define in i18n what the localized - * "Re: " is for this or that locale. - */ - if (preg_match("/^(vedr|sv|re|aw):\s*(.*)$/si", - $messages[$j]['SUBJECT-SORT'], $matches)){ - $messages[$j]['SUBJECT-SORT'] = $matches[2]; - } - } - return $messages; + return sqimap_get_small_header_list($imapConnection, $id); } @@ -543,8 +556,7 @@ function fillMessageArray($imapConnection, $id, $count) { function displayMessageArray($imapConnection, $num_msgs, $start_msg, $msort, $mailbox, $sort, $color, $show_num, $where=0, $what=0) { - global $imapServerAddress, $use_mailbox_cache, - $index_order, $checkall, + global $imapServerAddress, $use_mailbox_cache, $index_order, $indent_array, $thread_sort_messages, $allow_server_sort, $server_sort_order, $PHP_SELF; @@ -555,7 +567,7 @@ function displayMessageArray($imapConnection, $num_msgs, $start_msg, $urlMailbox = urlencode($mailbox); /* get indent level for subject display */ - if ($thread_sort_messages == 1 ) { + if ($thread_sort_messages == 1 && $num_msgs) { $indent_array = get_parent_level($imapConnection); } @@ -579,7 +591,7 @@ function displayMessageArray($imapConnection, $num_msgs, $start_msg, /* messages display */ - if ($num_msgs == 0) { + if (!$num_msgs) { /* if there's no messages in this folder */ echo html_tag( 'tr', html_tag( 'td', @@ -663,13 +675,18 @@ function mail_message_listing_beginning ($imapConnection, if (!isset($msg)) { $msg = ''; } - $moveURL = "move_messages.php?msg=$msg&mailbox=$urlMailbox" - . "&startMessage=$start_msg"; + $moveFields = '' . + '' . + ''; + +// $moveURL = "move_messages.php?msg=$msg&mailbox=$urlMailbox" +// . "&startMessage=$start_msg"; /* * This is the beginning of the message list table. * It wraps around all messages */ - echo "
\n" + echo '' ."\n" + . $moveFields . html_tag( 'table' , html_tag( 'tr', html_tag( 'td' , @@ -713,7 +730,7 @@ function mail_message_listing_beginning ($imapConnection, echo getButton('SUBMIT', 'markRead',_("Read")); echo getButton('SUBMIT', 'markUnread',_("Unread")); - echo getButton('SUBMIT', 'delete',_("Delete")) ." \n"; + echo getButton('SUBMIT', 'delete',_("Delete")) ." \n"; if (!strpos($php_self,'mailbox')) { $location = $php_self.'?mailbox=INBOX&startMessage=1'; } else { @@ -785,8 +802,17 @@ function mail_message_listing_end($num_msgs, $paginator_str, $msg_cnt_str, $colo function printHeader($mailbox, $sort, $color, $showsort=true) { global $index_order; echo html_tag( 'tr' ,'' , 'center', $color[5] ); - for ($i = 1; $i <= count($index_order); $i++) { - switch ($index_order[$i]) { + + /* calculate the width of the subject column based on the + * widths of the other columns */ + $widths = array(1=>1,2=>25,3=>5,4=>0,5=>1,6=>5); + $subjectwidth = 100; + foreach($index_order as $item) { + $subjectwidth -= $widths[$item]; + } + + foreach ($index_order as $item) { + switch ($item) { case 1: /* checkbox */ case 5: /* flags */ echo html_tag( 'td' ,' ' , '', '', 'width="1%"' ); @@ -813,7 +839,7 @@ function printHeader($mailbox, $sort, $color, $showsort=true) { echo "\n"; break; case 4: /* subject */ - echo html_tag( 'td' ,'' , 'left', '', '' ) + echo html_tag( 'td' ,'' , 'left', '', 'width="'.$subjectwidth.'%"' ) . '' . _("Subject") . ''; if ($showsort) { ShowSortButton($sort, $mailbox, 4, 5); @@ -821,7 +847,7 @@ function printHeader($mailbox, $sort, $color, $showsort=true) { echo "\n"; break; case 6: /* size */ - echo html_tag( 'td', '' . _("Size") . '', 'center', '', 'width="5%"' ); + echo html_tag( 'td', '' . _("Size") . '', 'center', '', 'width="5%" nowrap' ); break; } } @@ -1153,32 +1179,45 @@ function get_paginator_str($box, $start_msg, $end_msg, $num_msgs, return ($result); } -function processSubject($subject) { +function processSubject($subject, $threadlevel = 0) { global $languages, $squirrelmail_language; /* Shouldn't ever happen -- caught too many times in the IMAP functions */ if ($subject == '') return _("(no subject)"); - if (strlen($subject) <= 55) + $trim_at = 55; + + /* if this is threaded, subtract two chars per indentlevel */ + if($threadlevel > 0 && $threadlevel <= 10) + $trim_at -= (2*$threadlevel); + + if (strlen($subject) <= $trim_at) return $subject; - $ent_strlen = strlen($subject); - $trim_val=50; - $ent_offset=0; + $ent_strlen = $orig_len = strlen($subject); + $trim_val = $trim_at - 5; + $ent_offset = 0; /* * see if this is entities-encoded string * If so, Iterate through the whole string, find out * the real number of characters, and if more - * than 55, substr with an updated trim value. + * than 55, substr with an updated trim value. */ - while ( (($ent_loc = strpos($subject, '&', $ent_offset)) !== false) && - (($ent_loc_end = strpos($subject, ';', $ent_loc)) !== false) ) { - $trim_val += ($ent_loc_end-$ent_loc)+1; - $ent_strlen -= $ent_loc_end-$ent_loc; + $step = $ent_loc = 0; + while ( $ent_loc < $trim_val && (($ent_loc = strpos($subject, '&', $ent_offset)) !== false) && + (($ent_loc_end = strpos($subject, ';', $ent_loc+3)) !== false) ) { + $trim_val += ($ent_loc_end-$ent_loc); $ent_offset = $ent_loc_end+1; + ++$step; } - - if ($ent_strlen <= 55){ + + if (($trim_val > 50) && (strlen($subject) > ($trim_val))&& (strpos($subject,';',$trim_val) < ($trim_val +6))) { + $i = strpos($subject,';',$trim_val); + if ($i) { + $trim_val = strpos($subject,';',$trim_val); + } + } + if ($ent_strlen <= $trim_at){ return $subject; } @@ -1186,7 +1225,6 @@ function processSubject($subject) { function_exists($languages[$squirrelmail_language]['XTRA_CODE'])) { return $languages[$squirrelmail_language]['XTRA_CODE']('strimwidth', $subject, $trim_val); } - return substr($subject, 0, $trim_val) . '...'; } @@ -1226,16 +1264,16 @@ function getEndMessage($start_msg, $show_num, $num_msgs) { } function handleAsSent($mailbox) { - global $sent_folder, $draft_folder, $handleAsSent_result; - + global $handleAsSent_result; + /* First check if this is the sent or draft folder. */ - $handleAsSent_result = (($mailbox == $sent_folder) - || ($mailbox == $draft_folder)); + $handleAsSent_result = isSentMailbox($mailbox) || isDraftMailbox($mailbox); /* Then check the result of the handleAsSent hook. */ do_hook('check_handleAsSent_result', $mailbox); /* And return the result. */ - return ($handleAsSent_result); + return $handleAsSent_result; } + ?>