Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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