Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
the masquerade settings field that gets added to the user profile form does not have any weight assigned to it.
as soon as a user adds additional fields; since the masquerade field does not show on the manage fields form, it can't be moved relative to the new fields and ends up showing up in the middle of these new fields:
http://screencast.com/t/janRHvjvqj7
attached patch simply assigns that field a weight of 10 to move it down with the other admin fields
Comment | File | Size | Author |
---|---|---|---|
#7 | 1561352-masquerade-field-7.patch | 922 bytes | andypost |
#2 | field-extra-1561352-2.patch | 855 bytes | mokaspar |
masquerade-add_weight.patch | 507 bytes | liquidcms | |
Comments
Comment #1
andypostTo properly implement this functionality we need hook_field_extra_fields()
Comment #2
mokaspar CreditAttribution: mokaspar commentedHere's a patch using hook_field_extra_fields().
Comment #3
andypostGreat, +1 to be commited. let's get more reviews
Comment #4
Angry Dan CreditAttribution: Angry Dan commentedThis patch is working nicely for me, fixing issues with not seeing the link when using Display Suite.
I think, after all this time, it's probably safe to commit!
My only comment, I usually consider it a good practice to instantiate variables:
Thanks!
Comment #6
andypostcommited 8ee534e
Suppose this could be backported to 8x. probably to "advanced module"
Comment #7
andypostpushed ea7dae9
still waits for advanced module
Comment #9
andypostpushed to 8.x-2.x