7

我有以下表单对象来管理复杂的嵌套表单。

形式

= simple_form_for(@profile_form, :url => profiles_path) do |f|
  ...

路线

resources :profiles

控制器

class ProfilesController < ApplicationController
  def new
    @profile_form = ProfileForm.new
  end

  def edit
    @profile_form = ProfileForm.new(params[:id])
  end

  def create
    @profile_form = ProfileForm.new
    if @profile_form.submit(params[:profile_form])
      redirect_to @profile_form.profile, notice: 'Profile was successfully created.'
    else
      render action: "new"
    end
  end

  def update
    @profile_form = ProfileForm.new(params[:id])
    if @profile_form.submit(params[:profile_form])
      redirect_to @profile_form.profile, notice: 'Profile was successfully updated.'
    else
      render action: "edit"
    end
  end
end

表单对象

class ProfileForm
  include ActiveModel::Validations
  include ActiveModel::Conversion
  extend ActiveModel::Naming

  def initialize(profile_id = nil)
    if profile_id
      @profile = Profile.find(profile_id)
      @person = profile.person
    end
  end
  ...
  def submit(params)
    profile.attributes = params.slice(:available_at)
    person.attributes = params.slice(:first_name, :last_name)

    if valid?
      profile.save!
      person.save!
      true
    else
      false
    end
  end
  def self.model_name
    ActiveModel::Name.new(self, nil, "Profile")
  end

  def persisted?
    false
  end
end

但是现在，当我使用此表单create操作编辑对象时，它会被调用。那么我应该如何重构这个表单呢？下面的代码update创建另一个 Profile 对象。

4

3 回答 3

6

simple_form_for在内部使用form_for来完成其工作。form_for使用该方法persisted?来确定对象是否已经持久化在数据库中。如果它已经被持久化form_for，将生成一个带有PUT方法的表单来更新对象，否则它将生成一个带有POST方法的表单来创建新对象。因此，您必须为您的表单对象实现该persisted?方法。你可以像这样实现它：

class ProfileForm
  # ...
  def persisted?
    @person.persisted? && @profile.persisted?
  end
  # ...
end

更新如果@person是nil，即没有Person关联到Profile，我想你会创建一个新Person的关联到@profile。在这种情况下，可以安全地假设 a至少与ProfileFormis一样长，因此：persisted?@profilepersisted?

class ProfileForm
  # ...
  def persisted?
    @profile.persisted?
  end
  # ...
end

更新为避免错误undefined local variable or method `id'，您必须为 定义id方法ProfileForm，如下所示：

class ProfileForm
  # ...
  def id
    @profile.id
  end
  # ...
end
于 2013-06-12T12:14:14.380 回答
0 
replace

  = simple_form_for(@profile_form, :url => profiles_path) do |f|

   with

   = simple_form_for(@profile_form, :url => {:controller => "profiles"}) do |f|
于 2013-06-12T11:50:11.713 回答
0

看这里：http://apidock.com/rails/ActionView/Helpers/FormHelper/apply_form_for_options！

如果您希望表单更新您的记录，您应该编写方法ProfileForm#persisted?以便它返回。true

于 2013-06-12T11:58:04.853 回答