Follow-up from #1653026: [META] Use properly typed values in module configuration.

Problem/Motivation

All integers, Booleans, and even octal numbers in config object files are converted to strings.

Proposed resolution

#1653026: [META] Use properly typed values in module configuration has fixed core, so no need to convert all data types to string anymore.

Remaining tasks

entity.view_mode.taxonomy_term.full.yml
taxonomy.settings.yml
views.view.test_filter_taxonomy_index_tid.yml
views.view.test_groupwise_term.yml
views.view.test_taxonomy_node_term_data.yml

User interface changes

NO

API changes

NO

Parent: #1653026: [META] Use properly typed values in module configuration

Files: 
CommentFileSizeAuthor
#3 2105997-taxonomy-config-cleanup-3.patch9.49 KBvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 59,406 pass(es).
[ View ]

Comments

danilenko_dn’s picture

Assigned:Unassigned» danilenko_dn
Status:Active» Needs review
StatusFileSize
new10.55 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 2105997-1-taxonomy-yaml-variable-type.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Huw about items_per_page_options: '5, 10, 20, 40, 60' ?

I think it's needed as string, because arrays should looks like:
items_per_page_options: [5, 10, 20, 40, 60]

The last submitted patch, 2105997-1-taxonomy-yaml-variable-type.patch, failed testing.

vijaycs85’s picture

Issue summary:View changes
Status:Needs work» Needs review
StatusFileSize
new9.49 KB
PASSED: [[SimpleTest]]: [MySQL] 59,406 pass(es).
[ View ]

Re-rolling...

vijaycs85’s picture

Status:Needs review» Closed (duplicate)
Related issues:+#2167623: Add test for all default configuration to ensure schema exists and is correct

The 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.