2013-03-04 14 views
5

I miei messaggi flash vengono visualizzati due volte e la mia ricerca sul Web mi informa che ciò è dovuto al rendering e al reindirizzamento della visualizzazione dei messaggi. Penso di aver bisogno di usare flash.now [] o il flash [] da qualche parte per risolvere la questione, ma non riesco a capire dove deve andareMessaggi flash visualizzati due volte nei binari

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 

layout/application.html .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

provare a fissare che '' <% end %> chiamata lì prima della 'div '. Non vedo alcun problema con il codice del controller. Inoltre, non penso che sia necessario usare 'flash.discard' – jvnill

+0

come modificato sopra (è questo che intendevi?) Ma lo stesso problema sta accadendo. – tessad

+0

no. è necessario chiudere il div prima del ciclo. prova a indentare il tuo codice correttamente in modo che corrisponda tag aperto/chiuso – jvnill

risposta

1

Si può fare qualcosa di simile, al fine di salvare alcune righe di codice, e visualizzare i messaggi solo una volta:

<%- 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 %> 

Utilizzando flash.discard, si mostra il messaggio flash di evitare di rendere il doppio

+0

grazie - l'ho aggiunto al mio layout di applicazione (invece di quello che è sopra). Ora continua a essere visualizzato due volte, ma senza la stencil di bootstrap (che era solo su uno di essi) ... qualche idea del perché ciò potrebbe accadere? – tessad

+0

È fastidioso che, anche scartando il messaggio flash, il flash venga visualizzato due volte. Cosa succede se metti un <% = debug flash%> prima di mostrare il flash, per vedere se hai dei messaggi flash duplicati? Questo potrebbe essere il problema. – rorra

+0

dove andrebbe? – tessad

Problemi correlati