message for SchemaIncompleteException uses config_name:property which is unclear

Created on 26 February 2024, 4 months ago

Updated 16 April 2024, 2 months ago

Problem/Motivation

Errors in config schema which are checked by ConfigSchemaChecker throw a SchemaIncompleteException with messages like this:

> Drupal\Core\Config\Schema\SchemaIncompleteException: Schema errors for action_link.action_link.test_changed with the following errors: action_link.action_link.test_changed:plugin_config missing schema

It's not clear what the 'foo.bar:biz.bax' format means (and it's not documented in code either -- 🐛 SchemaCheckTrait::checkConfigSchema() and checkValue() don't property document the @return Active ).

Changing this to explicitly say config foo.bar property biz.bax would be clearer.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

🐛 Bug report

Status

Needs work

Version

11.0 🔥

Component

Configuration →

Last updated about 2 hours ago

Maintained by
🇬🇧United Kingdom @alexpott
🇺🇸United States @tim.plunkett

Created by

🇬🇧United Kingdom joachim

Live updates comments and jobs are added and updated live.

DX (Developer Experience)
Enhances developer experience.

Novice
It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Merge Requests

!6770message for SchemaIncompleteException uses config_name:property which is unclear
Open
🇬🇧United Kingdom joachim
updated 2 months ago

Comments & Activities

Issue created by @joachim
Merge request !6770Changed exception error message to be clearer. → (Open) created by joachim
Status changed to Needs review 4 months ago1:33pm 26 February 2024
Comment 4 months ago →
🇬🇧United Kingdom joachim
Pipeline finished with Failed
4 months ago
Total: 659s
#104146
Status changed to Needs work 4 months ago2:20pm 26 February 2024
Comment 4 months ago →
🇺🇸United States smustgrave
Appears to have test failures.
Comment 4 months ago →
🇬🇧United Kingdom joachim
Tagging as novice, because test failures should be just a case of updating the expected exception messages.
First commit to issue fork.
Pipeline finished with Failed
2 months ago
Total: 862s
#147020
Comment 2 months ago →
🇫🇮Finland markusVJH
I added a commit to try to update the tests to match the new format of the expectation message, apologies if I jumped the shark on that one. I am not yet sure should be done about the test failures @joachim.

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024