kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Only prepend a single module when defining deprecation wrappers.

Browse source 
I could not find any reason why each method got its own prepended
module here, and all tests appear to pass with my change.
This commit is contained in:
Charles Oliver Nutter 2015-10-13 11:16:49 -05:00
parent de732e0015
commit d29f7fbb10
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
activesupport/lib/active_support/deprecation/method_wrappers.rb
View file

@ -30,16 +30,16 @@ module ActiveSupport
deprecator = options.delete(:deprecator) || ActiveSupport::Deprecation.instance deprecator = options.delete(:deprecator) || ActiveSupport::Deprecation.instance
method_names += options.keys method_names += options.keys
method_names.each do |method_name| mod = Module.new do
mod = Module.new do method_names.each do |method_name|
define_method(method_name) do |*args, &block| define_method(method_name) do |*args, &block|
deprecator.deprecation_warning(method_name, options[method_name]) deprecator.deprecation_warning(method_name, options[method_name])
super(*args, &block) super(*args, &block)
end end
end end
target_module.prepend(mod)
end end
target_module.prepend(mod)
end end
end end
end end