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

Schema in place

Schema not yet in place
action.settings.yml

Files: 
CommentFileSizeAuthor
#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

Comments

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
FileSize
396 bytes
PASSED: [[SimpleTest]]: [MySQL] 50,563 pass(es). View

Adding schema file.

vijaycs85’s picture

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

Updated code style and verified with config_inspector

2013-02-24_233505.png

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

FileSize
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.