Be more prescriptive on super usage to make Ruby 2.6 happy
This commit is contained in:
parent
35277eff21
commit
2a4eba8435
@ -13,10 +13,13 @@ module VagrantPlugins
|
||||
raise ArgumentError,
|
||||
"Expected `Broker' to be provided" if @broker.nil?
|
||||
|
||||
if self.method(:initialize).super_method.parameters.empty?
|
||||
sup = self.method(:initialize).super_method
|
||||
if sup.parameters.empty?
|
||||
super()
|
||||
else
|
||||
elsif !opts.empty? && sup.parameters.detect{ |type, _| type == :keyreq || type == :keyrest }
|
||||
super
|
||||
else
|
||||
super(*args, &block)
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
@ -11,10 +11,13 @@ module VagrantPlugins
|
||||
def initialize(*args, **opts, &block)
|
||||
@logger = Log4r::Logger.new(self.class.name.downcase)
|
||||
|
||||
if self.method(:initialize).super_method.parameters.empty?
|
||||
sup = self.method(:initialize).super_method
|
||||
if sup.parameters.empty?
|
||||
super()
|
||||
else
|
||||
elsif !opts.empty? && sup.parameters.detect{ |type, _| type == :keyreq || type == :keyrest }
|
||||
super
|
||||
else
|
||||
super(*args, &block)
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
@ -16,10 +16,13 @@ module VagrantPlugins
|
||||
@mapper.add_argument(broker)
|
||||
end
|
||||
|
||||
if self.method(:initialize).super_method.parameters.empty?
|
||||
sup = self.method(:initialize).super_method
|
||||
if sup.parameters.empty?
|
||||
super()
|
||||
else
|
||||
elsif !opts.empty? && sup.parameters.detect{ |type, _| type == :keyreq || type == :keyrest }
|
||||
super
|
||||
else
|
||||
super(*args, &block)
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user