Item8978: Config.spec misses {Ldap}{MailAttribute}

Priority:	CurrentState:	AppliesTo:	Component:	WaitingFor:
Normal	Waiting for Release	Extension	LdapContrib	Main.MichaelDaum

LdapContrib's Config.spec misses {Ldap}{MailAttribute}, making it impossible to be changed from configure. The default value mail is good, but not always correct.

-- GilmarSantosJr - 05 May 2010

Oh sure. Have a try to add it ton Config.spec.

-- MichaelDaum - 08 Jun 2010

I made this change:

$ git diff
diff --git a/LdapContrib/lib/Foswiki/Contrib/LdapContrib/Config.spec b/LdapContrib/lib/Foswiki/Contrib/LdapContrib/Config.spec
index 19bce32..ed6e26a 100644
--- a/LdapContrib/lib/Foswiki/Contrib/LdapContrib/Config.spec
+++ b/LdapContrib/lib/Foswiki/Contrib/LdapContrib/Config.spec
@@ -107,6 +107,11 @@ $Foswiki::cfg{Ldap}{UserScope} = 'sub';
 $Foswiki::cfg{Ldap}{LoginAttribute} = 'uid';
 
 # **STRING**
+# The user mail attribute. This is the attribute name used to fetch
+# users e-mail.
+$Foswiki::cfg{Ldap}{MailAttribute} = 'mail';
+
+# **STRING**
 # The user's wiki name attribute. This is the attribute to generate
 # the WikiName from. 
 $Foswiki::cfg{Ldap}{WikiNameAttribute} = 'cn';

And it worked. May I commit?

-- GilmarSantosJr - 08 Jun 2010

Sure.

-- MichaelDaum - 08 Jun 2010

Done. I marked as Waiting for Release to remind you this fix is not released yet

-- GilmarSantosJr - 09 Jun 2010