diff options
author | Morgan Haskel <morgan@puppetlabs.com> | 2014-10-09 13:33:06 -0400 |
---|---|---|
committer | Morgan Haskel <morgan@puppetlabs.com> | 2014-10-09 13:33:06 -0400 |
commit | 0c0f7e13cdb407ca57d2808de9e6dc6ace9d856b (patch) | |
tree | 44ecfd905094e549eaed7b25d57c32ef840747c4 /lib/puppet/parser/functions/zip.rb | |
parent | 9bbc62d12ab665115b74920cdfbe55205786f328 (diff) | |
parent | 23bc7d51bd3aca0c3a3391bb628a797dc768422a (diff) |
Merge pull request #262 from tremble/MODULES-707
Modules 707
Diffstat (limited to 'lib/puppet/parser/functions/zip.rb')
-rw-r--r-- | lib/puppet/parser/functions/zip.rb | 28 |
1 files changed, 1 insertions, 27 deletions
diff --git a/lib/puppet/parser/functions/zip.rb b/lib/puppet/parser/functions/zip.rb index 2b56e9c..3074f28 100644 --- a/lib/puppet/parser/functions/zip.rb +++ b/lib/puppet/parser/functions/zip.rb @@ -27,33 +27,7 @@ Would result in: raise(Puppet::ParseError, 'zip(): Requires array to work with') end - flatten = arguments[2] if arguments[2] - - if flatten - klass = flatten.class - - # We can have either true or false, or string which resembles boolean ... - unless [FalseClass, TrueClass, String].include?(klass) - raise(Puppet::ParseError, 'zip(): Requires either ' + - 'boolean or string to work with') - end - - if flatten.is_a?(String) - # We consider all the yes, no, y, n and so on too ... - flatten = case flatten - # - # This is how undef looks like in Puppet ... - # We yield false in this case. - # - when /^$/, '' then false # Empty string will be false ... - when /^(1|t|y|true|yes)$/ then true - when /^(0|f|n|false|no)$/ then false - when /^(undef|undefined)$/ then false # This is not likely to happen ... - else - raise(Puppet::ParseError, 'zip(): Unknown type of boolean given') - end - end - end + flatten = function_str2bool([arguments[2]]) if arguments[2] result = a.zip(b) result = flatten ? result.flatten : result |