diff options
| author | Daniel Pittman <daniel@puppetlabs.com> | 2011-04-18 13:22:35 -0700 |
|---|---|---|
| committer | Daniel Pittman <daniel@puppetlabs.com> | 2011-04-19 10:53:26 -0700 |
| commit | 36b100a9cffdfcfe6cb661e0f41bb5cb05b19adf (patch) | |
| tree | 363e57f1d40b85966c09e395bb6c0b3e52f5bc62 /lib/puppet/application | |
| parent | 2cf692c878741fa92c18dcf2062b2a0ba96e3ca8 (diff) | |
| download | puppet-36b100a9cffdfcfe6cb661e0f41bb5cb05b19adf.tar.gz puppet-36b100a9cffdfcfe6cb661e0f41bb5cb05b19adf.tar.xz puppet-36b100a9cffdfcfe6cb661e0f41bb5cb05b19adf.zip | |
maint: print 'false' in the default render method.
We used to skip printing if the return value from an action was false; this
made it impossible to display "false" to the user, which was a more meaningful
output in some cases than nothing.
maint: print 'false' in the default render method.
We used to skip printing if the return value from an action was false; this
made it impossible to display "false" to the user, which was a more meaningful
output in some cases than nothing.
Reviewed-By: Max Martin <max@puppetlabs.com>
Diffstat (limited to 'lib/puppet/application')
| -rw-r--r-- | lib/puppet/application/face_base.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/puppet/application/face_base.rb b/lib/puppet/application/face_base.rb index df828b5a2..7b5bffe54 100644 --- a/lib/puppet/application/face_base.rb +++ b/lib/puppet/application/face_base.rb @@ -151,7 +151,7 @@ class Puppet::Application::FaceBase < Puppet::Application # Call the method associated with the provided action (e.g., 'find'). if @action result = @face.send(@action.name, *arguments) - puts render(result) if result + puts render(result) unless result.nil? else if arguments.first.is_a? Hash puts "#{@face} does not have a default action" |