Follow up for #1953404: Add config schema to field and instance config entities

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. As a part of schema addition to field module(#1953404: Add config schema to field and instance config entities), found that we have to provide schema for field types, field widgets and field storage.

Proposed solution

Create a configuration schema for field types, field widget and field storage (if applicable) defined in link module.
Schema in place

Schema not in place
field.link.settings
field.link.instance_settings
field.link.value

field_widget.link_default.settings

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vijaycs85’s picture

Title: Copy of Provide config schema to field types, widgets and storage in link module » Provide config schema to field types, widgets and storage in link module
aitiba’s picture

Status: Active » Needs review
FileSize
1.02 KB

After reading all the link provided and taking a look to the similar issue, this is what I have. :-)

swentel’s picture

Title: Provide config schema to field types, widgets and storage in link module » Provide config schema to field types and storage in link module
Status: Needs review » Needs work
piyuesh23’s picture

Status: Needs work » Needs review
FileSize
893 bytes

Moved field_widget out of the schema. Attaching the updated patch.

Status: Needs review » Needs work

The last submitted patch, 1983906-provide-config-schema-in-lin-module-4.patch, failed testing.

piyuesh23’s picture

Patch rerolled. Attaching a fixed patch.

piyuesh23’s picture

Status: Needs work » Needs review
aitiba’s picture

With the last commit of the 8.x branch and on this branch I make the "git apply *.patch" and I dont get any errors. I dont know if this information is that is needed to review the issue or I need to be sure that on drupal screens go well too? If I need to be sure of this, ¿how can I make this on this case?

piyuesh23’s picture

@aitiba The best way to test this would be using config_inspector module(https://drupal.org/project/config_inspector).
For details on how to use it, checkout this link https://drupal.org/node/1910624#comment-7088154.

sushantpaste’s picture

@piyuesh23 Small modification in patch will be

 - # Schema for configuration files of the number module.
 + # Schema for configuration files of the link module.

S.

sushantpaste’s picture

Here is the updated patch.

S.

aspilicious’s picture

Status: Needs review » Needs work

This one needs work, sequence needs a '-' when defining the sequence.
And newline needed on the end.

vijaycs85’s picture

thanks for the review @aspilicious. Here is the updated one with inspector screenshot...

aspilicious’s picture

+  mapping:
+    title:
+      type: integer
+      label: 'Allow link text'

This should be radio buttons, but it's understandable we can't determine that while processing the form...
Going to rtbc when it comes back green.

aspilicious’s picture

In the file patch we decided to got with the string for the radio, can't we do that here to?

vijaycs85’s picture

Status: Needs review » Closed (duplicate)
Related issues: +#2167623: Add test for all default configuration to ensure schema exists and is correct

The patch on this issue has been updated as part of #2167623: Add test for all default configuration to ensure schema exists and is correct. As this issue doesn't have any test to confirm/validate the schema, making this change and closing this issue as duplicate of #2167623: Add test for all default configuration to ensure schema exists and is correct. The contributors of this issue (in commit message) is copied to #2167623: Add test for all default configuration to ensure schema exists and is correct.