has_one嵌套属性不保存

时间:2014-05-01 08:42:00

标签: ruby-on-rails ruby rails-activerecord model-associations strong-parameters

我有两个模型Project和ProjectPipeline。

我想创建一个Project表单,其中还包含ProjectPipeline模型中的字段。我已成功创建表单但是当我点击保存时,值不会存储在数据库中。

project.rb

class Project < ActiveRecord::Base

  has_one :project_pipeline

  accepts_nested_attributes_for :project_pipeline

  self.primary_key = :project_id
end

projectpipeline.rb

class ProjectPipeline < ActiveRecord::Base

  belongs_to :project, autosave: :true

  validates_uniqueness_of :project_id

end

我并不总是想要一个项目管道,但是在适当的条件下,基于用户查看项目。我希望构建项目管道字段,但只有在用户选择保存/填充它们时才会保存。

因此,当项目显示时,我使用project_id构建一个临时项目管道:来自params [:id](不确定我是否真的需要这样做)。然后,当项目保存时,我使用create_attributes。但如果它已经被创建或构建,我只想让has_one和belongs_to关联启动,然后使用update_attributes。

我的问题是,当我试图保存时,我要么正在点击“禁止属性”#39;如果我使用params [:project_pipeline],或者如果我使用project_params则根本没有保存,则会出错。我检查并重新检查了我在project_params中的所有字段,甚至尝试使用project_pipeline_params,但感觉不对。

这让我疯了,我需要睡觉。

projects_controller.rb

def show
  @project = Project.find(params[:id])
  if @project.project_pipeline
  else
    @project.build_project_pipeline(project_id: params[:id])
  end
  autopopulate
end



def update
    @project = Project.find(params[:id])
    if @project.project_pipeline
    else
      @project.build_project_pipeline(project_id: params[:id], project_type: params[:project_pipeline][:project_type], project_stage: params[:project_pipeline][:project_stage])
    end
    if @project.update_attributes(project_params)
      flash[:success] = "Project Updated"
      redirect_to [@project]
    else
      render 'edit'
    end
  end


def project_params
  params.require(:project).permit(:user_id, project_pipeline_attributes:[:project_id,:project_type,:project_stage,
      :product_volume,:product_value,:project_status,:outcome, :_destroy])
end

show.html.haml

    - provide(:title, "Show Project")
%h1= @project.project_title
= simple_form_for(@project) do |f|
  = f.input  :id, :as => :hidden, :value => @project, :readonly => true  
  = f.input :user_id, label: 'Assigned to Account Manager', :collection => @account_managers, :label_method => lambda { |r| "#{r.first_name} #{r.last_name}" }
  = f.input :project_id, :readonly => true
  = f.input :status, :readonly => true
  = f.input :project_stage, :readonly => true

  - if @project.project_codename = "project pipeline"
    = simple_fields_for @project.project_pipeline do |i|
      %h2 Project Pipeline
      - if @project.user_id == current_user.id
        = i.input :project_volume, label: 'Project Status', collection: @project_status
        = i.input :project_value, label: 'Project Status', collection: @project_status
        = i.input :project_status, label: 'Project Status', collection: @project_status
      = i.input :outcome, label: 'Outcome', collection: @outcome


    = f.submit 'Save'

如果你已经走到这一步,我真诚地感谢你。

2 个答案:

答案 0 :(得分:2)

解决方案

你需要在这里改变一些事情。首先:

= simple_fields_for @project.project_pipeline do |i|

当您传递对象时,rails不知道它与父对象关联,因此将创建一个名为project[project_pipeline]而不是project[project_pipeline_attributes]的字段。相反,您需要传递关联名称并在表单构建器上调用此方法:

= f.simple_fields_for :project_pipeline do |i|

这将检查您是否已定义project_pipeline_attributes=方法(使用accept_nested_attributes_for`并将其视为关联。然后在您的控制器中将您的show动作更改为:

def update
  @project = Project.find(params[:id])
  @project.assign_attributes(project_params)
  if @project.save
    flash[:success] = "Project Updated"
    redirect_to @project
  else
    render 'edit'
  end
end

一切都应该奏效。作为单独的注释,由于您在嵌套参数中允许:_destroy属性,我假设您希望能够使用嵌套属性删除记录。如果是,则需要在allow_destroy: true来电中添加accepts_nested_attributes_for

现在有点造型:

您可以稍微改善您的表演动作。首先,我注意到如果还没有声明,你在每一个动作中都在构建一个空管道。这意味着你可能应该将这个逻辑移到你的模型中:

class Project < AR::Base

  after_initalize :add_pipeline

  private

  def add_pipeline
    project_pipeline || build_project_pipeline
  end
end

你也有神秘的方法prepopulate - 很可能它也应该是模特关注的。

另一点:这句语法:

if something
else
  # do sth
end

在某种程度上非常受欢迎,并使代码难以理解。相反,使用:

if !something
  # do something
end

或(首选)

unless something
  # do something
end

答案 1 :(得分:0)

我不确定你的描述是否存在问题,但有一点是,默认情况下,带有has_one的update_attributes将重建子节点(!),因此你将失去初始化的属性。您应该向update_only: true提供de accepts_nested_attributes_for选项。 您可以在此处找到更多相关信息in the rails docs。这条线是这样的:

accepts_nested_attributes_for :project_pipeline, update_only: true

考虑到after_initialize,这将导致每个项目总是有一个管道。虽然这可能是可取的,但不一定,取决于你的域名,所以我会对此有点小心。

干杯, 尼尔斯