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 actions module.

Schema in place

Schema not yet in place

#8 actions-schema-form.png7.45 KBsandipmkhairnar
#8 1912266-action-schema-8.patch475 bytessandipmkhairnar
PASSED: [[SimpleTest]]: [MySQL] 52,540 pass(es). View
#4 1912266-action-schema-4.patch463 bytesvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 52,246 pass(es). View
#4 2013-02-24_233505.png13.34 KBvijaycs85
#3 1912266-action-schema-3.patch396 bytesvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 50,563 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


Gábor Hojtsy’s picture

Issue tags: +Configuration schema

Tagging for config schema.

Gábor Hojtsy’s picture

Issue tags: -sprint

Remove from sprint.

vijaycs85’s picture

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

Adding schema file.

vijaycs85’s picture

13.34 KB
463 bytes
PASSED: [[SimpleTest]]: [MySQL] 52,246 pass(es). View

Updated code style and verified with config_inspector


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

The last submitted patch, 1912266-action-schema-4.patch, failed testing.

vijaycs85’s picture

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

#4: 1912266-action-schema-4.patch queued for re-testing.

vijaycs85’s picture

Still green and valid - Ready for review :)

sandipmkhairnar’s picture

475 bytes
PASSED: [[SimpleTest]]: [MySQL] 52,540 pass(es). View
7.45 KB

make a small change in label. Patch is working fine for me.
schema form

tim-e’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me

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.