Create configuration schemas for update module

updating schema as per code style in http://drupal.org/node/1905070#codestyle and verified in config_inspector

Are these the labels used on the forms where forms exist to edit these values? Eg. "Check for updates" looks odd for a day interval value. My initial reaction was if its an on/off switch it should be a boolean. Now seeing from the schema it is an interval. What about other labels?

I checked the module and updated the same label whichever available. Updated my own for remaining.

Assigning for review.

@Gabor, yeah that is how it is presented in the config forms. These are actually radio buttons which have labels that clarify the meaning:

  $form['update_check_frequency'] = array(
    '#type' => 'radios',
    '#title' => t('Check for updates'),
    '#default_value' => $config->get('check.interval_days'),
    '#options' => array(
      '1' => t('Daily'),
      '7' => t('Weekly'),
    ),
    '#description' => t('Select how frequently you want to automatically check for new releases of your currently installed modules and themes.'),
  );

Would it be better to use 'Update check frequency' here?

Got an answer from alexpott on #6. For the moment the best practice is to use the existing strings.

+ label: 'E-mail addresses to nofity when updates are available'
nofity -> notify

+    notification:
+      type: mapping
+      label: 'Notification settings'
+      mapping:
 ...
+        threshold:
+          type: integer
+          label: 'E-mail notification threshold'

notification.threshold should not be an integer but a string containing 'all' or 'security' according to the form:

  $form['update_notification_threshold'] = array(
    '#type' => 'radios',
    '#title' => t('E-mail notification threshold'),
    '#default_value' => $config->get('notification.threshold'),
    '#options' => array(
      'all' => t('All newer versions'),
      'security' => t('Only security updates'),
    ),
  );

+    notification:
+      type: mapping
+      label: 'Notification settings'
+      mapping:
+        emails:
+          type: sequence
+          label: 'E-mail addresses to nofity when updates are available'
+          sequence:
+            - type: string
+              label: 'E-mail'

This should be type 'email' instead of type 'string'.

small changed in label. patch is working fine for me.

...

this notice / error showed up in config-inspector..

I have reverted and again apply same patch. Its working fine for me.

@sandipmkhairnar my sincere apologies,
i had disabled the 'Update Manager' during the initial install (which is my normal workflow for a dev environment) and then re-enabled it to review your patch, this caused the site default email to somehow be excluded from the config (but that's obviously outside the scope of this issue -- i will raise that as a separate CMI tagged bug report)

your patch applies cleanly, i will follow up with a review..

taken from the coding style guide..

The schema items should have labels anyway, which should describe them well.

+++ b/core/modules/update/config/schema/update.schema.ymlundefined
@@ -0,0 +1,42 @@
+          label: 'Check for updates'

this label doesn't mention that it is how many "days" have passed since the last check.. it should specify number of days since last check

+++ b/core/modules/update/config/schema/update.schema.ymlundefined
@@ -0,0 +1,42 @@
+          label: 'Timeout'

Timeout? Timeout (ms) might be better, without an interval measurement being specified i don't know what this value represents. (i assume milliseconds but 5ms seems a ridiculously low threshold for a HTTP request.. is it 500ms?)

+++ b/core/modules/update/config/schema/update.schema.ymlundefined
@@ -0,0 +1,42 @@
+        threshold:
+          type: string
+          label: 'E-mail notification threshold'

this 'string' contains a zero.. is there a reason this needs to be a string? (to contain a threshold value?)

@joates Thanks for comment. Updated labels as per comment.

As per @pfrenssen comment in #8 notification.threshold should not be an integer but a string containing 'all' or 'security' according to the form.

+          label: 'Number of days since last check'

this is a good improvement for this label but i think it could be equally good as just 'Days since last check'

+          label: 'Timeout(ms)'

there should be a single space between like this.. 'Timeout (ms)'

Thanks @joates. Updated labels as per comment.

@sandipmkhairnar..

just a friendly tip, looks like you are having some problems with yr patch uploading and creating an interdiff workflow, i would suggest uploading a complete and working patch at this stage (so that it can pass the tests and be applied by a reviewer) -- the interdiff part is basically optional (although some argue it's "best practice") the documentation here states:

You should supply one whenever you update a significant patch in the issue queues

(i added the bold) i don't mean that your patch is insignificant in this context 'significant' means large and/or complex.

the .patch file is really whats needed :))

thanks for your great work on this issue sandipmkhairnar.

small change to 'Timeout' label..

core/modules/update/update.fetch.inc: $end = time() + config('update.settings')->get('fetch.timeout');

its actually seconds not milliseconds (ms)

'Timeout in seconds' seems to be the most appropriate label.. your thoughts ?

patch looks great to me !

thanks @vijaycs85 and @sandipmkhairnar for your work on this issue.

screenshot..

Committed to 8.x.