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 datetime module.
Schema in place

Schema not yet in place
field.datetime.settings
field.datetime.instance_settings
field.datetime.value

field_widget.datetime_default.value
field_widget.datetime_datelist.value

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vijaycs85’s picture

Status: Active » Needs review
FileSize
1.41 KB

Initial patch...

swentel’s picture

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

Removed the widget from the schema file. Attaching the patch.

piyuesh23’s picture

Status: Needs work » Needs review
aspilicious’s picture

Status: Needs review » Needs work

Default value isn't a string. When I inspect it with the config inspector it tries to add an array in the default value. Which doesn't work. So this probably needs a special mapping.

vijaycs85’s picture

Issue summary: View changes
Status: Needs work » Needs review
Issue tags: +LONDON_2013_DECEMBER
FileSize
942 bytes
517 bytes

Thanks for the test @aspilicious. Fixed and tested with config_inspector.

vijaycs85’s picture

Adding config inspector screenshot...

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

Thnx! :)

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

yched’s picture

Status: Fixed » Needs review
FileSize
542 bytes

Quick followup: the user_register_form setting is long gone now :-)

yched’s picture

Status: Needs review » Fixed

Never mind, we'll fix this as part of #1989468: Weird messing with 'default_value_function' in date widgets ?.
Sorry for the noise.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.