In schema.yml devel.settings.error_handlers is a mapping with two child elements: "error_handler_type" and "error_handler_value"; in config.devel.yml those elements have some default values (0 and 1).
But nowhere in code those elements are used and if we save the devel config form and then we look at generated configuration those element aren't present.
When we try to submit the config form in a test (where the schema validity is checked for consistency) the test fails.

I'm missing something or this is a bug?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lussoluca created an issue. See original summary.

lussoluca’s picture

Status: Active » Needs review
FileSize
1.12 KB

With this patch my tests on configuration form are green

itarato’s picture

I'd say the schema definition is fine, as it's really a map (at least the way Drupal Form API is handling it, and also by definition sequence is for lists where the key is unknown, however here the key is important). However the change of the setting seems sensible.

lussoluca’s picture

Yes, schema is correct but it doesn't match with the settings form

moshe weitzman’s picture

Status: Needs review » Reviewed & tested by the community

+ 1: 1 looks really odd. But if thats right, go ahead and commit.

  • lussoluca committed 90b2213 on 8.x-1.x
    Issue #2657032 by lussoluca: Unused element in schema and config
    
lussoluca’s picture

Status: Reviewed & tested by the community » Closed (fixed)

Committed to 8.x-1.x
Thanks!