Spin off from #641670: Move $form['#field'] meta information into $form_state

The 'edit field settings' form has an widget element to enter a default value for the field.
The code reading and validating the entered can be made simpler if we force the form values for this element to be at top-level ('#parents' => array())

CommentFileSizeAuthor
#3 field-_ui_default_value-686226-3.patch4.54 KByched
field-_ui_default_value.patch4.19 KByched

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

field-_ui_default_value.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, field-_ui_default_value.patch, failed testing.

yched’s picture

yched commented
Status: Needs work » Needs review
StatusFileSize
new field-_ui_default_value-686226-3.patch4.54 KB

Rerolled.

yched’s picture

yched commented
Assigned: Unassigned » yched

+ easier tracking

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

#3: field-_ui_default_value-686226-3.patch queued for re-testing.

yched’s picture

yched commented

#3 is green

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Reviewed & tested by the community

Thanks, yched! This is a really nice clean-up.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Agreed. Committed to CVS HEAD. Thanks yched.

Status: Fixed » Closed (fixed)

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