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 <[EMAIL PROTECTED]> wrote: > hi there, > > > The following patches (see links below) provide each: > > 1. - 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. > > 2. - 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/
