search

plone / plone.app.ldap

Plone UI and integration of the functionality provided by LDAPMultiPlugins, LDAPUserFolder and PloneLDAP
3 stars 10 forks source link

Under "LDAP Schema" configuration tab: The "Plone property" called "mail" is actually "email" #9

Closed jenskl closed 8 years ago

jenskl commented 11 years ago

There is a default "Plone Property" named "mail" under "LDAP Schema". I think this is a typo, very hard to find, as one wants to map an ldap property of mailaddress to this "mail" property, one would think that this maps a user's mail setting in Plone.

However, the Plone property for mail adresses is actually named "email".

Please fix this typo as it is rather puzzling while setting up plone.app.ldap to match properties between ldap and plone.

MatthewWilkes commented 11 years ago

If you have any python experience you can feel free to open a pull request for this, which would get it fixed even faster. Typo fixes don't require you to have filled in the contributor agreement.

adaugherity commented 10 years ago

This report is wrong -- the configured default is to map the LDAP attribute "mail" to the Plone property "email", and has been since at least 2008 (the oldest commit shown for https://github.com/plone/plone.app.ldap/blob/master/plone/app/ldap/engine/storage.py).

Perhaps @jenskl is hitting #7. After performing the workaround for that (moving ldap-plugin to the top of acl_users/plugins:Properties Plugins via the ZMI), email addresses are indeed shown for LDAP users, without having to modify the LDAP schema.