search

example42 / puppi

Puppet module to manage applications deployments and servers local management
http://www.example42.com
Other
142 stars 84 forks source link

params_lookup future parser fix #124

Closed claytono closed 9 years ago

claytono commented 9 years ago

With future parser, the empty string is now true, not false. Many modules using params_lookup assume that it will return a false value if it is unsuccessful. This changes params_lookup to return undef instead of an empty string when no value could be found.

ChrisWiegman commented 9 years ago

This has broken existing example42 modules including APT, Apache and PHP (I haven't tested others) as they are expecting the empty string rather than the null return.

alvagante commented 9 years ago

Sorry, had to test this better :-/