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

locale.config.it.tour.tour.tour-test.yml
tour.tour.tour-test-2.yml
tour.tour.tour-test.yml

User interface changes

NO

API changes

NO

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

Files: 
CommentFileSizeAuthor
#6 2106005-tour-config-cleanup-6.patch3.01 KBvijaycs85
PASSED: [[SimpleTest]]: [MySQL] 59,419 pass(es).
[ View ]

Comments

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

Status:Needs review» Needs work

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

Status:Needs work» Needs review
StatusFileSize
new2.48 KB
FAILED: [[SimpleTest]]: [MySQL] 59,318 pass(es), 1 fail(s), and 0 exception(s).
[ View ]

Boolean true should be lowercase

The last submitted patch, 2106005-3-yaml-variable-type.patch, failed testing.

Assigned:danilenko_dn» nonsie
Issue summary:View changes
Status:Needs work» Needs review
StatusFileSize
new1.65 KB
PASSED: [[SimpleTest]]: [MySQL] 59,397 pass(es).
[ View ]

Looks like the patch failed because _access is still true as string not true as boolean

Title:Make sure all YML files in Tour module has no type-casting to string.Make sure all YML files in Tour module has no type-casting to string + update stings with quotes
Issue summary:View changes
StatusFileSize
new3.01 KB
PASSED: [[SimpleTest]]: [MySQL] 59,419 pass(es).
[ View ]

Fix the label and body text as well...

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.