This project is archived and is in readonly mode.

Changeset [75ba102a80965b2612df0253d1278581a88b8d66] 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...

Committed by José Valim

  • 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

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>