[Tickets #12298] fix _buildSearchQuery() to generate correct search filter

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/12298
------------------------------------------------------------------------------
  Ticket             | 12298
  Created By         | Joerg.Pulz at frm2.tum.de
  Summary            | fix _buildSearchQuery() to generate correct search
                     | filter
  Queue              | Turba
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 3. High
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


Joerg.Pulz at frm2.tum.de (2013-06-04 10:26) wrote:

Turba generates wrong LDAP search filters when manually searching an  
contact or when Kronolith asks for the freebusyUrl.

The generated search filter for an manually contact search looks like:
(|)(mail=*<searchstring>*)(telephonenumber=*<searchstring>*)

but should look like:
(cn=*<searchstring>*)(mail=*<searchstring>*)(telephonenumber=*<searchstring>*)

The generated search filter for Kronolith automated freebusyUrl search  
  looks like:
(|)

but should look like:
(mail=user at domain.tld)

The attached patch fixes this. I don't know why "$key == 'OR'" causes  
this error as the value of $key is "0" when this occurs.



Joerg.Pulz at frm2.tum.de (2013-06-04 10:26) uploaded: turba_ldapfilter_fix.diff

http://bugs.horde.org/h/services/download/?module=whups&actionID=download_file&file=turba_ldapfilter_fix.diff&ticket=12298&fn=%2Fturba_ldapfilter_fix.diff