Schema issues introduced [#2826556]

Problem/Motivation

In the latest release a schema issue was introduced.

Drupal\Core\Config\Schema\SchemaIncompleteException: Schema errors for
    simple_sitemap.custom with the following errors: simple_sitemap.custom:0
    missing schema

I guess its not possible to use config_object and sequence on the same level.

Proposed resolution

We should save the custom config in a sequence key "links"

Remaining tasks

User interface changes

API changes

Data model changes

Comment	File	Size	Author
#2	schema_issues_introduced-2826556-2.patch	2.42 KB	chr.fritsch
#2	8.x-2.x: PHP 7 & MySQL 5.5, D8.3 11 pass

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

11 November 2016 at 09:10

chr.fritsch created an issue. See original summary.

Comment #2

chr.fritsch

he/him

German

🇩🇪🇪🇺🌍

CreditAttribution: chr.fritsch at Thunder commented 11 November 2016 at 09:11

File	Size
schema_issues_introduced-2826556-2.patch	2.42 KB
8.x-2.x: PHP 7 & MySQL 5.5, D8.3 11 pass

Here is a patch.

Maybe we should add an update hook

Comment #3

chr.fritsch

he/him

German

🇩🇪🇪🇺🌍

CreditAttribution: chr.fritsch at Thunder commented 11 November 2016 at 09:11

Status:

Active

» Needs review

Comment #4

11 November 2016 at 11:50

gbyte.co committed f77dc99 on 8.x-2.x

Issue #2826556 by chr.fritsch: Schema issues introduced

Comment #5

gbyte CreditAttribution: gbyte as a volunteer and commented 11 November 2016 at 11:55

Version:	8.x-2.7	» 8.x-2.x-dev
Status:	Needs review	» Fixed

I've modified your code by changing the config calls from setData() to set(), as we can now target the 'links' key and don't need to unset the _core variable, added the update hook and enabled strict config checking in tests. Thanks and say hello to the Thunder team from me!