As per in #1946404-42: Convert forms in field_ui.admin.inc to the new form interface :

HEAD patch
"Field edit" form
Class FieldSettingsForm FieldEditForm
Form id field_ui_field_settings_form field_ui_field_edit_form
Route field_ui.settings.$entity_type field_ui.field_edit.$entity_type
Path $field_overview_path/$field_name/field-settings $field_overview_path/$field_name/field
"Field instance edit" form
Class FieldInstanceEditForm (unchanged)
Form id field_ui_field_edit_form field_ui_field_instance_edit_form
Route field_ui.instance_edit.$entity_type (unchanged)
Path $field_overview_path/$field_name (unchanged)

.

+ Fix stale references to the former field_ui_*_form() functions in phpdocs.

CommentFileSizeAuthor
#1 field_ui-form_names-1981198-1.patch20.09 KByched

Comments

yched’s picture

Status: Active » Needs review
StatusFileSize
new20.09 KB

Patch

Status: Needs review » Needs work

The last submitted patch, field_ui-form_names-1981198-1.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review

#1: field_ui-form_names-1981198-1.patch queued for re-testing.

amateescu’s picture

Status: Needs review » Reviewed & tested by the community

Looks like a nice cleanup.

alexpott’s picture

Title: More accurate names and form_ids for Field UI form classes » Change notice: More accurate names and form_ids for Field UI form classes
Priority: Normal » Critical
Status: Reviewed & tested by the community » Active
Issue tags: +Needs change record

Committed f5945be and pushed to 8.x. Thanks!

yched’s picture

Priority: Critical » Normal
Status: Active » Fixed

Thanks !

Created http://drupal.org/node/1986552 (also documents the URL changes that were made in #1946404: Convert forms in field_ui.admin.inc to the new form interface)

star-szr’s picture

Title: Change notice: More accurate names and form_ids for Field UI form classes » More accurate names and form_ids for Field UI form classes
Issue tags: -Needs change record

Reverting title and tags.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

fix route name