2013-03-04 1 views
5

Meine Flash-Nachrichten erscheinen zweimal und meine Web-Forschung sagt mir, dies wird aufgrund der Render-und Redirect-Anzeige der Nachrichten. Ich glaube, ich brauche zu verwenden flash.now [] oder flash [] irgendwo diese zu sortieren, aber ich kann nicht herausfinden, wo esFlash-Meldungen erscheinen zweimal in Schienen

guidelines_controller.rb

def update 
    @guideline = Guideline.find(params[:id]) 

    respond_to do |format| 
    if @guideline.update_attributes(params[:guideline]) 
     @guideline.update_attribute(:updated_by, current_user.id) 
     format.html { redirect_to @guideline, notice: 'Guideline was successfully updated.' } 
     format.json { head :no_content } 
    else 
     format.html { render action: "show" } 
     format.json { render json: @guideline.errors, status: :unprocessable_entity } 
    end 
    end 
end 

Layouts/application.html gehen muss .erb

<div class="container"> 

    <% flash.each do |type, message| %> 

     <div class="alert <%= flash_class type %>"> 
      <button class="close" data-dismiss="alert">x</button> 
      <%= message %> 
     </div> 
    <% end %> 
</div> 

application_helper.rb 

def flash_class(type) 
    case type 
    when :alert 
    "alert-error" 
    when :notice 
    "alert-success" 
    else 
    "" 
    end 
end 

guideline_controller.rb

def show 
    @guideline = Guideline.find(params[:id]) 
    if @guideline.updated_by 
    @updated = User.find(@guideline.updated_by).profile_name 
    end 

     if User.find(@guideline.user_id) 
    @created = User.find(@guideline.user_id).profile_name 
     end 

    respond_to do |format| 
     format.html # show.html.erb 
     format.json { render json: @guideline } 

    end 
    end 
+0

versuchen Festsetzung, dass '' <% end %> Anruf dort vor dem 'div '. Ich sehe kein Problem mit Ihrem Controller-Code. Ich glaube auch nicht, dass Sie "flash.discard" verwenden müssen – jvnill

+0

haben wie oben bearbeitet (ist das, was Sie gemeint?), Aber das gleiche Problem passiert. – tessad

+0

nein. Sie müssen das Div zuerst vor der Schleife schließen. versuchen Sie, Ihren Code richtig einzurücken, damit er mit öffnenden und schließenden Tags übereinstimmt/code – jvnill

Antwort

1

Sie können, um etwas tun zu speichern einige Zeilen Code, und zeigen Sie die Nachrichten nur einmal:

<%- if flash.any? %> 
    <%- flash.keys.each do |flash_key| %> 
    <%- next if flash_key.to_s == 'timedout' %> 
    <div class="alert-message <%= flash_key %>"> 
     <a class="close" data-dismiss="alert" href="#"> x</a> 
     <%= flash.discard(flash_key) %> 
    </div> 
    <%- end %> 
<%- end %> 

von flash.discard verwenden, zeigen Sie die Flash-Nachricht eine avoid Rendering zweimal

+0

danke - Ich habe das zu meinem Anwendungslayout hinzugefügt (anstelle dessen, was oben ist). Jetzt wird es immer noch zweimal angezeigt, aber ohne den Bootstrap-Styline (der nur auf einem von ihnen war) ... irgendwelche Ideen, warum das passieren könnte? – tessad

+0

Es ist ärgerlich, dass sogar die Flash-Nachricht verwerfen, Sie erhalten den Blitz zweimal angezeigt. Was passiert, wenn Sie vor dem Anzeigen des Flash ein <% = Debug-Flash%> setzen, um zu sehen, ob Sie Flash-Nachrichten dupliziert haben? Das könnte das Problem sein. – rorra

+0

wohin würde das gehen? – tessad