holm/egroupware
1
1
Fork 0
forked from extern/egroupware
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix broken default undeleted flag filter

Browse Source
This commit is contained in:
Klaus Leithoff 2014-01-14 15:54:48 +00:00
parent b5ca559968
commit 092be08c96
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
mail/inc/class.mail_bo.inc.php
View File

@ -1403,12 +1403,12 @@ class mail_bo
$try2useCache = false; $try2useCache = false;
$filter = $this->createIMAPFilter($_folderName, $_filter); $filter = $this->createIMAPFilter($_folderName, $_filter);
//_debug_array($filter); //_debug_array($filter);
//error_log(__METHOD__.__LINE__.array2string($filter));
if($this->icServer->hasCapability('SORT')) { if($this->icServer->hasCapability('SORT')) {
if (self::$debug) error_log(__METHOD__." Mailserver has SORT Capability, SortBy: $_sort Reverse: $_reverse"); if (self::$debug) error_log(__METHOD__." Mailserver has SORT Capability, SortBy: $_sort Reverse: $_reverse");
$sortOrder = $this->_getSortString($_sort, $_reverse); $sortOrder = $this->_getSortString($_sort, $_reverse);
if ($_reverse && in_array(Horde_Imap_Client::SORT_REVERSE,$sortOrder)) $_reverse=false; // as we reversed the result already if ($_reverse && in_array(Horde_Imap_Client::SORT_REVERSE,$sortOrder)) $_reverse=false; // as we reversed the result already
if (self::$debug) error_log(__METHOD__." Mailserver runs SORT: SortBy: $sortOrder Filter: $filter"); if (self::$debug) error_log(__METHOD__." Mailserver runs SORT: SortBy: $sortOrder Filter: ".array2string($filter));
$sortResult = $this->icServer->search($_folderName, $filter, array( $sortResult = $this->icServer->search($_folderName, $filter, array(
'sort' => $sortOrder,)); 'sort' => $sortOrder,));
// if there is an PEAR Error, we assume that the server is not capable of sorting // if there is an PEAR Error, we assume that the server is not capable of sorting
@ -1528,10 +1528,11 @@ class mail_bo
//_debug_array($_criterias); //_debug_array($_criterias);
if (self::$debug) error_log(__METHOD__.__LINE__.' Criterias:'.(!is_array($_criterias)?" none -> returning $all":array2string($_criterias))); if (self::$debug) error_log(__METHOD__.__LINE__.' Criterias:'.(!is_array($_criterias)?" none -> returning $all":array2string($_criterias)));
if(!is_array($_criterias)) { if(!is_array($_criterias) || $_criterias['status']=='any') {
return $imapFilter->flag('DELETED', $set=false); $imapFilter->flag('DELETED', $set=false);
return $imapFilter;
} }
#error_log(print_r($_criterias, true)); //error_log(__METHOD__.__LINE__.print_r($_criterias, true));
$queryValid = false; $queryValid = false;
if(!empty($_criterias['string'])) { if(!empty($_criterias['string'])) {
$criteria = strtoupper($_criterias['type']); $criteria = strtoupper($_criterias['type']);
@ -1627,7 +1628,8 @@ class mail_bo
error_log(__METHOD__.__LINE__.' '.$query_str['query']); error_log(__METHOD__.__LINE__.' '.$query_str['query']);
} }
if($queryValid==false) { if($queryValid==false) {
return $imapFilter->flag('DELETED', $set=false); $imapFilter->flag('DELETED', $set=false);
return $imapFilter;
} else { } else {
return $imapFilter; return $imapFilter;
} }