Skip to content

Commit

Permalink
dont call downcase() on undef variable
Browse files Browse the repository at this point in the history
This got probably introduced during some debugging. The variable is
always undef. This works on Puppet 4 and 5 because downcase() works with
undef by accident. This got fixed in the latest puppet 6. It now results
in a compile error.
  • Loading branch information
bastelfreak committed Sep 15, 2018
1 parent aaeb555 commit 865c27a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion manifests/extension.pp
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@
$_settings.each |$settings_name, $settings_hash| {
$so_name = $multifile_settings ? {
true => downcase($settings_name),
false => pick(downcase($so_name), downcase($name), downcase($settings_name)),
false => pick(downcase($name), downcase($settings_name)),
}

php::extension::config { $settings_name:
Expand Down

0 comments on commit 865c27a

Please sign in to comment.