From 36b100a9cffdfcfe6cb661e0f41bb5cb05b19adf Mon Sep 17 00:00:00 2001 From: Daniel Pittman Date: Mon, 18 Apr 2011 13:22:35 -0700 Subject: 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 --- lib/puppet/application/face_base.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/puppet') 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" -- cgit