Write configuration schema for node types

Initial patch...

form from config_inspector...

I've had a look over this and am not sure I understand correctly how the schema files work, so here a summary and a couple of examples of the things I found and if I'm right I'll post the rest of them in more detail:

• I can't see the label "Content type" on the content type edit form (line 13 of node.schema.yml)
• Line 18: should it be uuid and not uid ?
• Is the file structured according to how the edit form is structured, or according to the structure of the data? help (line 27) and title_label (line 33) are under "Submission form settings" in the edit form, but are elsewhere in node.schema.yml
• Comment Settings and Menu Settings are completely missing - is that correct?

As I say, there are a few other things I found but wanted to make sure I understand this correctly before reporting stuff!

While testing out the node schema, found configuration issue for with "Content type settings", Schema say the value for following fields,

• Published
• Promoted to front page
• Sticky at top of lists
• Create new revision

should be a boolean, but actually it saving values as a string, code snippet is as below,

      status: status
      promote: promote
      sticky: sticky
      revision: revision

I have update patch from comment number 1, which changes the type from boolean to string.

Attached are updated patch and the short summary what I have updated.

Thanks for working on this @rahul.shinde. Can see that you have created separate issue #2031131: Configuration saving issue with "Content type settings". for the format issue. IMHO, we need to push this as it is blocking other follow ups in config_translation module (e.g. #2029407: Add support for node types )

+++ b/core/modules/node/config/schema/node.schema.ymlundefined
@@ -7,3 +7,68 @@ node.settings:
+    status:
+      type: boolean
+      label: 'Status'

This is an enabled/disabled status on the (config) entity.

See
ConfigEntityListController

and
ConfigEntityBase

  /**
   * The enabled/disabled status of the configuration entity.
   *
   * @var bool
   */
  public $status = TRUE;

re #3 @johnheaven

Comment Settings and Menu Settings are completely missing - is that correct?

menu links, are content, and not configuration... so that is stored in the database.

wait, that's the default, that would be used when a actual article was created...

so, where is that stored?

is it like language.settings.yml?

$ cat menu.settings.yml 
main_links: main
secondary_links: account
override_parent_selector: '0'

nah, that would say something about article

So, I looked at this closely.

first is some screenshots and notes so that I can remember what I was thinking later.
Patch attached for changes that I think need to be made.

Notes

s01

s02

+++ b/core/modules/node/config/schema/node.schema.ymlundefined
@@ -7,3 +7,68 @@ node.settings:
+    help:
+      type: text
+      label: 'Explanation or submission guidelines'

label is fine,
but why is the config thing called "help"?
Maybe we used to call this help text.
Not the problem of this issue.

preview s03

preview s04 disabled

preview s05 required

So.. it's not boolean. And is being saved with numbers 0, 1, 2 to represent the select I think.
Changed this to be integer.

but feels weird, maybe should be saving as string: optional, disabled, required? (not the problem of this issue.)

s06 status (show submitted by status)

Changed the label in the schema to better match the label in the UI.

again, weird naming in saving the config, submitted sounds like is it submitted or not, not if it is displaying the submitted by authoring info. (not the problem of this issue)

s07 just noting that the language settings are saved elsewhere, language.settings.yml. ok.

Changes

uid:
to
uuid:

preview type to int

better label on submitted

Also,

better label on status, the config entity status. See #6 (it's the purple note in the s01 overview screenshot).
A concern though, is that this is "status" in many config entities, so if this is a good idea, should change the label in those other schemas also. (would be a separate issue to change the other places)

--

patch coming.

patch for the changes from #8.

Thanks for working on this @YesCT. Improvements in #9 looks good.

one minor note:

+++ b/core/modules/node/config/schema/node.schema.ymlundefined
@@ -41,7 +41,7 @@ node.type.*:
-      label: 'Status'
+      label: 'Enabled status of the configuration entity'

it doesn't sound right. Status is the flag to say whether it is enabled/disabled. We can rename it something, but this property is there in all config and entityConfig. So we might need to all?

yeah, I was just being persnickety.
We dont want to change it everywhere as part of this patch, and it's really not our problem here what it's named.

I put it back.
So I think this should be ok now.

Committed dc2304f and pushed to 8.x. Thanks!