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

Schema in place

Schema not yet in place
openid.settings.yml

Files: 
CommentFileSizeAuthor
#13 1919188-openid-schema.png24.53 KBsandipmkhairnar
#10 1919188-openid-schema-10.patch472 bytesrvilar
PASSED: [[SimpleTest]]: [MySQL] 52,231 pass(es). View
#8 1919188-openid-schema-8.patch473 bytesrvilar
PASSED: [[SimpleTest]]: [MySQL] 52,232 pass(es). View
#6 1919188-openid-schema-6.patch474 bytesrvilar
PASSED: [[SimpleTest]]: [MySQL] 52,202 pass(es). View
#4 1919188-openid-schema-4.patch651 bytescharmi
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1919188-openid-schema-4.patch. Unable to apply patch. See the log in the details link for more information. View
#1 1919188-openid-schema-1.patch368 bytesvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 50,751 pass(es). View

Comments

vijaycs85’s picture

Status: Active » Needs review
FileSize
368 bytes
PASSED: [[SimpleTest]]: [MySQL] 50,751 pass(es). View

Adding schema file...

Status: Needs review » Needs work
Issue tags: -Configuration system, -D8MI, -language-config, -Configuration schema

The last submitted patch, 1919188-openid-schema-1.patch, failed testing.

vijaycs85’s picture

Status: Needs work » Needs review
Issue tags: +Configuration system, +D8MI, +language-config, +Configuration schema

#1: 1919188-openid-schema-1.patch queued for re-testing.

charmi’s picture

FileSize
651 bytes
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1919188-openid-schema-4.patch. Unable to apply patch. See the log in the details link for more information. View

Added code and styles in openid module.

Status: Needs review » Needs work

The last submitted patch, 1919188-openid-schema-4.patch, failed testing.

rvilar’s picture

Status: Needs work » Needs review
FileSize
474 bytes
PASSED: [[SimpleTest]]: [MySQL] 52,202 pass(es). View

Attached a new patch to fix the test

vijaycs85’s picture

Thanks @rvilar, here is some minor label changes:

+++ b/core/modules/openid/config/schema/openid.schema.ymlundefined
@@ -0,0 +1,9 @@
+  type: mapping
+  label: 'Openid text settings'

Can we change "Openid text settings" to "Openid settings"

+++ b/core/modules/openid/config/schema/openid.schema.ymlundefined
@@ -0,0 +1,9 @@
+      type: string
+      label: 'Xri proxy resolver:'

and "proxy resolver for XRI" for this?

rvilar’s picture

FileSize
473 bytes
PASSED: [[SimpleTest]]: [MySQL] 52,232 pass(es). View

New patch with label modifications

vijaycs85’s picture

+++ b/core/modules/openid/config/schema/openid.schema.ymlundefined
@@ -0,0 +1,9 @@
+      type: string
+      label: 'Proxy resolver for XRI:'

Can we remove ':' here please?

Also thinking 'Openid settings' might need to be 'OpenID settings'?, but we can leave that to @YesCT or @Gabor.

rvilar’s picture

FileSize
472 bytes
PASSED: [[SimpleTest]]: [MySQL] 52,231 pass(es). View

New patch

vijaycs85’s picture

for some reasons, I don't see fields in config_inspector.

rvilar’s picture

I see them without any problems.

sandipmkhairnar’s picture

FileSize
24.53 KB

I have reviewed the patche and its working fine.
schema form

pfrenssen’s picture

Assigned: Unassigned » pfrenssen

Assigning for review.

pfrenssen’s picture

Assigned: pfrenssen » Unassigned
Status: Needs review » Reviewed & tested by the community

I did not see the openid schema either in the configuration inspector, until I enabled the OpenID module :D

Looks good, RTBC!

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.