无法找到没有ID的列表

时间:2013-04-17 19:15:51

标签: ruby-on-rails-3 activerecord

我有一个名为Listings的资源和一个名为email

的资源

列出has_many:电子邮件 电子邮件belongs_to:listing

我正在尝试将列表ID传递给email.listing_id,但我一直收到此错误:

Couldn't find Listing without an ID ActiveRecord::RecordNotFound in EmailsController#create

这是我的代码:

class EmailsController < InheritedResources::Base
    def new
        @listing = Listing.find(params[:listing_id])
        @email = Email.new

    end

    def create
        @listing = Listing.find(params[:listing_id])
        @email = Email.new(params[:email])
        @email.user_id = current_user.id


    end

从我的商品详情页面中我传递了listing.id

<%=link_to "Email Listing", new_email_path(:listing_id => @listing.id)

电子邮件_form

  <%= f.input :email %>
  <%= f.input :message %>
  <%= f.hidden_field :listing_id, :value => @listing.id %>
  %= f.button :submit, :value=> "Send Email" %>

参数:

"email"=>{"email"=>"xyz@z.com",
 "message"=>"hello",
 "listing_id"=>"3"},
 "commit"=>"Send Email"}

1 个答案:

答案 0 :(得分:3)

实际上很容易..你的参数是

params[:email][:listing_id]不是params[:listing_id]

啊,答案被删除了,你复制粘贴了你发送的参数,所以我不再证明这是这种情况,但是你从那里粘贴的东西,似乎params的形式

只需更改

@listing = Listing.find(params[:listing_id])

@listing = Listing.find(params[:email][:listing_id])

顺便说一下,还有很多需要改进的地方......

第一。您可以使用嵌套资源,因此您不必关心放置hidden_​​input。其次,您可以创建一个before_filter方法来获取正确的@listing并使用@ listing.emails.build来正确设置关系。

相关问题