Problem/Motivation

The "Manage Display" portion of the Field UI loops through all known regions, and displays the fields for each of them.
Previous to 8.3.0, this list of regions was hardcoded to "Content" and "Disabled".

Now with the Field Layout module customizing this list of regions, a very confusing bug can arise.
When a region's machine name is changed during development, any field assigned to that region will no longer be shown in the UI.
This is mitigated by an easy workaround: change to another layout, and change back.

Proposed resolution

Because we can't (and shouldn't) do tracking of changes to the layout definitions, there is no reasonable way to update each instance of the layout.
Instead, the Field UI should show all fields with invalid regions in the "disabled" region.

Remaining tasks

User interface changes

Fields in invalid regions will be displayed as "Disabled"

API changes

N/A

Data model changes

N/A

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett created an issue. See original summary.

tim.plunkett’s picture

Issue summary: View changes
Status: Active » Needs review
FileSize
2.22 KB

Status: Needs review » Needs work

The last submitted patch, 2: 2870410-region-2.patch, failed testing.

tim.plunkett’s picture

Issue summary: View changes
Status: Needs work » Needs review
FileSize
1.5 KB
3.72 KB
DyanneNova’s picture

Status: Needs review » Reviewed & tested by the community

I've tested and the patch successfully moves the orphaned fields into the disabled region.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed 992ed45 to 8.4.x and 57bd004 to 8.3.x. Thanks!

@DyanneNova thanks for reviewing the issue. When you do post a review, be sure to describe what you reviewed and how. This helps other reviewers understand why you considered the issue RTBC (and is considered for issue credit).

  • alexpott committed 992ed45 on 8.4.x
    Issue #2870410 by tim.plunkett: Field UI should show fields with unknown...

  • alexpott committed 57bd004 on 8.3.x
    Issue #2870410 by tim.plunkett: Field UI should show fields with unknown...

Status: Fixed » Closed (fixed)

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