Hi Daniel
Your patches seem to work but they address a very specific requirement for a RoundCube installation. The upcoming plugin-API should provide enough hooks to satisfy such needs without adding another config option to the roundcube code base.
The generic LDAP class is a good approach and we'll add it to the trunk once it covers all LDAP operations (read and write). The rcube_usr_data class could then be added as a plugin that hooks into the creation of new users.
For now, let's just keep this as a patch (you should create a patch ticket on trac.roundcube.net) and we'll get back to it once the plugin API is ready.
Thanks a lot for your effort!
Thomas
On Thu, Oct 30, 2008 at 20:32, Daniel Mealha Cabrita dancab@utfpr.edu.br wrote:
hi there,
The following patches (see links below) provide each:
- rc_ldap_gen.patch (rcube_ldap_generic class) A generic interface for LDAP access (currently read-only operations). Essentially it's rcube_ldap class split in two, and it's a superclass of
rcube_ldap. rcube_ldap class behavior remains the same.
- rc_init_id.patch (rcube_usr_data class) It's a modular subsystem to collect the initial user data (such as name,
e-mail address, organization). It currently supports mechanisms 'ldap' and 'fixed'. Read main.inc.php.dist for instructions on how to use this. *REQUIRES* rc_ldap_gen.patch to be applied previously.
Both patches should work with the current development version.
The first patch replaces the previous one sent by me yesterday.
Feedback is appreciated.
-- Daniel Mealha Cabrita Divisao de Suporte Tecnico AINFO / Reitoria / UTFPR http://www.utfpr.edu.br
--- 8< --- detachments --- 8< --- The following attachments have been detached and are available for viewing. http://detached.gigo.com/rc/6c/QYMfn8rX/rc_init_id.patch http://detached.gigo.com/rc/6c/QYMfn8rX/rc_ldap_gen.patch Only click these links if you trust the sender, as well as this message. --- 8< --- detachments --- 8< ---
List info: http://lists.roundcube.net/dev/
List info: http://lists.roundcube.net/dev/