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, -- Pierre
