@corbosman
I did some improments in you great plugin (visual, ldap and sql search). Now the LDAP search is working fine. I think you can make this the 1.3 .
I tested with RC r2602, it's working fine with LDAP and SQL (I have 2 LDAP databases working with RC).
Thank you for the plugin development, is a great feature for rcmail.
Hi, again thanks for your patch! Im not sure though I agree with some of the changes you made. I incorporated the visual changes as they're nice, but the main issue, LDAP support, is imho not the right way to do it.
I think by far the most people will not be using LDAP, but just the default SQL. Your changes now force them, and everyone else, to always have to enter a search to find any address. Originally, this wasnt necessary as you'd immediately get a list of all addresses and you could immediately double click one. Very fast, unlike your version imho.
Also, i think there must be a way to make it work with list_records. See, if you go to the normal addressbook task, top-right, it also shows a list with all your contacts without having to search right? This uses the list_records method as well. Id rather find out why this isnt working for you. For me it actually is, im using FreeLDAP.org to test, and im seeing all my contacts.
Maybe you can put some debug calls in the code and find out whats going on.
Regards,
Cor