This project is archived and is in readonly mode.

Changeset [75ba102a80965b2612df0253d1278581a88b8d66] by José Valim

Revision 75ba102a80965b2612df0253d1278581a88b8d66, committed by José Valim · December 26th, 2009 @ 07:28 PM

Remove ActionView inline logging to ActiveSupport::Notifications and create ActionController::Base#log_event, so everything can be logged within one listener. Also expose log_process_action as a hook for different modules to include their own information during the action processing. This allow ActiveRecord to hook and any other ORM. Finally, this commit changes 'Processing' and 'Rendering' in logs to 'Processed' and 'Rendered' because at the point it's logged, everying already happened.
http://github.com/rails/rails/commit/75ba102a80965b2612df0253d12785...

20 changed files

A actionpack/lib/action_controller/metal/active_record_runtime.rb
R actionpack/lib/action_controller/metal/benchmarking.rb
R actionpack/test/controller/benchmark_test.rb
M actionpack/lib/abstract_controller/logger.rb
M actionpack/lib/action_controller/base.rb
M actionpack/lib/action_controller/caching.rb
M actionpack/lib/action_controller/caching/fragments.rb
M actionpack/lib/action_controller/metal/filter_parameter_logging.rb
M actionpack/lib/action_controller/metal/logger.rb
M actionpack/lib/action_controller/rails.rb
M actionpack/lib/action_view/render/rendering.rb
M actionpack/test/abstract_unit.rb
M actionpack/test/controller/base_test.rb
M actionpack/test/controller/caching_test.rb
M actionpack/test/controller/filter_params_test.rb
M actionpack/test/controller/logging_test.rb
M actionpack/test/controller/render_test.rb
M activerecord/lib/active_record/rails.rb
M activesupport/lib/active_support/notifications/instrumenter.rb
M activesupport/test/notifications_test.rb