如何在rails 4中声明嵌套表单的对象

时间:2015-04-29 16:14:40

标签: ruby-on-rails ruby-on-rails-4 nested-forms has-one

我正在接近使用rails的第二周,几乎已经完成了我的应用程序的结构(我想/希望)。我正在为慈善机构建立一个员工网站,作为他们工作的一部分收集和擦拭计算机(在将它们发送到非洲学校之前!)。

我已经将我的应用设置为拥有员工,计算机和擦除模型/类,并在表单中嵌套表格以获取计算机详细信息。当我创建一台新计算机时,我可以毫无问题地输入擦除详细信息,如果我编辑已经具有擦除详细信息的现有计算机的擦除详细信息,它也可以很好地工作。但是,当我向现有计算机添加擦除详细信息而不删除详细信息时,我收到以下错误:

undefined method `staff=' for nil:NilClass
Extracted source (around line #49):

如何将computer.wipe.staff =正确设置为当前用户。请注意,要编辑或提交擦除详细信息,用户必须登录。

我的模型如下(忽略不相关的验证等):

class Staff < ActiveRecord::Base
  has_many :wipes
  before_save { self.staff_email = staff_email.downcase }
  validates :staff_name, presence: true, length: { minimum: 5, maximum: 50}
  has_secure_password
end

class Computer < ActiveRecord::Base
  has_one :wipe, dependent: :destroy
  accepts_nested_attributes_for :wipe
end

class Wipe < ActiveRecord::Base
  belongs_to :computer#, dependent: destroy
  belongs_to :staff
  validates :staff_id, presence: true
 #validates :computer_id, presence: true
  validates :action_taken, presence: true, length: { minimum: 2, maximum: 250 }
end

我的电脑控制器如下:

class ComputersController < ApplicationController
  before_action :set_computer, only: [:edit, :update, :show]
  before_action :require_user, except: [:new, :create]

  layout :new_layout, only: [:new, :update]

  def new
    @computer = Computer.new
    @computer.build_wipe
  end

  def create
    @computer = Computer.new(computer_params)
    if logged_in?
      @computer.wipe.staff = current_user
    end
    if @computer.save
      flash[:success] = "You're computer's details have been submitted successfully!"
      redirect_to computers_path
    else
      render :new, layout: new_layout
    end
  end

  def edit
    unless @computer.wipe
      @computer.build_wipe
    end
  end

  def update
    @computer.wipe.staff = current_user #This is the line not working, yet seems to work correctly in the create action
    if @computer.update(computer_params)
      flash[:success] = "The computer's details have been updated successfully!"
      redirect_to computer_path(@computer)
    else
      render :edit
    end
  end

  private

    #Whitelisting variables
    def computer_params
      params.require(:computer).permit(:manufacturer, :computer_type, :specification, :donor, :model_no, :serial_no, :product_key, :turingtrack, :picture, wipe_attributes: [:id, :action_taken, :staff_id])
    end

    def set_computer
      @computer = Computer.find(params[:id])
    end

end

我的观点是:

<div class="row">
  <div class="well col-md-8 col-md-offset-2">
    <%= form_for(@computer, html: { multipart: true }) do |f| %>

      <%= f.label :manufacturer %>
      <%= f.text_field :manufacturer, :placeholder => "Toshiba" %>
      <%= f.label :computer_type, "Computer Type" %>
      <%= f.text_field :computer_type, :placeholder => "Laptop"%>

      <% if logged_in? %>
        <br><strong>Wiping Details:<br><br></strong>
        <%= f.fields_for :wipe do |wipe_form| %>
          <%= wipe_form.label :action_taken %>
          <%= wipe_form.text_field :action_taken %>
        <% end %>
      <% end %>

      <%= f.submit(@computer.new_record? ? "Submit Details" : "Submit Edited Details", class: "btn btn-success") %>
    <% end %>

  </div>
</div>

此外,目前我没有在计算机表单中使用嵌套擦除表单时设置computer_id,它似乎自动分配它(例如,如果我创建了一台新计算机/编辑一个并输入擦除详细信息,该擦除的computer_id是也创建/编辑的计算机的computer_id。这个假设是否正确?我在某处读到了无法验证父母外键的信息,但它太简短了,无法理解。当我尝试在擦除模型中验证jenkins: formatter: name: pretty,junit parameters: output_path: ,build/logs/behat 时会抛出错误,因此验证被注释掉了。为什么这样,并且可以这样离开吗?

谢谢大家

2 个答案:

答案 0 :(得分:0)

基于错误:

    undefined method `staff=' for nil:NilClass.

这表明 @ computer.wipe 为零。因此,当您的 set_computer 方法根据提供的ID正确查找计算机时, @computer 没有关联的擦除

编辑:这可能与您的 build_wipe 方法中的问题无法正确设置划像有关。

答案 1 :(得分:0)

我似乎已经解决了我的问题,但我不太明白为什么会这样。 我在编辑和更新操作中都构建了一个擦除对象。第一个是在表单中创建输入字段,第二个是在分配工作人员时使用的实例。请注意,当我在控制台中检查是否已创建两个擦除对象时,根据需要只有一个擦除对象。

def edit
  unless @computer.wipe
    @computer.build_wipe
  end
end

def update
  if @computer.wipe 
  else
    @computer.build_wipe
  end
  @computer.wipe.staff = current_user
  if @computer.update(computer_params)
    flash[:success] = "The computer's details have been updated successfully!"
    redirect_to computer_path(@computer)
  else
    render :edit
  end
end