On Tue, Feb 19, 2008 at 12:33 PM, Emanuele Rocca ema@galliera.it wrote:
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.
so where do we stand on this? I'd be for "fixing" the comments. In general, I'd like to support the syntax most people are familiar wit and use. So I have no idea about general LDAP usage. But please let us know so we can fix that before we release the next version.
The current code in rcube_ldap.inc is perfectly OK, and I see that the comment has been fixed by thomasb in r1129.
Maybe you could also add an &'d example?
example: &(status=act)(visible=true)
That would clarify the proper usage even further, in my opinion.
Thanks guys. ciao, ema _______________________________________________ List info: http://lists.roundcube.net/dev/