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

Schema in place

Schema not yet in place
file.settings.yml

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vijaycs85’s picture

Status: Active » Needs review
FileSize
545 bytes

Adding schema file...

vijaycs85’s picture

Updating patch with code style from http://drupal.org/node/1905070#codestyle.

Status: Needs review » Needs work

The last submitted patch, 1919166-file-schema-2.patch, failed testing.

vijaycs85’s picture

FileSize
728 bytes

Label fix...

vijaycs85’s picture

Status: Needs work » Needs review
YesCT’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
567.92 KB

The schema file meets the standards in http://drupal.org/node/1905070#codestyle , looks good.

I did a manual visual comparison between the schema and the data yml, looks good.

I could not find where a form might be in the UI for description and icon, so did my best guess, which was to add a file field to a content type and then create that content.

comparison-file.png

So. rtbc.

--
only small thing I noticed was an extra blank like in the file.settings.yml

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.