This is to fix puppetdoc boolean parameters. Puppetdoc defers sending parameters to Puppet::Util::Setting, and in this case, boolean parameters are stored as a boolean value.
Signed-off-by: Brice Figureau <[email protected]> --- lib/puppet/util/settings.rb | 2 ++ spec/unit/util/settings.rb | 7 +++++++ 2 files changed, 9 insertions(+), 0 deletions(-) diff --git a/lib/puppet/util/settings.rb b/lib/puppet/util/settings.rb index 4fbf602..0129fe7 100644 --- a/lib/puppet/util/settings.rb +++ b/lib/puppet/util/settings.rb @@ -830,6 +830,8 @@ Generated on #{Time.now}. when /^false$/i; false when /^true$/i; true when /^\d+$/i; Integer(value) + when true; true + when false; false else value.gsub(/^["']|["']$/,'').sub(/\s+$/, '') end diff --git a/spec/unit/util/settings.rb b/spec/unit/util/settings.rb index c700c0c..5e9ab3a 100755 --- a/spec/unit/util/settings.rb +++ b/spec/unit/util/settings.rb @@ -104,6 +104,13 @@ describe Puppet::Util::Settings do @settings[:bool].should == true end + it "should consider a cli setting with a boolean as an argument to be a boolean" do + # Turn it off first + @settings[:bool] = false + @settings.handlearg("--bool", true) + @settings[:bool].should == true + end + it "should clear the cache when setting getopt-specific values" do @settings.setdefaults :mysection, :one => ["whah", "yay"], :two => ["$one yay", "bah"] @settings[:two].should == "whah yay" -- 1.6.0.2 --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en -~----------~----~----~----~------~----~------~--~---
