In Users self edit section "LDAP Entries that have been provisioned from this Drupal user. " always displays. When a user logs in and goes to "edit" their account the is a "widget" that allows them to add ldap entries titled "LDAP Entries that have been provisioned from this Drupal user." I see this field in People>Account Settings>Manage Fields/Display and it is set to be a hidden text field. I do not see how to hide this option from users. I've attached a screen shot of what is displayed.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

johnbarclay’s picture

Can you reattach the screenshot? This may be a duplicate of #1831224: LDAP User: Fields visible to user in user/uid/edit form when LDAP modules disabled. Is the ldap_user module enabled when the user sees these? Or do they just appear when its disabled?

jdowner12’s picture

FileSize
126.05 KB

It sounds very similar , but not quite, the problem is that, if I'm understanding correctly the hidden field is actually showing.

johnbarclay’s picture

Title: In Users self edit section "LDAP Entries that have been provisioned from this Drupal user. " always displays. » LDAP User: user entity field 'ldap_user_prov_entries' has group heading showing in user profile form
Component: Security » Code

Gotcha now. Definately a different issue. What is happenning is that the field is hidden, but since it has a cardinality of "FIELD_CARDINALITY_UNLIMITED" a heading is showing even though each instance of the field is hidden.

This should be fixed with css or a field widget override. A quick fix is just to edit the field and remove the label and description in admin/config/people/accounts/fields. The little drag and drop box would still show though.

There must be a better way of dealing with this situation. Maybe it would be simpler just to require a hidden field module.

johnbarclay’s picture

Version: 7.x-2.0-beta2 » 7.x-2.x-dev
jdowner12’s picture

I installed in my test environment but didn't see any differences. Should I be looking for something in particular?

johnbarclay’s picture

No. Nothing has changed on this issue yet. If you are generally testing with 7.x-2.0-dev, just test whatever you would normally test and see if any new bugs are introduced.

froboy’s picture

FileSize
2.67 KB

It looks like http://drupal.org/project/field_hidden might do the trick for this.

In the mean time could we at least shorten the field labels? I was doing some work importing users via feeds and these long names made for an unusably-wide dropdown menu. I've attached a patch with updated labels, although we'd also probably have to have a db update for this too.

johnbarclay’s picture

I think requiring field_hidden is the way to go for ldap_user module. If someone wants to write the patch it would be great. Its silly to duplicate field_hidden's functionality and fields can't remain visible.

Simon Georges’s picture

Status: Active » Needs review

Changing status, as there is a patch.

jdowner12’s picture

great will test in the next few days and let you know.

johnbarclay’s picture

Status: Needs review » Needs work

a css fix to hide the fields is probably most appropriate at this point until drupal 8.

yalet’s picture

I believe the attached patch does the trick as far as css goes.

yalet’s picture

Status: Needs work » Needs review
johnbarclay’s picture

I committed this. Thanks.

johnbarclay’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.