public ldap search features implemented

[Pieter Hoekstra]

Hi Justin,

It works pretty good, a little rough maybe, but a good start.

I've found one bug! When there is only one server, the server-dropdown
list is not displayed. This is fine. But, the javascript filling the
searchfield requires this dropdownlist! So, when there is only one
server, the searchfields are not filled and the searchfunction does not
work!!

Another thing that confused me, was the $rcmail_config['ldap_public']
config. The default value is false. But I wanted to enable it, so I set
it to true. But this doesn't work. It appears that you can set it to
false for disabling it, OR specify servers to enable it. I know this is
just the way PHP works, but it did confuse me... Maybe a better way, to
solve this, is not specify the $rcmail_config['ldap_public'] = false;
option in the default config, and that this option is disabled because
there aro no servers defined.

Few suggestions:
* Could the interface be more integrated with the addressbookinterface?
(Listing the results on the left)
* Could there be a "List all" option?
* I do no realy like the Searchfield dropdown. Could there be an option
so that automaticly all fields are searched?
* The searchfields could use aliasses...
* Optional integration with the Autocompletion feature would be great!

Overall nice work!
Pieter.


Justin Randell schreef:

> hi all,
>
> i've just committed some code enabling searching of public ldap 
> servers from roundcube. there may be a delay until the code is 
> available via the public sourceforge cvs servers, but it should be 
> there in the next day or so.
>
> the ldap search button will not work until you have configured a 
> public ldap server in the config/main.inc.php file. there is an 
> example configuration for the Verisign directory in 
> config/main.inc.php.dist.
>
> i'd appreciate some testing and bug hunting :-)
>
> also, the ldap icon in the addressbook page reflects my complete lack 
> of artistic abilities - please, could someone with some skills replace 
> it?
>
> cheers
> justin
>
>