Hello Colin,
* Colin Alston <[EMAIL PROTECTED]>, [2008-02-19 12:18 +0200]:
> On 19/02/2008 12:14 Emanuele Rocca wrote:
>> - $filter = '(&('.$this->prop['filter'] .')' . $filter . ')';
>> + $filter = '(&'.$this->prop['filter'] . $filter . ')';
>
> I don't grok this patch, the property filter _should_ be in brackets -
> ldap AND syntax being (&(arg)(arg)).
We have 'filter' => '(accountStatus=active)' in main.inc.php, which
produces "bad search filters".
So, two options: reverting that patch or stating in main.inc.php that
'filter' should be without surrounding brackets; the current comment is:
// &'d with search queries. ex: (status=act).
I'd be for the latter.
Thanks!
ciao,
ema
_______________________________________________
List info: http://lists.roundcube.net/dev/
