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

Schema in place

Schema not yet in place
search.settings.yml

Files: 
CommentFileSizeAuthor
#6 1919196-search-schema-5.patch2.13 KBvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 52,258 pass(es). View
#4 1919196-search-schema-4.patch3.86 KBvijaycs85
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1919196-search-schema-4.patch. Unable to apply patch. See the log in the details link for more information. View
#4 2013-02-25_001504.png20.17 KBvijaycs85
#1 1919196-search-schema-1.patch1.27 KBvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 50,752 pass(es). View

Comments

vijaycs85’s picture

Status: Active » Needs review
FileSize
1.27 KB
PASSED: [[SimpleTest]]: [MySQL] 50,752 pass(es). View

Adding schema file...

Status: Needs review » Needs work

The last submitted patch, 1919196-search-schema-1.patch, failed testing.

vijaycs85’s picture

Status: Needs work » Needs review

test passed..

vijaycs85’s picture

FileSize
20.17 KB
3.86 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1919196-search-schema-4.patch. Unable to apply patch. See the log in the details link for more information. View

Updated code style and verified with config_inspector

2013-02-25_001504.png

Status: Needs review » Needs work

The last submitted patch, 1919196-search-schema-4.patch, failed testing.

vijaycs85’s picture

Status: Needs work » Needs review
FileSize
2.13 KB
PASSED: [[SimpleTest]]: [MySQL] 52,258 pass(es). View

Please ignore patch in #4
2013-02-25_001504.png

pfrenssen’s picture

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.