Problem/Motivation

When an extra field with settings is configured, it does not comply with the configuration schema of Drupal.

Steps to reproduce

  • Install the Extra Field Settings Provider Example module.
  • Install the Configuration Inspector module.
  • Go to admin/structure/types/manage/page/display.
  • Place Example Node Label to the Content region.
  • Click on the settings icon next to the Example Node Label, and save the display settings as-is.
  • Go to admin/reports/config-inspector.
  • The core.entity_view_display.node.page.default config contains errors.
    • The content.extra_field_example_node_label.settings.wrapper has a missing schema.
    • The content.extra_field_example_node_label.settings.link_to_entity has a missing schema.

Proposed resolution

Add a 'type' config key to the extra fields, so a config schema for these extra fields can be created.

Remaining tasks

Create patch with proposed resolution.

User interface changes

None.

API changes

Only additions - it is now also possible to create a config schema for the settings of extra fields, but it doesn't change anything for fields that don't have one yet.

Data model changes

None.

Comments

bartlangelaan created an issue. See original summary.

bartlangelaan’s picture

Assigned: bartlangelaan » Unassigned
Status: Active » Needs review
StatusFileSize
new1.19 KB
new79.89 KB

See attached patch.
See also the screenshot for the changed result.

grevil’s picture

Status: Needs review » Reviewed & tested by the community

Just tested the patch through #3270794: Write tests for basic functionality! Without this patch, the tests would throw schema errors, with this patch applied the schema errors were gone and the view_display could be created correct and successfully!

anybody’s picture

Status: Reviewed & tested by the community » Fixed
anybody’s picture

  • Anybody committed 7c4394b on 8.x-2.x
    Issue #3278684 by bartlangelaan, Grevil: Invalid config schema
    

Status: Fixed » Closed (fixed)

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