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 <javascript:>> 
> 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+unsubscr...@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.

Reply via email to