Redirect_to flash

luislavena · March 7, 2012, 6:58pm

I am trying to understand why redirect_to only accepts :notice and
:alert

rails/rails/blob/ead93c5be5b0f1945b7d0302f1aae4685ee3f2fb/actionpack/lib/action_controller/metal/flash.rb#L12


      
          module Flash
            extend ActiveSupport::Concern
          
          
  included do
              delegate :flash, :to => :request
              delegate :alert, :notice, :to => "request.flash"
              helper_method :alert, :notice
            end
          
          
  protected
              def redirect_to(options = {}, response_status_and_flash = {}) #:doc:
                if alert = response_status_and_flash.delete(:alert)
                  flash[:alert] = alert
                end
          
          
      if notice = response_status_and_flash.delete(:notice)
                  flash[:notice] = notice
                end
          
          
      if other_flashes = response_status_and_flash.delete(:flash)
                  flash.update(other_flashes)

Is there any reason I may be missing to not allow something like
:success like this:

redirect_to suppliers_path, :success => ‘Supplier was successfully
created.’ }

Noel

noel · March 7, 2012, 7:57pm

I believe those two are just the most common.

Look at the rest of the code, though. It looks like you can specify
other
flash values like so:

redirect_to suppliers_path, :flash => { :success => ‘Supplier was
successfully created.’ }

noel · March 8, 2012, 2:44am

On Wed, Mar 7, 2012 at 10:56 AM, Tim S. [email protected] wrote:

I believe those two are just the most common.

Look at the rest of the code, though. It looks like you can specify other
flash values like so:

redirect_to suppliers_path, :flash => { :success =>‘Supplier was
successfully created.’}

Thanks Tim, I didn’t get that from the code.

When I used redirect_to I was expecting
redirect_to suppliers_path, :
would produce flash[:]

At least this is a good workaround for me.

Noel