Problem/Motivation

In config_split.schema.schema.yml the config_split.config_split.* uuid property is defined as a string. However, this overrides the config_entity config schema for this property, losing the constraint validation added in #2870878: Add config validation for UUIDs

Proposed resolution

Remove the uuid property from the config_split.config_split config entity schema, so its definition is provided by the config_entity data type. This results in less code for Config Split and improves uuid data support.

Remaining tasks

  1. Write a patch
  2. Review
  3. Commit

User interface changes

None.

API changes

None.

Data model changes

The uuid property of config_split.config_split now has constraints by Drupal core.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

idebr created an issue. See original summary.

idebr’s picture

Issue summary: View changes
Status: Active » Needs review
FileSize
39.7 KB
39.39 KB
786 bytes

Attached patch removes the uuid property from the config_split.split config entity schema, so its definition is provided by the config_entity data type. This results in less code for Config Split and improves uuid data support.

Before:

After:

  • bircher committed e021e80 on 8.x-1.x
    Issue #3003959 by idebr: Defer config_split.config_split.*.uuid...
bircher’s picture

Status: Needs review » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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