在表单中找不到Rails - Id

Question

有段时间我被困在这个问题上。我试图通过一个表单传递3个Id，以便将数据保存到我的数据库中。

def new
  @person = Person.find(params[:person])
  @honored = Person.find(params[:honored])
  @group = Group.find(params[:group_id])
  @honor = Honor.new
end

def create
  @person = Person.find(current_person)
  @honor = Honor.create(:group => Group.find(params[:group_id]),
           :person => Person.find(params[:person]),
           :honored => Person.find(params[:honored]))
 if @honor.valid?
  flash[:success] = "Honor created."
  redirect_to (:back)
 else
  redirect_to (:back)
 end
end

在我看来，我在其中调用new方法：

<% @asked_groupmembership.each do |agm| %>
<%= link_to "Create Honor", new_honor_path(:group_id => @group.id, :person => current_person.id,
  :honored => agm.member.id) %> 

我的表格：

<% form_for @honor do |f| %>

 <%= f.hidden_field :group_id, :value => @group.id %>
 <%= f.hidden_field :person, :value => current_person.id %>
 <%= f.hidden_field :honored, :value => @honored.id %>

 <div class="field">
<%= f.label :texto %><br />
<%= f.text_field :texto %>
 </div>

当我单击submit按钮时，会得到以下错误：

Couldn't find Group without an ID

app/controllers/honors_controller.rb:22:in `create'

{"utf8"=>"✓",
 "authenticity_token"=>"C7wofMY4VcyfdS10oc3iglex8nZVBMQ0Nh22nMiaOqs=",
 "honor"=>{"group_id"=>"39",
 "person"=>"2",
 "honored"=>"44",
 ...
                },
 "commit"=>"Insert"}

我怎么才能解决这个问题？谢谢。

##EDITED##

class Honor < ActiveRecord::Base
  belongs_to :person, :class_name => 'Person', :foreign_key => "person_id"
  belongs_to :honored, :class_name => 'Person', :foreign_key => "honored_id"
  belongs_to :group, :class_name => 'Group', :foreign_key => "group_id"

Answer 1

您需要使用以下内容更新create方法：

def create
  @person = Person.find(current_person)
  @honor = Honor.create(:group_id => params[:honor][:group],
           :person_id => params[:honor][:person],
           :honored_id => params[:honor][:honored])
 if @honor.save
 ...
end

因为在你提交表格后，你得到了你的数据在对角线:荣誉。您可以在问题中的代码中看到传递给控制器的哈希：

"honor"=>{"group"=>"39",
 "person"=>"2",
 "honored"=>"44",

Answer 2

为了让它更易读，让我们这样做

表格：

<% form_for @honor do |f| %>
  <%= f.hidden_field :group_id, :value => @group.id %>
  <%= f.hidden_field :person_id, :value => current_person.id %>
  <%= f.hidden_field :honored_id, :value => @honored.id %>
  ...
<% end %>

主计长：

def new
  @person = Person.find(params[:person])
  @honored = Person.find(params[:honored])
  @group = Group.find(params[:group_id])
  @honor = Honor.new
end

def create
  @person = Person.find(current_person)
  @honor = Honor.create(
    :group => Group.find(params[:honor][:group_id]),
    :person => Person.find(params[:honor][:person_id]),
    :honored => Person.find(params[:honor][:honored_id])
  )

  if @honor.save
    ...
  end
end

表单中的每个隐藏字段都将作为params[:honor]中的param到达控制器。

现在，看看控制器，有些东西看上去不太对。如果没有错误，create方法将创建一个新实例并保存该对象。调用create，然后调用save，但您可以调用new (而不是create)，然后调用save，或者继续使用create，然后再调用valid?。就我个人而言，我认为new然后save过程更干净。

示例：

def create
  @person = Person.find(current_person)
  @honor = Honor.new(
    :group => Group.find(params[:honor][:group_id]),
    :person => Person.find(params[:honor][:person_id]),
    :honored => Person.find(params[:honor][:honored_id])
  )

  if @honor.save
    ...
  end
end