With the integration of ldap_data module and the profile module, is the LDAP entry tab now redundant? Clicking on the LDAP entry tab shows no fields. I think all of the fields should now be under other tabs, so can we now remove this tab, or does it still serve a purpose for something?

CommentFileSizeAuthor
#2 ldapentrytab.diff2.88 KBgeodaniel

Comments

geodaniel’s picture

Version: 4.6.x-1.x-dev » master

This also seems to be the case in the 4.7 version

geodaniel’s picture

Version: master » 4.7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new2.88 KB

I've attached a patch which should remove the LDAP Entry tab from the account edit page, as I don't believe it's needed now that attribute mapping is done through the profile module.

(warning: it's not fully tested)

pablobm’s picture

Assigned: Unassigned » pablobm
Status: Needs review » Fixed

Hi, you are right, that category and the "Editing LDAP attributes directly" section on the ldapdata.module are redundant now. Anyway, I'll leave them there for now, in order not to mess up setups that might be using it. I will remove them for Drupal 5, though.

However, I have made the category not to show when it shouldn't (i.e.: when there is nothing under it). That should at least stop confusing matters for users.

geodaniel’s picture

Thanks Pablo!

Anonymous’s picture

Status: Fixed » Closed (fixed)