ruby-on-railstreeancestrythreaded-comments

Unable to assign a parent id to nested comments, using Ancestry gem (ruby on rails)


I am creating nested comments (like you find on Reddit). I am able to create parent comments, but when I try to create a child comment, it simply renders as a parent comment.

In my rails console, the "ancestry" field comes back "nil".

This is my comments controller:

class CommentsController < ApplicationController
  before_action :set_comment, only: [:show, :edit, :update, :destroy]
  before_filter :authenticate_user!

  def show
    @comment = Comment.find(params[:id])
  end

  def new
    @link = Link.find(params[:link_id])
    @comment = Comment.new(:parent_id => params[:parent_id])
    @comments = Comment.all
  end

  def create
    @link = Link.find(params[:link_id])

    @parent = Link.find(params[:link_id]) if params[:link_id]
    @parent = Comment.find(params[:comment_id]) if params[:comment_id]

    @comment = @parent.comments.new(comment_params)
    @comment.user = current_user

    respond_to do |format|
      if @comment.save
        format.html { redirect_to @link, notice: 'Comment was successfully created.' }
        format.json { render json: @comment, status: :created, location: @comment }
      else
        format.html { render action: "new" }
        format.json { render json: @comment.errors, status: :unprocessable_entity }
      end
    end
  end

  def destroy
    @comment.destroy
    respond_to do |format|
      format.html { redirect_to :back, notice: 'Comment was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  private

    def set_comment
      @comment = Comment.find(params[:id])
    end


    def comment_params
      params.require(:comment).permit(:link_id, :body, :user_id)
    end
end

Here is my _comment_form partial

<%= div_for(comment) do %>
    <div class="comments_wrapper clearfix">
        <div class="pull-left">
            <p class="lead"><%= comment.body %></p>
            <p><small>Submitted <strong><%= time_ago_in_words(comment.created_at) %> ago</strong> by <%= comment.user.email %></small></p>
                    <div id="reply" style="display:none;">
                <%= form_for [@comment = Comment.new(:parent_id => params[:parent_id])] do |f| %>
                <%= f.hidden_field :parent_id %>
                  <%= f.text_area :body %> <br>
                  <%= f.submit %>
                  <% end %>
                </div>
        </div>

        <div class="actions btn-group pull-right">
            <button onClick="$('#reply').show()" class="btn btn-sm btn-default">Reply</button>

            <% if comment.user == current_user -%>
                <%= link_to 'Destroy', comment, method: :delete, data: { confirm: 'Are you sure?' }, class: "btn btn-sm btn-default" %>
            <% end %>
        </div>
    </div>

<% end %>

These are my routes

Rails.application.routes.draw do
  resources :comments
  devise_for :users
  devise_for :installs
  resources :links do
    member do 
      put "like", to: "links#upvote"
      put "dislike", to: "links#downvote"
    end
    resources :comments
  end

  root to: "links#index"
end

Solution

  • Had this problem before; the answer is here:

    Ancestry gem in Rails and Mutli Nesting

    enter image description here

    The problem with ancestry (this is why we changed back to acts_as_tree) is that you have to define all the ancestors in the ancestry column (as opposed to just the parent_id column of acts_as_tree).

    Thus, when you call the .children of an object (where you've literally just populated ancestry with top-level parents) is a list of children for that parent (no others).

    What you need is to reference the entire ancestry line. This is quite tricky, but can be achieved using the code below:

    #app/views/links/index.html.erb
    <%= render @link.comments if @post.comments.any? %>
    
    #app/views/links/_comment.html.erb
    <%= comment.title %>
    <%= render "form", locals: {link: @link} %>
    <%= render comment.children if comment.has_children? # > adds recursion (multi level nesting) %>
    
    #app/views/links/_form.html.erb
    <%= form_for link.comments.new do |c| %>
       <%= c.text_field :body %>
       <%= c.submit %>
    <% end %>
    

    The controller is as follows:

    #app/controllers/comments_controller.rb
    class CommentsController < ApplicationController
       def create
          @link = Link.find params[:link_id]
          @comment = @link.comments.new ancesrtry: parent(params[:parent_id])
       end
    
       private
    
       def parent(param)
           parents = Comment.find(param).pluck(:parent)
           "#{parents}/#{param}" #-> ruby automatically returns last line
       end
    end
    

    This should set the correct path for you, and the partials should give you the appropriate recursion required for multi level nesting.