This is a sub-issue of #1910624: [META] Introduce and complete configuration schemas in all of core.


#1866610: Introduce Kwalify-inspired schema format for configuration introduced the idea of config schema. The changelog leads to (hopefully extensive) documentation on the format at While there are little cleanups planned for the format overall, the current format is a result of months of back and forths, so it should be perfectly fine to apply it more widely to core.

Proposed solution

Create a configuration schema for node module.

Schema in place

Schema not yet in place

#6 1919186-node-schema-6.patch458 bytesvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 52,217 pass(es). View
#6 2013-02-24_222910.png12.85 KBvijaycs85
#1 1919186-node-schema-1.patch352 bytesvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 50,987 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


vijaycs85’s picture

Status: Active » Needs review
352 bytes
PASSED: [[SimpleTest]]: [MySQL] 50,987 pass(es). View

Adding schema file...

Status: Needs review » Needs work
Issue tags: -Configuration system, -D8MI, -language-config, -Configuration schema

The last submitted patch, 1919186-node-schema-1.patch, failed testing.

vijaycs85’s picture

Status: Needs work » Needs review
Issue tags: +Configuration system, +D8MI, +language-config, +Configuration schema

#1: 1919186-node-schema-1.patch queued for re-testing.

vijaycs85’s picture

FAILED: [[SimpleTest]]: [MySQL] Drupal installation failed.

locally installation seems fine - re-testing.

alexpott’s picture

Status: Needs review » Needs work
+++ b/core/modules/node/config/node.schema.ymlundefined
@@ -0,0 +1,7 @@
+    "items_per_page":

no need for double quotes

vijaycs85’s picture

Status: Needs work » Needs review
12.85 KB
458 bytes
PASSED: [[SimpleTest]]: [MySQL] 52,217 pass(es). View

Thanks for the review @alexpott. Updated:

1. Code style - as per
2. Added label
3. Verified with config_inspector (screenshot below)

pfrenssen’s picture

Assigned: Unassigned » pfrenssen

Assigning for review.

pfrenssen’s picture

Assigned: pfrenssen » Unassigned
Status: Needs review » Reviewed & tested by the community

Looks good!

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

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