diff options
author | Markus Roberts <Markus@reality.com> | 2010-07-09 18:06:37 -0700 |
---|---|---|
committer | Markus Roberts <Markus@reality.com> | 2010-07-09 18:06:37 -0700 |
commit | 07b15bf6fa2a2183f73fcb9b6740c7df75c8b47b (patch) | |
tree | f4f20be112f0c8832cb2e9681935a04bb0d197cf /lib/puppet/property.rb | |
parent | 8d1fbe4586c91682cdda0cb271649e918fd9778b (diff) | |
download | puppet-07b15bf6fa2a2183f73fcb9b6740c7df75c8b47b.tar.gz puppet-07b15bf6fa2a2183f73fcb9b6740c7df75c8b47b.tar.xz puppet-07b15bf6fa2a2183f73fcb9b6740c7df75c8b47b.zip |
Code smell: Avoid unneeded blocks
Replaced 45 occurances of
(DEF)
begin
(LINES)
rescue(.*)
(LINES)
end
end
with
3 Examples:
The code:
def find(name)
begin
self.const_get(name.to_s.capitalize)
rescue
puts "Unable to find application '#{name.to_s}'."
Kernel::exit(1)
end
end
becomes:
def find(name)
self.const_get(name.to_s.capitalize)
rescue
puts "Unable to find application '#{name.to_s}'."
Kernel::exit(1)
end
The code:
def exit_on_fail(message, code = 1)
begin
yield
rescue RuntimeError, NotImplementedError => detail
puts detail.backtrace if Puppet[:trace]
$stderr.puts "Could not #{message}: #{detail}"
exit(code)
end
end
becomes:
def exit_on_fail(message, code = 1)
yield
rescue RuntimeError, NotImplementedError => detail
puts detail.backtrace if Puppet[:trace]
$stderr.puts "Could not #{message}: #{detail}"
exit(code)
end
The code:
def start
begin
case ssl
when :tls
@connection = LDAP::SSLConn.new(host, port, true)
when true
@connection = LDAP::SSLConn.new(host, port)
else
@connection = LDAP::Conn.new(host, port)
end
@connection.set_option(LDAP::LDAP_OPT_PROTOCOL_VERSION, 3)
@connection.set_option(LDAP::LDAP_OPT_REFERRALS, LDAP::LDAP_OPT_ON)
@connection.simple_bind(user, password)
rescue => detail
raise Puppet::Error, "Could not connect to LDAP: #{detail}"
end
end
becomes:
def start
case ssl
when :tls
@connection = LDAP::SSLConn.new(host, port, true)
when true
@connection = LDAP::SSLConn.new(host, port)
else
@connection = LDAP::Conn.new(host, port)
end
@connection.set_option(LDAP::LDAP_OPT_PROTOCOL_VERSION, 3)
@connection.set_option(LDAP::LDAP_OPT_REFERRALS, LDAP::LDAP_OPT_ON)
@connection.simple_bind(user, password)
rescue => detail
raise Puppet::Error, "Could not connect to LDAP: #{detail}"
end
Diffstat (limited to 'lib/puppet/property.rb')
-rw-r--r-- | lib/puppet/property.rb | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/lib/puppet/property.rb b/lib/puppet/property.rb index ec700fbdf..0ded4c557 100644 --- a/lib/puppet/property.rb +++ b/lib/puppet/property.rb @@ -69,11 +69,9 @@ class Puppet::Property < Puppet::Parameter # Call the provider method. def call_provider(value) - begin provider.send(self.class.name.to_s + "=", value) - rescue NoMethodError + rescue NoMethodError self.fail "The #{provider.class.name} provider can not handle attribute #{self.class.name}" - end end # Call the dynamically-created method associated with our value, if |