X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=functions%2Fimap_search.php;h=fb9e95b53b1115e462ca796696863ad59e247431;hb=dd628162b2f12906e91c827183eda8135570ba7a;hp=70793f8c437d70d1adc9c4dadd815548149eb46a;hpb=8ceffac2c31e48c1db2012a2dc68070d6b0834ee;p=squirrelmail.git
diff --git a/functions/imap_search.php b/functions/imap_search.php
index 70793f8c..fb9e95b5 100644
--- a/functions/imap_search.php
+++ b/functions/imap_search.php
@@ -20,8 +20,9 @@ require_once('../functions/mime.php');
function sqimap_search($imapConnection, $search_where, $search_what, $mailbox,
$color, $search_position = '', $search_all, $count_all) {
- global $msgs, $message_highlight_list, $squirrelmail_language, $languages,
- $index_order, $pos;
+ global $message_highlight_list, $squirrelmail_language, $languages,
+ $index_order, $pos, $allow_charset_search, $uid_support,
+ $imap_server_type;
$pos = $search_position;
@@ -33,26 +34,31 @@ function sqimap_search($imapConnection, $search_where, $search_what, $mailbox,
$search_what = ereg_replace('[ ]{2,}', ' ', $search_what);
$multi_search = explode(' ', $search_what);
$search_string = '';
- foreach ($multi_search as $multi_search_part) {
- $search_string .= $search_where . ' "' . $multi_search_part . '" ';
- }
-/*
- if (count($multi_search)==1) {
- $search_string = $search_where . ' ' . '"' . $multi_search[0] . '"';
+
+ /* it seems macosx does not support the prefered search
+ syntax so we fall back to the older style. This IMAP
+ server has a problem with multiple search terms. Instead
+ of returning the messages that match all the terms it
+ returns the messages that match each term. Could be fixed
+ on the client side, but should be fixed on the server
+ as per the RFC */
+
+ if ($imap_server_type == 'macosx') {
+ foreach ($multi_search as $multi_search_part) {
+ $search_string .= $search_where . ' ' .$multi_search_part . ' ';
+ }
}
else {
- $search_string = '';
- $count = count($multi_search);
- for ($x=0;$x<$count;$x++) {
- trim($multi_search[$x]);
- $search_string = $search_string . ' ' . $search_where . ' "' . $multi_search[$x] . '"';
- }
+ foreach ($multi_search as $multi_search_part) {
+ $search_string .= $search_where . ' {' . strlen($multi_search_part)
+ . "}\r\n" . $multi_search_part . ' ';
+ }
}
-*/
+
$search_string = trim($search_string);
/* now use $search_string in the imap search */
- if (isset($languages[$squirrelmail_language]['CHARSET']) &&
+ if ($allow_charset_search && isset($languages[$squirrelmail_language]['CHARSET']) &&
$languages[$squirrelmail_language]['CHARSET']) {
$ss = "SEARCH CHARSET "
. strtoupper($languages[$squirrelmail_language]['CHARSET'])
@@ -62,7 +68,7 @@ function sqimap_search($imapConnection, $search_where, $search_what, $mailbox,
}
/* read data back from IMAP */
- $readin = sqimap_run_command($imapConnection, $ss, true, $result, $message);
+ $readin = sqimap_run_command($imapConnection, $ss, false, $result, $message, $uid_support);
/* try US-ASCII charset if search fails */
if (isset($languages[$squirrelmail_language]['CHARSET'])
@@ -73,13 +79,12 @@ function sqimap_search($imapConnection, $search_where, $search_what, $mailbox,
}
unset($messagelist);
- $msgs = '';
/* Keep going till we find the SEARCH response */
foreach ($readin as $readin_part) {
/* Check to see if a SEARCH response was received */
if (substr($readin_part, 0, 9) == '* SEARCH ') {
- $messagelist = explode(' ', substr($readin_part, 9));
+ $messagelist = preg_split("/ /", substr($readin_part, 9));
} else if (isset($errors)) {
$errors = $errors.$readin_part;
} else {
@@ -90,129 +95,25 @@ function sqimap_search($imapConnection, $search_where, $search_what, $mailbox,
/* If nothing is found * SEARCH should be the first error else echo errors */
if (isset($errors)) {
if (strstr($errors,'* SEARCH')) {
- if ($search_all != 'all') {
- echo '