Hi again,
I few months ago I sent a patch to this list for a more general ldap
search on email addresses but no one replied to it: http://lists.roundcube.net/mail-archive/roundcube.dev/2006/06/118/
I tried to patch my ldapsearchform.inc but not succeeded in, so I did it "in my own way" ; here is my patch :
http://pedrov.kwain.net/~pedrov/rcube/ldapsearchform.diff
The original LDAP search doesn't work for me, as users in my directory mostly have more than one email address, so the search results looked strange. This patch doesn't change the default search behavior.
On my copy, it seems to be working nice. In fact I had not multiple-email contacts but I tried and did not encountered problems. The search works well, and it prints multiple mail addresses whenever it is necessary. Here is a screenshot of what it looks like :
http://pedrov.kwain.net/~pedrov/rcube/rcube_multiple_mail.png
So maybe it would make sense to integrate it in your patch and extend the search function generally to more LDAP attributes like birthday, telephone numbers etc. Most LDAP implementations provide these attributes so why not use them.
Why not ? I think we should focus on the "InetOrgPerson" structure, which must be the most common.
Right now the ldap search isn't working anyway in SVN as there seems to be a major rewrite going on: http://lists.roundcube.net/mail-archive/roundcube.dev/2006/09/136/
Perhaps should I wait a little then, instead of flooding the dev list with patches about LDAP :-)
regards,