Switch from hook_form_alter() to hook_field_settings_form() [#1111180]

To add the Field Permissions options to the Field Settings, we currently use a hook_form_alter(). We might gain benefit out of using hook_field_settings_form() instead.

Comment	File	Size	Author
#3	1111180.patch	2.99 KB	robloach
#2	1111180.patch	4.6 KB	robloach

Comments

Comment #1

nguyentran commented 30 September 2011 at 08:58

How to use and save value, thanks.

Comment #2

robloach

he/him

commented 13 October 2011 at 16:49

Status:

Active

» Needs work

Status	File	Size
new	1111180.patch	4.6 KB

Turns out field_ui_field_settings_form() only calls hook_field_settings_form() for the module that creates the field. This seems like a core bug as it means modules outside the field defining module can have field settings.