Modifying BLT Configuration definition is not accurate

[uenrico]

My system information:

• Operating system type: Windows10 (host), DrupalVM (guest)
• Operating system version: Ubuntu 16.04.3 LTS
• BLT version: 8.9.2

BLT Configuration definition said "BLT configuration can be customized by overriding the value of default variable values... Values loaded from the later files will overwrite values in earlier files."

That definition is not accurate because when project.yml and project.local.yml are loaded an array_merge is executed (arrayMergeRecursiveDistinct) instead of a real array override.

project.yml (partial)

modules:
    local:
        enable: [dblog, devel, seckit, views_ui]
        uninstall: [acsf, acquia_connector, shield]

project.local.yml (partial)

modules:
    local:
        enable: [dblog, devel, seckit, views_ui]
        uninstall: {  }

I get the following output:

modules:
    local:
        enable: [dblog, devel, seckit, views_ui]
        uninstall: [acsf, acquia_connector, shield]

And I expected this to happen:
modules.local.uninstall should be empty if we are expecting an override,

[grasmash]

You're right, the array value must return true for isset() in order for the override to take effect. Have you tried setting the value to null ?

https://github.com/acquia/blt/blob/8.x/src/Robo/Common/ArrayManipulator.php#L31

[uenrico]

uninstall: null works for me

[grasmash]

Documentation has been updated.

[uenrico]

Checking logs I got a php warning msg using uninstall: null when executing blt setup

PHP Warning: implode(): Invalid arguments passed in /vendor/acquia/blt/src/Robo/Commands/Setup/ToggleModulesCommand.php on line 70

Warning: implode(): Invalid arguments passed in /vendor/acquia/blt/src/Robo/Commands/Setup/ToggleModulesCommand.php on line 70