Ticket #1485201 (closed Patches: fixed)

Opened 21 months ago

Last modified 7 weeks ago

Automatically add mail_domain to ldap contacts (Patch Attached)

Reported by: hayalci Owned by:
Priority: 5 Milestone: 0.3-stable
Component: LDAP connection Version: svn-trunk
Severity: minor Keywords:
Cc: ben@…

Description

Hi,

I made a patch to automatically add "mail_domain" from the config if the ldap search for "mail" field does not return a string with "@".

This will be useful in situations where ldap server does not hold complete e-mail adresses but the e-mail can be generated from other fields. It has no effect if email field indeed contains e-mail address.

In my setup main.inc.php includes this;

$rcmail_config['ldap_public']['hayalci'] = array(
    ....
    'email_field'   => 'uid',  // this field represents the contact's e-mail

If we include the patch to trunk, the help text may be;

// this field represents the contact's e-mail. mail_domain will be appended if no '@' character appears in the field content.

Attachments

autocomplete_ldap_mail_address.patch (0.8 KB) - added by hayalci 21 months ago.
Automatically append mail_domain if ldap address does not have '@'
fix_ldap_autocomplete_mail_domain.patch (0.6 KB) - added by hayalci 7 weeks ago.

Change History

Changed 21 months ago by hayalci

Automatically append mail_domain if ldap address does not have '@'

Changed 8 months ago by bjuteau

  • cc ben@… added

Changed 7 months ago by alec

  • status changed from new to closed
  • resolution set to fixed

Applied with changes in r2892.

Changed 7 weeks ago by hayalci

  • status changed from closed to reopened
  • resolution fixed deleted

There is a small problem with the ldap autocompletion. I am attaching a patch containing the fix.

Changed 7 weeks ago by hayalci

Changed 7 weeks ago by alec

  • status changed from reopened to closed
  • resolution set to fixed

Fixed in r3230.

Note: See TracTickets for help on using tickets.