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

Problem/motivation

#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 http://drupal.org/node/1905070. 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 text module.

Schema in place

Schema not yet in place
text.settings.yml

Files: 
CommentFileSizeAuthor
#5 text_patch_4.PNG21.31 KBrachit_gupta
#4 1919210-text-schema-4.patch459 bytesrahuldolas123
PASSED: [[SimpleTest]]: [MySQL] 52,221 pass(es). View
#1 1919210-text-schema-1.patch360 bytesvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 50,742 pass(es). View

Comments

vijaycs85’s picture

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

Adding schema file

YesCT’s picture

for people coming in to test and review, see: http://drupal.org/node/1910624#steps-to-test in the issue summary for #1910624: [META] Introduce and complete configuration schemas in all of core
Those are draft instructions, please suggest improvements to them.

dags’s picture

Assigned: Unassigned » dags
rahuldolas123’s picture

FileSize
459 bytes
PASSED: [[SimpleTest]]: [MySQL] 52,221 pass(es). View

Added code styles and labels.

rachit_gupta’s picture

FileSize
21.31 KB

Hi

I have tested the above patch with config inspector module and the patch works.
Attaching the screenshot for the same.

vijaycs85’s picture

Status: Needs review » Reviewed & tested by the community

Thanks @rahuldolas123 and @rachit_gupta. Setting it RTBC.

dags’s picture

Assigned: dags » Unassigned

Patch in #4 looks right. Unassigning from myself.

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.