Ruby on Rails Tuesday, June 27, 2017



On Tuesday, June 27, 2017 at 7:32:33 AM UTC-4, Walter Lee Davis wrote:

> On Jun 27, 2017, at 12:40 AM, fugee ohu <fuge...@gmail.com> wrote:
>
>
>
> On Wednesday, June 21, 2017 at 8:34:17 AM UTC-4, Walter Lee Davis wrote:
>
> > On Jun 21, 2017, at 3:52 AM, fugee ohu <fuge...@gmail.com> wrote:
> >
> > How do I give new.js.erb the right element name
> >    $('#comment_post)click.(function(){
> > In this line there's no id number
>
> I think you may have missed a few bits of my reply. We are not observing a click any more. Everything is out of the jQuery pool, and fully into the RJS pool. If you're viewing this in the Google Groups (Web) view, you may need to expand all of the little ... ellipses in order to see all of my threaded replies.
>
> in the second content_tag_for statement collection: post.comments fails to produce anthing for comment.body when the _comment.html.erb partial is rendered
>
> _post.erb.html
>
>  <%= content_tag_for(:div, post) do %>
>     <%= simple_format post.content %>
>     <% unless post.attachment.blank? %>
>       <%= image_tag(post.attachment, height: 250) %><br>
>     <% end %>
>     <%= content_tag_for(:div, post, 'comments') do %>
>         <%= render '/comments/comment', collection: post.comments %>
>         <%= content_tag_for(:div, post, 'comment_form_holder') do %>
>             <%= link_to 'Comment', new_comment_path('comment[commentable_id]': post.id, 'comment[commentable_type]': 'Post') %><br><br>
>         <% end %>
>     <% end %>
> <% end %>
>
>
> _comment.html.erb
>
> <%= content_tag_for(:div, @comments) do %>
>     <%= simple_format comment.body %>
>     <%= comment.post.user.first_name comment.post.user.last_name %>
> <% end %>


content_tag_for builds the DIV and gives it an ID based on a single item passed to it. I'm not sure what it would make of a collection of comments, which is what you're passing it here. What I think you need to do is change @comments to comment, to match the rest of the local variables inside the partial. @comments doesn't exist inside the partial unless you render it in a scope that already had that set as an instance variable. comment exists inside the partial because the partial is named comment, but that's only true if you are either rendering it with the "shortcut" collection: @comments, where @comments is an array or association of individual comment instances, or are rendering it one at a time, and passing in locals: { comment: @comment } or similar. Because you are doubly-nested at this point, inside _post.html.erb, it is best to stick with the local variables that you have passed along, and not rely on the surrounding render context to magically provide variables for you.

Try just changing @comments to comment in the first line of your _comment.html.erb, leave everything else alone, and see if it renders then. Just to be certain, check in the console whether you have any comments for that post you are trying to render. You should not see anything at all (the comment partial won't even render) if post.comments is empty.

Walter

I took out the div containers (content_tag_for) and just left:
    <%= simple_format comment.body %>
    <%= comment.post.user.first_name comment.post.user.last_name %>
but I still get the error that comment is undefined

--
You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rubyonrails-talk+unsubscribe@googlegroups.com.
To post to this group, send email to rubyonrails-talk@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/rubyonrails-talk/eea548b0-8979-4f0c-9662-9c720f5f35db%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

No comments:

Post a Comment