From 73045ee858e88c6fb27cb92b419643315b21a83c Mon Sep 17 00:00:00 2001 From: stekkel Date: Sun, 6 Jun 2004 01:43:35 +0000 Subject: [PATCH] Fixed created search strings. --- functions/imap_asearch.php | 780 +++++++++++++++++++------------------ 1 file changed, 398 insertions(+), 382 deletions(-) diff --git a/functions/imap_asearch.php b/functions/imap_asearch.php index 667e96149..1b129564c 100644 --- a/functions/imap_asearch.php +++ b/functions/imap_asearch.php @@ -1,131 +1,131 @@ => 'asequence', */ // Special handling, @see sqimap_asearch_build_criteria() /*'ALL' is binary operator */ - 'ANSWERED' => '', - 'BCC' => 'astring', - 'BEFORE' => 'adate', - 'BODY' => 'astring', - 'CC' => 'astring', - 'DELETED' => '', - 'DRAFT' => '', - 'FLAGGED' => '', - 'FROM' => 'astring', - 'HEADER' => 'afield', // Special syntax for this one, @see sqimap_asearch_build_criteria() - 'KEYWORD' => 'akeyword', - 'LARGER' => 'anum', - 'NEW' => '', + 'ANSWERED' => '', + 'BCC' => 'astring', + 'BEFORE' => 'adate', + 'BODY' => 'astring', + 'CC' => 'astring', + 'DELETED' => '', + 'DRAFT' => '', + 'FLAGGED' => '', + 'FROM' => 'astring', + 'HEADER' => 'afield', // Special syntax for this one, @see sqimap_asearch_build_criteria() + 'KEYWORD' => 'akeyword', + 'LARGER' => 'anum', + 'NEW' => '', /*'NOT' is unary operator */ - 'OLD' => '', - 'ON' => 'adate', + 'OLD' => '', + 'ON' => 'adate', /*'OR' is binary operator */ - 'RECENT' => '', - 'SEEN' => '', - 'SENTBEFORE' => 'adate', - 'SENTON' => 'adate', - 'SENTSINCE' => 'adate', - 'SINCE' => 'adate', - 'SMALLER' => 'anum', - 'SUBJECT' => 'astring', - 'TEXT' => 'astring', - 'TO' => 'astring', - 'UID' => 'asequence', - 'UNANSWERED' => '', - 'UNDELETED' => '', - 'UNDRAFT' => '', - 'UNFLAGGED' => '', - 'UNKEYWORD' => 'akeyword', - 'UNSEEN' => '' + 'RECENT' => '', + 'SEEN' => '', + 'SENTBEFORE' => 'adate', + 'SENTON' => 'adate', + 'SENTSINCE' => 'adate', + 'SINCE' => 'adate', + 'SMALLER' => 'anum', + 'SUBJECT' => 'astring', + 'TEXT' => 'astring', + 'TO' => 'astring', + 'UID' => 'asequence', + 'UNANSWERED' => '', + 'UNDELETED' => '', + 'UNDRAFT' => '', + 'UNFLAGGED' => '', + 'UNKEYWORD' => 'akeyword', + 'UNSEEN' => '' ); /** Imap SEARCH month names encoding - * @global array $imap_asearch_months - */ +* @global array $imap_asearch_months +*/ $imap_asearch_months = array( - '01' => 'jan', - '02' => 'feb', - '03' => 'mar', - '04' => 'apr', - '05' => 'may', - '06' => 'jun', - '07' => 'jul', - '08' => 'aug', - '09' => 'sep', - '10' => 'oct', - '11' => 'nov', - '12' => 'dec' + '01' => 'jan', + '02' => 'feb', + '03' => 'mar', + '04' => 'apr', + '05' => 'may', + '06' => 'jun', + '07' => 'jul', + '08' => 'aug', + '09' => 'sep', + '10' => 'oct', + '11' => 'nov', + '12' => 'dec' ); /** Error message titles according to imap server returned code - * @global array $imap_error_titles - */ +* @global array $imap_error_titles +*/ $imap_error_titles = array( - 'OK' => '', - 'NO' => _("ERROR : Could not complete request."), - 'BAD' => _("ERROR : Bad or malformed request."), - 'BYE' => _("ERROR : Imap server closed the connection."), - '' => _("ERROR : Connection dropped by imap-server.") + 'OK' => '', + 'NO' => _("ERROR : Could not complete request."), + 'BAD' => _("ERROR : Bad or malformed request."), + 'BYE' => _("ERROR : Imap server closed the connection."), + '' => _("ERROR : Connection dropped by imap-server.") ); /** - * Function to display an error related to an IMAP-query. - * We need to do our own error management since we may receive NO responses on purpose (even BAD with SORT or THREAD) - * so we call sqimap_error_box() if the function exists (sm >= 1.5) or use our own embedded code - * @global array imap_error_titles - * @param string $response the imap server response code - * @param string $query the failed query - * @param string $message an optional error message - * @param string $link an optional link to try again - */ +* Function to display an error related to an IMAP-query. +* We need to do our own error management since we may receive NO responses on purpose (even BAD with SORT or THREAD) +* so we call sqimap_error_box() if the function exists (sm >= 1.5) or use our own embedded code +* @global array imap_error_titles +* @param string $response the imap server response code +* @param string $query the failed query +* @param string $message an optional error message +* @param string $link an optional link to try again +*/ //@global array color sm colors array function sqimap_asearch_error_box($response, $query, $message, $link = '') { - global $imap_error_titles; + global $imap_error_titles; - if (!array_key_exists($response, $imap_error_titles)) - $title = _("ERROR : Unknown imap response."); - else - $title = $imap_error_titles[$response]; - if ($link == '') - $message_title = _("Reason Given: "); - else - $message_title = _("Possible reason : "); - if (function_exists('sqimap_error_box')) - sqimap_error_box($title, $query, $message_title, $message, $link); - else { //Straight copy of 1.5 imap_general.php:sqimap_error_box(). Can be removed at a later time - global $color; + if (!array_key_exists($response, $imap_error_titles)) + $title = _("ERROR : Unknown imap response."); + else + $title = $imap_error_titles[$response]; + if ($link == '') + $message_title = _("Reason Given: "); + else + $message_title = _("Possible reason : "); + if (function_exists('sqimap_error_box')) + sqimap_error_box($title, $query, $message_title, $message, $link); + else { //Straight copy of 1.5 imap_general.php:sqimap_error_box(). Can be removed at a later time + global $color; require_once(SM_PATH . 'functions/display_messages.php'); $string = "\n" . $title . "
\n"; if ($query != '') @@ -138,333 +138,333 @@ function sqimap_asearch_error_box($response, $query, $message, $link = '') $string .= $link; $string .= "

\n"; error_box($string,$color); - } + } } /** - * This is a convenient way to avoid spreading if (isset(... all over the code - * @param mixed $var any variable (reference) - * @param mixed $def default value to return if unset (default is zls (''), pass 0 or array() when appropriate) - * @return mixed $def if $var is unset, otherwise $var - */ +* This is a convenient way to avoid spreading if (isset(... all over the code +* @param mixed $var any variable (reference) +* @param mixed $def default value to return if unset (default is zls (''), pass 0 or array() when appropriate) +* @return mixed $def if $var is unset, otherwise $var +*/ function asearch_nz(&$var, $def = '') { - if (isset($var)) - return $var; - return $def; + if (isset($var)) + return $var; + return $def; } /** - * This should give the same results as PHP 4 >= 4.3.0's html_entity_decode(), - * except it doesn't handle hex constructs - * @param string $string string to unhtmlentity() - * @return string decoded string - */ +* This should give the same results as PHP 4 >= 4.3.0's html_entity_decode(), +* except it doesn't handle hex constructs +* @param string $string string to unhtmlentity() +* @return string decoded string +*/ function asearch_unhtmlentities($string) { - $trans_tbl = array_flip(get_html_translation_table(HTML_ENTITIES)); - for ($i=127; $i<255; $i++) /* Add &#; entities */ - $trans_tbl['&#' . $i . ';'] = chr($i); - return strtr($string, $trans_tbl); + $trans_tbl = array_flip(get_html_translation_table(HTML_ENTITIES)); + for ($i=127; $i<255; $i++) /* Add &#; entities */ + $trans_tbl['&#' . $i . ';'] = chr($i); + return strtr($string, $trans_tbl); /* I think the one above is quicker, though it should be benchmarked - $string = strtr($string, array_flip(get_html_translation_table(HTML_ENTITIES))); - return preg_replace("/&#([0-9]+);/E", "chr('\\1')", $string); + $string = strtr($string, array_flip(get_html_translation_table(HTML_ENTITIES))); + return preg_replace("/&#([0-9]+);/E", "chr('\\1')", $string); */ } /** - * Provide an easy way to dump the imap dialogue if $imap_asearch_debug_dump is TRUE - * @global bool imap_asearch_debug_dump - * @param string $var_name - * @param string $var_var - */ +* Provide an easy way to dump the imap dialogue if $imap_asearch_debug_dump is TRUE +* @global bool imap_asearch_debug_dump +* @param string $var_name +* @param string $var_var +*/ function s_debug_dump($var_name, $var_var) { - global $imap_asearch_debug_dump; - if ($imap_asearch_debug_dump) { - if (function_exists('sm_print_r')) //Only exists since 1.4.2 - sm_print_r($var_name, $var_var); //Better be the 'varargs' version ;) - else { - echo '
';
-			echo htmlentities($var_name);
-			print_r($var_var);
-			echo '
'; - } - } + global $imap_asearch_debug_dump; + if ($imap_asearch_debug_dump) { + if (function_exists('sm_print_r')) //Only exists since 1.4.2 + sm_print_r($var_name, $var_var); //Better be the 'varargs' version ;) + else { + echo '
';
+            echo htmlentities($var_name);
+            print_r($var_var);
+            echo '
'; + } + } } /** Encode a string to quoted or literal as defined in rfc 3501 - * - * - 4.3 String: - * A quoted string is a sequence of zero or more 7-bit characters, - * excluding CR and LF, with double quote (<">) characters at each end. - * - 9. Formal Syntax: - * quoted-specials = DQUOTE / "\" - * @param string $what string to encode - * @param string $charset search charset used - * @return string encoded string - */ +* +* - 4.3 String: +* A quoted string is a sequence of zero or more 7-bit characters, +* excluding CR and LF, with double quote (<">) characters at each end. +* - 9. Formal Syntax: +* quoted-specials = DQUOTE / "\" +* @param string $what string to encode +* @param string $charset search charset used +* @return string encoded string +*/ function sqimap_asearch_encode_string($what, $charset) { - if (strtoupper($charset) == 'ISO-2022-JP') // This should be now handled in imap_utf7_local? - $what = mb_convert_encoding($what, 'JIS', 'auto'); - if (preg_match('/["\\\\\r\n\x80-\xff]/', $what)) - return '{' . strlen($what) . "}\r\n" . $what; // 4.3 literal form - return '"' . $what . '"'; // 4.3 quoted string form + if (strtoupper($charset) == 'ISO-2022-JP') // This should be now handled in imap_utf7_local? + $what = mb_convert_encoding($what, 'JIS', 'auto'); + if (preg_match('/["\\\\\r\n\x80-\xff]/', $what)) + return '{' . strlen($what) . "}\r\n" . $what; // 4.3 literal form + return '"' . $what . '"'; // 4.3 quoted string form } /** - * Parses a user date string into an rfc 3501 date string - * Handles space, slash, backslash, dot and comma as separators (and dash of course ;=) - * @global array imap_asearch_months - * @param string user date - * @return array a preg_match-style array: - * - [0] = fully formatted rfc 3501 date string (--<4 digit year>) - * - [1] = day - * - [2] = month - * - [3] = year - */ +* Parses a user date string into an rfc 3501 date string +* Handles space, slash, backslash, dot and comma as separators (and dash of course ;=) +* @global array imap_asearch_months +* @param string user date +* @return array a preg_match-style array: +* - [0] = fully formatted rfc 3501 date string (--<4 digit year>) +* - [1] = day +* - [2] = month +* - [3] = year +*/ function sqimap_asearch_parse_date($what) { - global $imap_asearch_months; + global $imap_asearch_months; - $what = trim($what); - $what = ereg_replace('[ /\\.,]+', '-', $what); - if ($what) { - preg_match('/^([0-9]+)-+([^\-]+)-+([0-9]+)$/', $what, $what_parts); - if (count($what_parts) == 4) { - $what_month = strtolower(asearch_unhtmlentities($what_parts[2])); + $what = trim($what); + $what = ereg_replace('[ /\\.,]+', '-', $what); + if ($what) { + preg_match('/^([0-9]+)-+([^\-]+)-+([0-9]+)$/', $what, $what_parts); + if (count($what_parts) == 4) { + $what_month = strtolower(asearch_unhtmlentities($what_parts[2])); /* if (!in_array($what_month, $imap_asearch_months)) {*/ - foreach ($imap_asearch_months as $month_number => $month_code) { - if (($what_month == $month_number) - || ($what_month == $month_code) - || ($what_month == strtolower(asearch_unhtmlentities(getMonthName($month_number)))) - || ($what_month == strtolower(asearch_unhtmlentities(getMonthAbrv($month_number)))) - ) { - $what_parts[2] = $month_number; - $what_parts[0] = $what_parts[1] . '-' . $month_code . '-' . $what_parts[3]; - break; - } - } + foreach ($imap_asearch_months as $month_number => $month_code) { + if (($what_month == $month_number) + || ($what_month == $month_code) + || ($what_month == strtolower(asearch_unhtmlentities(getMonthName($month_number)))) + || ($what_month == strtolower(asearch_unhtmlentities(getMonthAbrv($month_number)))) + ) { + $what_parts[2] = $month_number; + $what_parts[0] = $what_parts[1] . '-' . $month_code . '-' . $what_parts[3]; + break; + } + } /* }*/ - } - } - else - $what_parts = array(); - return $what_parts; + } + } + else + $what_parts = array(); + return $what_parts; } /** - * Build one criteria sequence - * @global array imap_asearch_opcodes - * @param string $opcode search opcode - * @param string $what opcode argument - * @param string $charset search charset - * @return string one full criteria sequence - */ +* Build one criteria sequence +* @global array imap_asearch_opcodes +* @param string $opcode search opcode +* @param string $what opcode argument +* @param string $charset search charset +* @return string one full criteria sequence +*/ function sqimap_asearch_build_criteria($opcode, $what, $charset) { - global $imap_asearch_opcodes; + global $imap_asearch_opcodes; - $criteria = ''; - switch ($imap_asearch_opcodes[$opcode]) { - default: - case 'anum': - $what = str_replace(' ', '', $what); - $what = ereg_replace('[^0-9]+[^KMG]$', '', strtoupper($what)); - if ($what != '') { - switch (substr($what, -1)) { - case 'G': - $what = substr($what, 0, -1) << 30; - break; - case 'M': - $what = substr($what, 0, -1) << 20; - break; - case 'K': - $what = substr($what, 0, -1) << 10; - break; - } - $criteria = $opcode . ' ' . $what . ' '; - } - break; - case '': //aflag - $criteria = $opcode . ' '; - break; - case 'afield': /* HEADER field-name: field-body */ - preg_match('/^([^:]+):(.*)$/', $what, $what_parts); - if (count($what_parts) == 3) - $criteria = $opcode . ' ' . - sqimap_asearch_encode_string($what_parts[1], $charset) . ' ' . - sqimap_asearch_encode_string($what_parts[2], $charset) . ' '; - break; - case 'adate': - $what_parts = sqimap_asearch_parse_date($what); - if (isset($what_parts[0])) - $criteria = $opcode . ' ' . $what_parts[0] . ' '; - break; - case 'akeyword': - case 'astring': - $criteria = $opcode . ' ' . sqimap_asearch_encode_string($what, $charset) . ' '; - break; - case 'asequence': - $what = ereg_replace('[^0-9:\(\)]+', '', $what); - if ($what != '') - $criteria = $opcode . ' ' . $what . ' '; - break; - } - return $criteria; + $criteria = ''; + switch ($imap_asearch_opcodes[$opcode]) { + default: + case 'anum': + $what = str_replace(' ', '', $what); + $what = ereg_replace('[^0-9]+[^KMG]$', '', strtoupper($what)); + if ($what != '') { + switch (substr($what, -1)) { + case 'G': + $what = substr($what, 0, -1) << 30; + break; + case 'M': + $what = substr($what, 0, -1) << 20; + break; + case 'K': + $what = substr($what, 0, -1) << 10; + break; + } + $criteria = $opcode . ' ' . $what . ' '; + } + break; + case '': //aflag + $criteria = $opcode . ' '; + break; + case 'afield': /* HEADER field-name: field-body */ + preg_match('/^([^:]+):(.*)$/', $what, $what_parts); + if (count($what_parts) == 3) + $criteria = $opcode . ' ' . + sqimap_asearch_encode_string($what_parts[1], $charset) . ' ' . + sqimap_asearch_encode_string($what_parts[2], $charset) . ' '; + break; + case 'adate': + $what_parts = sqimap_asearch_parse_date($what); + if (isset($what_parts[0])) + $criteria = $opcode . ' ' . $what_parts[0] . ' '; + break; + case 'akeyword': + case 'astring': + $criteria = $opcode . ' ' . sqimap_asearch_encode_string($what, $charset) . ' '; + break; + case 'asequence': + $what = ereg_replace('[^0-9:\(\)]+', '', $what); + if ($what != '') + $criteria = $opcode . ' ' . $what . ' '; + break; + } + return $criteria; } /** - * Another way to do array_values(array_unique(array_merge($to, $from))); - * @param array $to to array (reference) - * @param array $from from array - * @return array uniquely merged array - */ +* Another way to do array_values(array_unique(array_merge($to, $from))); +* @param array $to to array (reference) +* @param array $from from array +* @return array uniquely merged array +*/ function sqimap_array_merge_unique(&$to, $from) { - if (empty($to)) - return $from; - $count = count($from); - for ($i = 0; $i < $count; $i++) { - if (!in_array($from[$i], $to)) - $to[] = $from[$i]; - } - return $to; + if (empty($to)) + return $from; + $count = count($from); + for ($i = 0; $i < $count; $i++) { + if (!in_array($from[$i], $to)) + $to[] = $from[$i]; + } + return $to; } /** - * Run the imap SEARCH command as defined in rfc 3501 - * @link http://www.ietf.org/rfc/rfc3501.txt - * @param resource $imapConnection the current imap stream - * @param string $search_string the full search expression eg "ALL RECENT" - * @param string $search_charset charset to use or zls ('') - * @return array an IDs or UIDs array of matching messages or an empty array - */ +* Run the imap SEARCH command as defined in rfc 3501 +* @link http://www.ietf.org/rfc/rfc3501.txt +* @param resource $imapConnection the current imap stream +* @param string $search_string the full search expression eg "ALL RECENT" +* @param string $search_charset charset to use or zls ('') +* @return array an IDs or UIDs array of matching messages or an empty array +*/ function sqimap_run_search($imapConnection, $search_string, $search_charset) { - //For some reason, this seems to happen and forbids searching servers not allowing OPTIONAL [CHARSET] - if (strtoupper($search_charset) == 'US-ASCII') - $search_charset = ''; - /* 6.4.4 try OPTIONAL [CHARSET] specification first */ - if ($search_charset != '') - $query = 'SEARCH CHARSET "' . strtoupper($search_charset) . '" ' . $search_string; - else - $query = 'SEARCH ' . $search_string; - s_debug_dump('C:', $query); - $readin = sqimap_run_command($imapConnection, $query, false, $response, $message, TRUE); + //For some reason, this seems to happen and forbids searching servers not allowing OPTIONAL [CHARSET] + if (strtoupper($search_charset) == 'US-ASCII') + $search_charset = ''; + /* 6.4.4 try OPTIONAL [CHARSET] specification first */ + if ($search_charset != '') + $query = 'SEARCH CHARSET "' . strtoupper($search_charset) . '" ' . $search_string; + else + $query = 'SEARCH ' . $search_string; + s_debug_dump('C:', $query); + $readin = sqimap_run_command($imapConnection, $query, false, $response, $message, TRUE); - /* 6.4.4 try US-ASCII charset if we tried an OPTIONAL [CHARSET] and received a tagged NO response (SHOULD be [BADCHARSET]) */ - if (($search_charset != '') && (strtoupper($response) == 'NO')) { - $query = 'SEARCH CHARSET US-ASCII ' . $search_string; - s_debug_dump('C:', $query); - $readin = sqimap_run_command($imapConnection, $query, false, $response, $message, TRUE); - } - if (strtoupper($response) != 'OK') { - sqimap_asearch_error_box($response, $query, $message); - return array(); - } + /* 6.4.4 try US-ASCII charset if we tried an OPTIONAL [CHARSET] and received a tagged NO response (SHOULD be [BADCHARSET]) */ + if (($search_charset != '') && (strtoupper($response) == 'NO')) { + $query = 'SEARCH CHARSET US-ASCII ' . $search_string; + s_debug_dump('C:', $query); + $readin = sqimap_run_command($imapConnection, $query, false, $response, $message, TRUE); + } + if (strtoupper($response) != 'OK') { + sqimap_asearch_error_box($response, $query, $message); + return array(); + } $messagelist = parseUidList($readin,'SEARCH'); - if (empty($messagelist)) //Empty search response, ie '* SEARCH' - return array(); + if (empty($messagelist)) //Empty search response, ie '* SEARCH' + return array(); - $cnt = count($messagelist); - for ($q = 0; $q < $cnt; $q++) - $id[$q] = trim($messagelist[$q]); - return $id; + $cnt = count($messagelist); + for ($q = 0; $q < $cnt; $q++) + $id[$q] = trim($messagelist[$q]); + return $id; } /** - * @global bool allow_charset_search user setting - * @global array languages sm languages array - * @global string squirrelmail_language user language setting - * @return string the user defined charset if $allow_charset_search is TRUE else zls ('') - */ +* @global bool allow_charset_search user setting +* @global array languages sm languages array +* @global string squirrelmail_language user language setting +* @return string the user defined charset if $allow_charset_search is TRUE else zls ('') +*/ function sqimap_asearch_get_charset() { - global $allow_charset_search, $languages, $squirrelmail_language; + global $allow_charset_search, $languages, $squirrelmail_language; - if ($allow_charset_search) - return $languages[$squirrelmail_language]['CHARSET']; - return ''; + if ($allow_charset_search) + return $languages[$squirrelmail_language]['CHARSET']; + return ''; } /** - * Convert sm internal sort to imap sort taking care of: - * - user defined date sorting (ARRIVAL vs DATE) - * - if the searched mailbox is the sent folder then TO is being used instead of FROM - * - reverse order by using REVERSE - * @param string $mailbox mailbox name to sort - * @param integer $sort_by sm sort criteria index - * @global bool internal_date_sort sort by arrival date instead of message date - * @global string sent_folder sent folder name - * @return string imap sort criteria - */ +* Convert sm internal sort to imap sort taking care of: +* - user defined date sorting (ARRIVAL vs DATE) +* - if the searched mailbox is the sent folder then TO is being used instead of FROM +* - reverse order by using REVERSE +* @param string $mailbox mailbox name to sort +* @param integer $sort_by sm sort criteria index +* @global bool internal_date_sort sort by arrival date instead of message date +* @global string sent_folder sent folder name +* @return string imap sort criteria +*/ function sqimap_asearch_get_sort_criteria($mailbox, $sort_by) { - global $internal_date_sort, $sent_folder; + global $internal_date_sort, $sent_folder; - $sort_opcodes = array ('DATE', 'FROM', 'SUBJECT', 'SIZE'); - if ($internal_date_sort == true) - $sort_opcodes[0] = 'ARRIVAL'; + $sort_opcodes = array ('DATE', 'FROM', 'SUBJECT', 'SIZE'); + if ($internal_date_sort == true) + $sort_opcodes[0] = 'ARRIVAL'; // if (handleAsSent($mailbox)) // if (isSentFolder($mailbox)) - if ($mailbox == $sent_folder) - $sort_opcodes[1] = 'TO'; - return (($sort_by % 2) ? '' : 'REVERSE ') . $sort_opcodes[($sort_by >> 1) & 3]; + if ($mailbox == $sent_folder) + $sort_opcodes[1] = 'TO'; + return (($sort_by % 2) ? '' : 'REVERSE ') . $sort_opcodes[($sort_by >> 1) & 3]; } /** - * @param string $cur_mailbox unformatted mailbox name - * @param array $boxes_unformatted selectable mailbox unformatted names array (reference) - * @return array sub mailboxes unformatted names - */ +* @param string $cur_mailbox unformatted mailbox name +* @param array $boxes_unformatted selectable mailbox unformatted names array (reference) +* @return array sub mailboxes unformatted names +*/ function sqimap_asearch_get_sub_mailboxes($cur_mailbox, &$mboxes_array) { - $sub_mboxes_array = array(); - $boxcount = count($mboxes_array); - for ($boxnum=0; $boxnum < $boxcount; $boxnum++) { - if (isBoxBelow($mboxes_array[$boxnum], $cur_mailbox)) - $sub_mboxes_array[] = $mboxes_array[$boxnum]; - } - return $sub_mboxes_array; + $sub_mboxes_array = array(); + $boxcount = count($mboxes_array); + for ($boxnum=0; $boxnum < $boxcount; $boxnum++) { + if (isBoxBelow($mboxes_array[$boxnum], $cur_mailbox)) + $sub_mboxes_array[] = $mboxes_array[$boxnum]; + } + return $sub_mboxes_array; } /** - * Create the search query strings for all given criteria and merge results for every mailbox - * @param resource $imapConnection - * @param array $mailbox_array (reference) - * @param array $biop_array (reference) - * @param array $unop_array (reference) - * @param array $where_array (reference) - * @param array $what_array (reference) - * @param array $exclude_array (reference) - * @param array $sub_array (reference) - * @param array $mboxes_array selectable unformatted mailboxes names (reference) - * @return array array(mailbox => array(UIDs)) - */ +* Create the search query strings for all given criteria and merge results for every mailbox +* @param resource $imapConnection +* @param array $mailbox_array (reference) +* @param array $biop_array (reference) +* @param array $unop_array (reference) +* @param array $where_array (reference) +* @param array $what_array (reference) +* @param array $exclude_array (reference) +* @param array $sub_array (reference) +* @param array $mboxes_array selectable unformatted mailboxes names (reference) +* @return array array(mailbox => array(UIDs)) +*/ function sqimap_asearch($imapConnection, &$mailbox_array, &$biop_array, &$unop_array, &$where_array, &$what_array, &$exclude_array, &$sub_array, &$mboxes_array) { - $search_charset = sqimap_asearch_get_charset(); - $mbox_msgs = array(); + $search_charset = sqimap_asearch_get_charset(); + $mbox_msgs = array(); $mbox_search = array(); - $search_string = ''; - $cur_mailbox = $mailbox_array[0]; - $cur_biop = ''; /* Start with ALL */ - /* We loop one more time than the real array count, so the last search gets fired */ - for ($cur_crit=0,$iCnt=count($where_array); $cur_crit <= $iCnt; ++$cur_crit) { - if (empty($exclude_array[$cur_crit])) { - $next_mailbox = $mailbox_array[$cur_crit]; - if ($next_mailbox != $cur_mailbox) { - $search_string = trim($search_string); /* Trim out last space */ - if ($cur_mailbox == 'All Folders') - $search_mboxes = $mboxes_array; - else if ((!empty($sub_array[$cur_crit - 1])) || (!in_array($cur_mailbox, $mboxes_array))) - $search_mboxes = sqimap_asearch_get_sub_mailboxes($cur_mailbox, $mboxes_array); - else - $search_mboxes = array($cur_mailbox); - foreach ($search_mboxes as $cur_mailbox) { + $search_string = ''; + $cur_mailbox = $mailbox_array[0]; + $cur_biop = ''; /* Start with ALL */ + /* We loop one more time than the real array count, so the last search gets fired */ + for ($cur_crit=0,$iCnt=count($where_array); $cur_crit <= $iCnt; ++$cur_crit) { + if (empty($exclude_array[$cur_crit])) { + $next_mailbox = (isset($mailbox_array[$cur_crit])) ? $mailbox_array[$cur_crit] : false; + if ($next_mailbox != $cur_mailbox) { + $search_string = trim($search_string); /* Trim out last space */ + if ($cur_mailbox == 'All Folders') + $search_mboxes = $mboxes_array; + else if ((!empty($sub_array[$cur_crit - 1])) || (!in_array($cur_mailbox, $mboxes_array))) + $search_mboxes = sqimap_asearch_get_sub_mailboxes($cur_mailbox, $mboxes_array); + else + $search_mboxes = array($cur_mailbox); + foreach ($search_mboxes as $cur_mailbox) { if (isset($mbox_search[$cur_mailbox])) { $mbox_search[$cur_mailbox]['search'] .= ' ' . $search_string; } else { @@ -472,44 +472,60 @@ function sqimap_asearch($imapConnection, &$mailbox_array, &$biop_array, &$unop_a } $mbox_search[$cur_mailbox]['charset'] = $search_charset; } - $cur_mailbox = $next_mailbox; - $search_string = ''; + $cur_mailbox = $next_mailbox; + $search_string = ''; - } - if (isset($where_array[$cur_crit]) && empty($exclude_array[$cur_crit])) { - $criteria = sqimap_asearch_build_criteria($where_array[$cur_crit], $what_array[$cur_crit], $search_charset); - if (!empty($criteria)) { - //$criteria = 'ALL '. $criteria; - $unop = $unop_array[$cur_crit]; - if (!empty($unop)) { - $criteria = $unop . ' ' . $criteria; - } else { - $criteria = 'ALL ' . $criteria; + } + if (isset($where_array[$cur_crit]) && empty($exclude_array[$cur_crit])) { + for ($crit = $cur_crit; $crit < count($where_array); $crit++) { + $criteria = trim(sqimap_asearch_build_criteria($where_array[$crit], $what_array[$crit], $search_charset)); + if (!empty($criteria) && empty($exclude_array[$crit])) { + if (asearch_nz($mailbox_array[$crit]) == $cur_mailbox) { + $unop = $unop_array[$crit]; + if (!empty($unop)) { + $criteria = $unop . ' ' . $criteria; + } + $aCriteria[] = array($biop_array[$crit], $criteria); + } } - /* We need to infix the next non-excluded criteria's biop if it's the same mailbox */ - $next_biop = ''; - for ($next_crit = $cur_crit+1; $next_crit <= count($where_array); $next_crit++) { - if (empty($exclude_array[$next_crit])) { - if (asearch_nz($mailbox_array[$next_crit]) == $cur_mailbox) { - $next_biop = asearch_nz($biop_array[$next_crit]); - if ($next_biop == 'OR' || $next_biop == 'ALL') { - $next_criterium = sqimap_asearch_build_criteria($where_array[$next_crit], $what_array[$next_crit], $search_charset); - // unset something - $exclude_array[$next_crit] = true; - $criteria .= $next_biop . ' '. $next_criterium; + // unset something + $exclude_array[$crit] = true; + } + $aSearch = array(); + for($i=0,$iCnt=count($aCriteria);$i<$iCnt;++$i) { + $cur_biop = $aCriteria[$i][0]; + $next_biop = (isset($aCriteria[$i+1][0])) ? $aCriteria[$i+1][0] : false; + if ($next_biop != $cur_biop && $next_biop == 'OR') { + $aSearch[] = 'OR '.$aCriteria[$i][1]; + } else if ($cur_biop != 'OR') { + $aSearch[] = 'ALL '.$aCriteria[$i][1]; + } else { // or only supports 2 search keys so we need to create a parenthized list + $prev_biop = (isset($aCriteria[$i-1][0])) ? $aCriteria[$i-1][0] : false; + if ($prev_biop == $cur_biop) { + $last = $aSearch[$i-1]; + if (!substr($last,-1) == ')') { + $aSearch[$i-1] = "(OR $last"; + $aSearch[] = $aCriteria[$i][1].')'; + } else { + $sEnd = ''; + while ($last && substr($last,-1) == ')') { + $last = substr($last,0,-1); + $sEnd .= ')'; } - } - } - } - //if ($next_biop == 'OR') - // $criteria = $next_biop . ' ' . $criteria; - $search_string .= $criteria; - //$cur_biop = asearch_nz($biop_array[$cur_crit]); - } - } + $aSearch[$i-1] = "(OR $last"; + $aSearch[] = $aCriteria[$i][1].$sEnd.')'; + } + } else { + $aSearch[] = $aCriteria[$i][1]; + } + } - } - } + } + $search_string .= implode(' ',$aSearch); + } + + } + } return ($mbox_search); }