Rails Ruby on Rails

Comments and changes to this ticket

• You flagged this item as spam.
  

  Denis Odorcic October 23rd, 2010 @ 04:05 AM

  Can't reproduce unfortunately, anyone else?

• 

  Aditya Sanghi October 23rd, 2010 @ 03:52 PM

  • State changed from “new” to “needs-more-info”
  • Importance changed from “” to “Low”

  It looks like you're doing a build on a post by default somewhere (perhaps a before_filter somewhere?). Please post part of your controller/model code and clear steps to reproduce the error.

• 

  Jeff Kreeftmeijer October 24th, 2010 @ 01:08 PM

  Automatic cleanup of spam.

• You flagged this item as spam.
  

  brpratt (at gmail) October 25th, 2010 @ 03:35 PM

  If it was doing a build by default wouldn't that be reflected in the db? Why would Comment.count show 0 and then my app gets an extra comment?

  This app is pretty much word-for-word from the rails guide for the blog app...
  http://guides.rubyonrails.org/getting_started.html

  I'll repost critical elements here to make it easier.

  comments controller:
  

  class CommentsController < ApplicationController
   def create
    @post = Post.find(params[:post_id])
    @comment = @post.comments.create(params[:comment])
    redirect_to post_path(@post)
   end
   def destroy
    @post = Post.find(params[:post_id])
    @comment = @post.comments.find(params[:id])
    @comment.destroy
    redirect_to post_path(@post)
   end
  end
  

  comments model:
  

  class Comment < ActiveRecord::Base
   belongs_to :post
  end
  

  posts model:
  

  class Post < ActiveRecord::Base
   validates :name,  :presence => true
   validates :title, :presence => true,
   has_many :comments
  end
  

• You flagged this item as spam.
  

  brpratt (at gmail) October 25th, 2010 @ 03:47 PM

  Interestingly I found in the form:

  <h2>Add a comment:</h2>
  <%= form_for([@post, @post.comments.build]) do |f| %>
    <%= f.error_messages %>
  
    <div class="field">
      <%= f.label :commenter %><br />
      <%= f.text_field :commenter %>
    </div>
    <div class="field">
      <%= f.label :body %><br />
      <%= f.text_area :body %>
    </div>
    <div class="actions">
      <%= f.submit %>
    </div>
  <% end %>
  

  If I changed the @post.comments.build object to @post.comments.new, it stopped returning a nil object in the array.

• You flagged this item as spam.
  

  brpratt (at gmail) October 25th, 2010 @ 03:47 PM

  Interestingly I found in the form:

  <h2>Add a comment:</h2>
  <%= form_for([@post, @post.comments.build]) do |f| %>
    <%= f.error_messages %>
  
    <div class="field">
      <%= f.label :commenter %><br />
      <%= f.text_field :commenter %>
    </div>
    <div class="field">
      <%= f.label :body %><br />
      <%= f.text_area :body %>
    </div>
    <div class="actions">
      <%= f.submit %>
    </div>
  <% end %>
  

  If I changed the @post.comments.build object to @post.comments.new, it stopped returning a nil object in the array.

• 

  Aditya Sanghi October 25th, 2010 @ 04:59 PM

  • State changed from “needs-more-info” to “invalid”

  Marking ticket as invalid for now as it's clear where your object is coming from.

  Please see documentation related to active record. It is clear that @post.comments.build is the way to build associations (not new).

  Doing build will create an object but not save it of course.

• You flagged this item as spam.
  

  bingbing March 26th, 2011 @ 01:14 AM

  luxury watch