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