mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
AS::Callbacks: improve __define_runner perfomance
This commit is contained in:
parent
49f08bb50c
commit
66a587cf55
1 changed files with 11 additions and 10 deletions
|
@ -378,18 +378,15 @@ module ActiveSupport
|
|||
module ClassMethods
|
||||
# Generate the internal runner method called by +run_callbacks+.
|
||||
def __define_runner(symbol) #:nodoc:
|
||||
body = send("_#{symbol}_callbacks").compile
|
||||
runner_method = "_run_#{symbol}_callbacks"
|
||||
name = __callback_runner_name(nil, symbol)
|
||||
undef_method(name) if method_defined?(name)
|
||||
|
||||
silence_warnings do
|
||||
runner_method = "_run_#{symbol}_callbacks"
|
||||
undef_method runner_method if method_defined?(runner_method)
|
||||
class_eval <<-RUBY_EVAL, __FILE__, __LINE__ + 1
|
||||
def #{runner_method}(key = nil, &blk)
|
||||
if key
|
||||
self.class.__run_keyed_callback(key, :#{symbol}, self, &blk)
|
||||
else
|
||||
#{body}
|
||||
end
|
||||
self.class.__run_callback(key, :#{symbol}, self, &blk)
|
||||
end
|
||||
private :#{runner_method}
|
||||
RUBY_EVAL
|
||||
|
@ -400,10 +397,10 @@ module ActiveSupport
|
|||
# If this called first time it creates a new callback method for the key,
|
||||
# calculating which callbacks can be omitted because of per_key conditions.
|
||||
#
|
||||
def __run_keyed_callback(key, kind, object, &blk) #:nodoc:
|
||||
name = "_run__#{self.name.hash.abs}__#{kind}__#{key.hash.abs}__callbacks"
|
||||
def __run_callback(key, kind, object, &blk) #:nodoc:
|
||||
name = __callback_runner_name(key, kind)
|
||||
unless object.respond_to?(name)
|
||||
str = send("_#{kind}_callbacks").compile(name, object)
|
||||
str = send("_#{kind}_callbacks").compile(key, object)
|
||||
class_eval <<-RUBY_EVAL, __FILE__, __LINE__ + 1
|
||||
def #{name}() #{str} end
|
||||
protected :#{name}
|
||||
|
@ -412,6 +409,10 @@ module ActiveSupport
|
|||
object.send(name, &blk)
|
||||
end
|
||||
|
||||
def __callback_runner_name(key, kind)
|
||||
"_run__#{self.name.hash.abs}__#{kind}__#{key.hash.abs}__callbacks"
|
||||
end
|
||||
|
||||
# This is used internally to append, prepend and skip callbacks to the
|
||||
# CallbackChain.
|
||||
#
|
||||
|
|
Loading…
Reference in a new issue