0

Перестали работать flash сообщения, после того как добавил Ajax для создания и удаления комментария. В чем может быть дело?

    class CommentsController < ApplicationController

  def create
    @post = Post.find(params[:post_id])
    @comment = @post.comments.create(comment_params)
    @comment.user_id = current_user.id
    if @comment.save
      respond_to do |format|
        format.html do
          flash[:success] = "Your comment has been posted."
          redirect_to @post
        end
        format.js
      end
    else
      flash.now[:danger] = "error"
      respond_to do |format|
        format.html { render @post }
        format.js
      end
    end
  end

  def destroy
    @post = Post.find(params[:post_id])
    @comment = Comment.find(params[:id])
    @comment.destroy


    respond_to do |format|
      format.html do
        flash[:success] = "Comment deleted."
        redirect_to post_path(@post)
      end
      format.js
    end
  end


  private
    def comment_params
      params.require(:comment).permit(:user_id, :content)
    end
end

create.js.erb:

$('#comments-list').append('<%= escape_javascript(render @comment) %>');
$("#commentCount").html("Comments(<%= @post.comments.count %>)");

destroy.js.erb:

$("#comments-list").html("<%= escape_javascript(render @post.comments) %>");
$("#commentCount").html("Comments(<%= @post.comments.count %>)");

Весь код проекта: https://bitbucket.org/Tiazar/callboard

1 ответ 1

2

Ну так ты никак их не отдаёшь на форму.

1. Ты делаешь flash[:success] = "Comment deleted." только если собираешься рендерить html. Видимо правильнее вынести эту строчку из respond_to.

2. У тебя в app/views/layouts/application.html.erb есть код:

<% flash.each do |key, value| %>
  <div class="alert alert-<%= key %>">
    <%= value %>
  </div>
<% end %>

Именно он и отображает эти сообщения при рендеринге страницы. В случае если ты отдаёшь ответ в аяксом то этот код не участвует в рендеринге. Тебе нужно вынести этот кусок кода например в app/views/layouts/_flash.html.erb Затем в app/views/layouts/application.html.erb вместо него сделать:

<div class="JS-headers">
  <%= render 'layouts/flash' %>
</div>

И в свой js-представления добавить строку:

$(".JS-headers").html("<%=j render 'layouts/flash' %>");
2
  • Должен ли пункт 2 работать без п1? Все вместе работает замечательно, спасибо за помощь.
    – Tiazar
    3 дек 2015 в 10:51
  • 1
    Нет конечно. п.2 обеспечивает вывод. П.1 - формирование сообщения. Поотдельности либо нечему выводить их либо некуда.
    – anoam
    4 дек 2015 в 8:08

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.