Rails Plugins

The exception_notifiable plugin should be updated to use the latest rails convention for rescuing exceptions. That convention being to define the handling of exceptions with rescue_from handlers. As opposed to the exception_notification convention of defining render_404 and render_500.

I propose the plugin should define 'exception_notify' as a corollary to the 'rescue_from' method.

And should override (alias_method_chain on) rescue_action_with_handler to notify of an exception that we care about.

Example:

old style was:

  include ExceptionNotifiable
  
  def render_500
    render :text => "500", :status => 500
  end

  def render_404
    render :text => "404", :status => 404
  end

proposed new style:

  include ExceptionNotifiable
    
  rescue_from ActiveRecord::RecordNotFound do |exception|
    render :text => "404", :status => 404
  end

  rescue_from RuntimeError do |exception|
    render :text => "500", :status => 500
  end
  
  exception_notify RuntimeError

(old style will still be supported)