2013-03-04 15 views
5

Moje wiadomości flash pojawiają się dwa razy, a moje badania sieci mówią mi, że jest to spowodowane renderowaniem i przekierowaniem wyświetlania wiadomości. Myślę, że trzeba użyć flash.now [] lub błysku [] gdzieś to rozwiązać, ale nie mogę pracować tam, gdzie trzeba iśćWiadomości flash pojawiające się dwa razy w szynach

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

spróbować ustalające, że '' <% end %> wywołanie tam przed 'div '. Nie widzę żadnego problemu z kodem kontrolera. Nie sądzę też, abyś użył 'flash.discard' – jvnill

+0

edytowanego jak powyżej (czy to masz na myśli?), Ale ten sam problem się dzieje. – tessad

+0

nope. musisz zamknąć div najpierw przed pętlą. spróbuj poprawnie wstawić swój kod, aby pasował do otwartych i zamykających tagów/kodu – jvnill

Odpowiedz

1

można zrobić coś takiego, żeby zaoszczędzić kilka linijek kodu i wyświetlanie wiadomości tylko raz:

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

Korzystając flash.discard, pokażesz Wiad to avoid renderowania dwukrotnie

+0

dzięki - dodałem to do mojego layoutu aplikacji (zamiast tego, co jest powyżej). Teraz nadal wyświetla się dwa razy, ale bez stylu startowego (który był tylko na jednym z nich) ... żadnych pomysłów, dlaczego tak się dzieje? – tessad

+0

To irytujące, że nawet odrzucając wiadomość flash, otrzymujesz flash wyświetlany dwukrotnie. Co się stanie, jeśli umieścisz <% = debug flash%> przed wyświetleniem lampy błyskowej, aby sprawdzić, czy masz powielone wiadomości flash? To może być problem. – rorra

+0

gdzie by to poszło? – tessad

Powiązane problemy