This is a sub-issue of #1910606: Improve the configurations schemas for Views significantly.
Problem/motivation
#1866610: Introduce Kwalify-inspired schema format for configuration introduced some config schema coverage for views and #1910606: Improve the configurations schemas for Views significantly extended it, but it is not complete. 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
Figure out the missing pieces that are not yet covered. Write schema file sections for them. Clean up / fix any issues in current schema.
Create a configuration schema for missing view fields plugins
Schema in place
Please refer core/modules/views/config/schema/views.field.schema.yml
Boolean.php
Broken.php
Counter.php
Custom.php
Date.php
FileSize.php
Links.php
MachineName.php
Numeric.php
PrerenderList.php
Serialized.php
Standard.php
TimeInterval.php
Url.php
Schema not yet in place
Dropbutton.php
Markup.php
Xss.php
Steps to review
Check Steps to check on meta.
Comment | File | Size | Author |
---|---|---|---|
#10 | 2019079-diff-3-10.txt | 1.17 KB | vijaycs85 |
#10 | 2019079-config-schema-view-field-10.patch | 1.11 KB | vijaycs85 |
#3 | 2019079-config-schema-views-fields-2.patch | 588 bytes | Nitesh Sethia |
#1 | 2019079-config-schema-views-fields-1.patch | 0 bytes | Nitesh Sethia |
Comments
Comment #1
Nitesh Sethia CreditAttribution: Nitesh Sethia commentedIssuing initial patch for remaining field types...
Comment #2
Nitesh Sethia CreditAttribution: Nitesh Sethia commentedIssuing initial patch for remaining field types...
Comment #3
Nitesh Sethia CreditAttribution: Nitesh Sethia commentedIssuing initial patch for remaining field types...
Comment #4.0
vijaycs85Updated issue summary.
Comment #5
neetu morwani CreditAttribution: neetu morwani commentedI went through Dropbutton.php , Markup.php , Xss.php files. None of the options is defined in defineOptions() function. Because there are no options, mapping for these could not be added in views.field.schema.yml file. I guess,Schema for the Dropbutton, Markup, Xss is in place. We can even add Dropbutton field in the view.
Comment #6
vijaycs853: 2019079-config-schema-views-fields-2.patch queued for re-testing.
Comment #9
vijaycs853: 2019079-config-schema-views-fields-2.patch queued for re-testing.
Comment #10
vijaycs85Updating more field plugins
Comment #11
vijaycs85Updating tags...
Comment #12
dawehnerThis is two time the same, this cannot be right. additional this extends the links class which actually has some configuration we should inherit from.
Comment #13
vijaycs85The 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.