diff options
| author | Morgan Haskel <morgan@puppetlabs.com> | 2015-09-21 10:56:08 -0700 |
|---|---|---|
| committer | Morgan Haskel <morgan@puppetlabs.com> | 2015-09-21 11:11:21 -0700 |
| commit | 799c38e14e1583e676e2b25a9c1782fd40e29fff (patch) | |
| tree | ca2f93dd6d459eac114f1e0b5cac05ac02d519cd /lib/puppet/parser/functions/parseyaml.rb | |
| parent | 9b1932c538354c1b360838c8cf7b942af314c99d (diff) | |
| download | puppet-stdlib-799c38e14e1583e676e2b25a9c1782fd40e29fff.tar.gz puppet-stdlib-799c38e14e1583e676e2b25a9c1782fd40e29fff.tar.bz2 | |
Fix backwards compatibility from #511
Maintain the old behavior in the case where the optional second
parameter isn't passed. Also, adding arity is backwards incompatible since
stdlib still supports 2.7, so remove that.
Diffstat (limited to 'lib/puppet/parser/functions/parseyaml.rb')
| -rw-r--r-- | lib/puppet/parser/functions/parseyaml.rb | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/lib/puppet/parser/functions/parseyaml.rb b/lib/puppet/parser/functions/parseyaml.rb index d38b3ef..66d0413 100644 --- a/lib/puppet/parser/functions/parseyaml.rb +++ b/lib/puppet/parser/functions/parseyaml.rb @@ -3,7 +3,7 @@ # module Puppet::Parser::Functions - newfunction(:parseyaml, :type => :rvalue, :arity => -2, :doc => <<-EOS + newfunction(:parseyaml, :type => :rvalue, :doc => <<-EOS This function accepts YAML as a string and converts it into the correct Puppet structure. @@ -16,8 +16,12 @@ be returned if the parsing of YAML string have failed. begin YAML::load(arguments[0]) || arguments[1] - rescue Exception - arguments[1] + rescue Exception => e + if arguments[1] + arguments[1] + else + raise e + end end end |