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 field module.
Schema in place
Schema not yet in place
field.settings.yml
Comment | File | Size | Author |
---|---|---|---|
#11 | 1919164-11.jpg | 27.26 KB | sidharthap |
#10 | 1919164-field-schema-10.patch | 481 bytes | sandipmkhairnar |
#6 | 1919164-field-schema-6.patch | 480 bytes | sandipmkhairnar |
#6 | field-schema-form.png | 16.7 KB | sandipmkhairnar |
#5 | 1919164-field-schema-5.patch | 452 bytes | Vikas Negi |
Comments
Comment #1
vijaycs85Adding schema file...
Comment #2
Vikas Negi CreditAttribution: Vikas Negi commentedRe-rolling with code style fixes (http://drupal.org/node/1905070#codestyle)
Comment #4
vijaycs85missing 'the' as per code style :(
typo/mistake?
Comment #5
Vikas Negi CreditAttribution: Vikas Negi commentedFixed changes as per the above comments.
Comment #6
sandipmkhairnar CreditAttribution: sandipmkhairnar commentedCorrect the label for purge batch site and update the patch
Comment #7
vijaycs85One minor issue - Comment missing a full stop(".") at the end.
Comment #9
vijaycs85#6: 1919164-field-schema-6.patch queued for re-testing.
Comment #10
sandipmkhairnar CreditAttribution: sandipmkhairnar commentedThanks @vijaycs85. Update patch as per the comment
Comment #11
sidharthapI tested the patch #10, It works. Attach the screen shot. Thank you @sandipmkhairnar.
Comment #12
vijaycs85Thanks @sidharthap and @sandipmkhairnar. Looks good to go to me too.
Comment #13
webchickCommitted to 8.x, thanks! I'll push once testbot has caught up a bit.
Comment #14.0
(not verified) CreditAttribution: commentedUpdated issue summary.