Closed (fixed)
Project:
Lightweight Directory Access Protocol
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Dec 2012 at 22:38 UTC
Updated:
23 Apr 2013 at 05:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
johnbarclay commentedCan 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?
Comment #2
jdowner12 commentedIt sounds very similar , but not quite, the problem is that, if I'm understanding correctly the hidden field is actually showing.
Comment #3
johnbarclay commentedGotcha 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.
Comment #4
johnbarclay commentedComment #5
jdowner12 commentedI installed in my test environment but didn't see any differences. Should I be looking for something in particular?
Comment #6
johnbarclay commentedNo. 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.
Comment #7
froboyIt 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.
Comment #8
johnbarclay commentedI 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.
Comment #9
simon georges commentedChanging status, as there is a patch.
Comment #10
jdowner12 commentedgreat will test in the next few days and let you know.
Comment #11
johnbarclay commenteda css fix to hide the fields is probably most appropriate at this point until drupal 8.
Comment #12
yalet commentedI believe the attached patch does the trick as far as css goes.
Comment #13
yalet commentedComment #14
johnbarclay commentedI committed this. Thanks.
Comment #15
johnbarclay commented