Make sure all YML files in Action module has no type-casting to string.

+++ b/core/modules/action/tests/action_bulk_test/config/views.view.test_bulk_form.ymlundefined
@@ -22,35 +22,35 @@ display:
+          override: TRUE

In other patches you use lowercase for boolean statements.

Which should they be uppercase or lowercase. Pick one.

Uppercase.
EDIT: chx's point below trumps all other conventions.

lowercase. Bring the issue up with the symfony yaml guys. It dumps lowercase.

'0' => false, '1' => true. This is only for: config/views.view.test_bulk_form.yml. Will work thru the rest, if this is OK.

Please provide review feedback on the patch above, and i'll script through the rest of the tree -- thx.

Yes this looks good! Careful with scripting. Perhaps share the script with us? You can't wholesale search-replace 1 and 0 to true and false, sometimes they are numbers.

Script attached:

#!/bin/sed -f
# convert 0 to false and '1' to true, and also lower-case TRUE/FALSE
# Usage: boolconvert.sed <yml-file-name>

# searching for :<space><tab>(whole-word) '1' or '0'
# whole-word reason: we don't want to mess with '-1', 100, etc
# Bug: "token: 1 black sheep" will become "token: true black sheep"

s/:[   ]*'\<1\>'/: true/
s/:[   ]*'\<0\>'/: false/
s/:[    ]*\<0\>/: false/
s/:[    ]*\<1\>/: true/
s/:[    ]*FALSE/: false/
s/:[    ]*TRUE/: true/

Also, how best to walk through directory tree to pick up relevant yml files? This is the rough version:

# Pick up config files in ./modules/<module-name>/config/*.yml
#
find .  -name '*.yml' | grep "\/modules\/\<[a-z]*\>\/config\/.*yml\>"  | xargs sed -i -f ../boolconvert.sed

Better to do the above by hand?

+++ b/core/modules/action/tests/action_bulk_test/config/views.view.test_bulk_form.yml
@@ -81,57 +81,57 @@ display:
-          group: '1'
...
+          group: true

Group should definitely be a number, this is the group used when the condition groups are built for the query. So 1 is the default group here.

updating views.view.test_bulk_form.yml