Make sure all YML files in Standard profile has no type-casting to string. [#2106029]

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

block.block.bartik.breadcrumbs.yml
block.block.bartik.content.yml
block.block.bartik.footer.yml
block.block.bartik.help.yml
block.block.bartik.login.yml
block.block.bartik.powered.yml
block.block.bartik.search.yml
block.block.bartik.tools.yml
block.block.seven.breadcrumbs.yml
block.block.seven.content.yml
block.block.seven.help.yml
block.block.seven.login.yml
editor.editor.basic_html.yml
editor.editor.full_html.yml
entity.display.node.article.default.yml
entity.display.node.article.teaser.yml
entity.form_display.node.article.default.yml
field.field.node.field_image.yml
field.field.node.field_tags.yml
field.instance.node.article.field_image.yml
field.instance.node.article.field_tags.yml
filter.format.basic_html.yml
filter.format.full_html.yml
filter.format.restricted_html.yml
node.type.article.yml
node.type.page.yml
rdf.mapping.comment.node__comment.yml
rdf.mapping.node.article.yml
rdf.mapping.node.page.yml
rdf.mapping.taxonomy_term.tags.yml
system.cron.yml
taxonomy.vocabulary.tags.yml
user.role.administrator.yml

User interface changes

API changes

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

Comment	File	Size	Author
#7	2106029-remove-string-casting-7.patch	27.33 KB	Sivaji_Ganesh_Jojodae
#7
#3	remove-string-casting-2106029.patch	29.91 KB	Barrett
#3

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

mtift

he/him

English

Minnesota, USA

CreditAttribution: mtift commented 23 October 2013 at 22:12

Issue tags:

+Novice

Tagging

Comment #2

Barrett CreditAttribution: Barrett commented 29 October 2013 at 02:21

Assigned:

Unassigned

» Barrett

I'll take this one.

Comment #3

Barrett CreditAttribution: Barrett commented 31 October 2013 at 00:52

Status:

Active

» Needs review

File	Size
remove-string-casting-2106029.patch	29.91 KB

Comment #4

derheap CreditAttribution: derheap commented 14 November 2013 at 09:13

The Patch fails. Needs a Reroll.

Comment #5

derheap CreditAttribution: derheap commented 14 November 2013 at 09:15

Issue summary:	View changes
Issue tags:		+Needs reroll

Comment #6

derheap CreditAttribution: derheap commented 14 November 2013 at 09:19

Status:

Needs review

» Needs work

Comment #7

Sivaji_Ganesh_Jojodae CreditAttribution: Sivaji_Ganesh_Jojodae commented 15 November 2013 at 10:04

Status:

Needs work

» Needs review

File	Size
2106029-remove-string-casting-7.patch	27.33 KB

Patch rerolled.

@Barrett, sorry for interjecting. Lately realized that this issue is assigned to you.

Comment #8

vijaycs85

London, UK

CreditAttribution: vijaycs85 commented 16 November 2013 at 00:06

Status:

Needs review

» Needs work

+++ b/core/profiles/standard/config/block.block.bartik_breadcrumbs.yml
@@ -1,24 +1,24 @@
+status: 0
...
+  label_display: 0
...
+    visibility: 0

+++ b/core/profiles/standard/config/block.block.bartik_content.yml
@@ -1,24 +1,24 @@
+status: 1
...
+  label_display: 0
...
+    visibility: 0

+++ b/core/profiles/standard/config/block.block.bartik_help.yml
@@ -1,24 +1,24 @@
+status: 1
...
+  label_display: 0

+++ b/core/profiles/standard/config/block.block.bartik_login.yml
@@ -1,24 +1,24 @@
+status: 1

+++ b/core/profiles/standard/config/block.block.bartik_powered.yml
@@ -1,24 +1,24 @@
+status: 1

+++ b/core/profiles/standard/config/block.block.bartik_search.yml
@@ -1,8 +1,8 @@
+status: 1

@@ -10,15 +10,15 @@ settings:
+    visibility: 0

+++ b/core/profiles/standard/config/block.block.seven_breadcrumbs.yml
@@ -1,24 +1,24 @@
+status: 0
...
+  label_display: 0
...
+    visibility: 0

+++ b/core/profiles/standard/config/block.block.seven_content.yml
@@ -1,24 +1,24 @@
+status: 1
...
+  label_display: 0
...
+    visibility: 0

+++ b/core/profiles/standard/config/block.block.seven_help.yml
@@ -1,24 +1,24 @@
+  label_display: 0
...
+    visibility: 0

+++ b/core/profiles/standard/config/block.block.seven_login.yml
@@ -1,25 +1,25 @@
+    visibility: 0

+++ b/core/profiles/standard/config/field.field.node.field_image.yml
@@ -26,9 +26,9 @@ settings:
+locked: 0
...
+translatable: 0

+++ b/core/profiles/standard/config/field.field.node.field_tags.yml
@@ -4,15 +4,15 @@ name: field_tags
+active: 1
...
+locked: 0
...
+translatable: 0
...
so on...

All boolean values should be true/false instead of 0/1.

+++ b/core/profiles/standard/config/block.block.seven_help.yml
@@ -1,24 +1,24 @@
+  label: System Help

+++ b/core/profiles/standard/config/block.block.seven_login.yml
@@ -1,25 +1,25 @@
+  label: User login

+++ b/core/profiles/standard/config/field.instance.node.article.field_image.yml
@@ -5,18 +5,18 @@ field_uuid: 748beaea-5074-4ff2-b51c-28a643d37c3a
+  file_extensions: png gif jpg jpeg

+++ b/core/profiles/standard/config/field.instance.node.article.field_tags.yml
@@ -5,10 +5,10 @@ field_uuid: 60db47f4-54fb-4c86-a439-5769fbda4bd1
+description: Enter a comma-separated list of words to describe your content.

all multi-word strings should be in in single quotes. Seems most of them are correct at the moment.