I've done a patch to add this functionality : add radio button in the admin panel to choose between hide, disable or show email field in profile.

the «cas.all.psy.patch» include this feature and spenserj & my correction to the «Modifications to LDAP Integration and LDAP Groups functionality» bugreport
the «cas.psy.patch» include just the feature.

I use ldap_integration code as a template, to do this feature.

Best

CommentFileSizeAuthor
cas.psy_.patch5.19 KBPsy-dupe
cas.all_.psy_.patch8.54 KBPsy-dupe

Comments

metzlerd’s picture

Version: master » 7.x-1.x-dev
Status: Needs review » Needs work

Because this patch includes LDAP code that is being completely tossed and refactored, I need to see this rerolled into current 7.1.x code. Showing email addy seems like a good idea, so if the submitter doesn't get this reworked early we'll try and revisit in 7.x-2 or later.

bfroehle’s picture

Status: Needs work » Closed (duplicate)

Psy: I'm going to close this issue as a duplicate of #1082602: Clean up admin page which aims to revamp the configuration page in general.