2

我努力把它消化成一个标题。

我正在使用 SimpleForm 构建具有一个或多个字段集的批量编辑页面 - 一个用于已构建但尚未保存的 ActiveRecord 模型集合中的每条记录。

我的表格如下所示：

= simple_form_for :courses, method: :patch do |f|
  - @courses.each do |course|
    = field_set_tag do
      = f.simple_fields_for 'courses[]', course do |c|
        = c.input :title
        .row
          .medium-6.columns
            = c.input :start_date, as: :string, input_html: { class: 'input-datepicker' }
          .medium-6.columns
            = c.input :end_date, as: :string, input_html: { class: 'input-datepicker' }
  = f.submit 'Save', class: 'primary button'

一条记录的参数哈希如下所示：

"courses"=>{"courses"=>[{"title"=>"Course Y", "start_date"=>"2017-09-26", "end_date"=>"2017-07-31"}]}

使用数组，而对于两条记录，它看起来像这样：

"courses"=>{"courses"=>{"1"=>{"title"=>"Course X", "start_date"=>"2018-01-16", "end_date"=>"2018-07-30"}, "2"=>{"title"=>"Course Y", "start_date"=>"2017-09-26", "end_date"=>"2018-07-30"}}}

带有字符串整数键控哈希。

当我尝试使用强参数时，这会成为一个问题。经过多次黑客攻击，我最终得到了这段代码，它适用于多条记录，但仅在提交一条记录时失败：

ActionController::Parameters
  .new(courses: params[:courses][:courses].values)
  .permit(courses: [:title, :start_date, :end_date])
  .require(:courses)

param is missing or the value is empty: courses突出显示上面的行失败.require(:courses)。

通过协调单记录情况与多记录情况可以“解决”该问题：

if params[:courses][:courses].is_a?(Array)
  params[:courses][:courses] = { '1': params[:courses][:courses][0] }
end

但感觉应该有一种更简单的方法。

有没有更好的方法来为这个用例编写表单？我是否错过了具有强参数的技巧？

我正在使用rails 5.0.5和simple_form 3.5.0。

4

3 回答 3

1

“但感觉应该有一种更简单的方法。”

是的，使用 ajax 发送单独的创建/更新请求。这可以对用户透明地完成，并提供更简单的代码和更好的用户体验。

Rails 具有fields_for并且accepts_nested_attributes可用于在单个请求中创建/更新多个子记录和父记录。但它确实需要一个将记录组合​​在一起的关联，即使在这种情况下，在验证方面也会变得非常糟糕和令人费解。

您想对其进行设置，以便为每条记录提供一个单独的表单：

- courses.each do |c|
  = render partial: 'courses/_form', course: c

表格实际上没有任何内容：

# courses/_form.haml.erb
= simple_form_for course, remote: true, html: { 'data-type' => 'json', class: 'course_form'} do |f|
  = c.input :title
  .row
    .medium-6.columns
      = c.input :start_date, as: :string, input_html: { class: 'input-datepicker' }
    .medium-6.columns
      = c.input :end_date, as: :string, input_html: { class: 'input-datepicker' }
  = f.submit 'Save', class: 'primary button'

我们不使用js.erb模板，而是使用'data-type' => 'json' 并编写自己的处理程序，因为它更容易定位正确的表单：

$(document).on('ajax:success', '.course_form', function(event, xhr, status){
  var $form = $(this);
  alert('Course created');
  if (this.method.post) {
    // changes form to update instead.
    this.method = 'patch';
    this.action = xhr.getResponseHeader('Location');
  } 
});

$(document).on('ajax:error', '.course_form', function(event, xhr, status){
  var $form = $(this);
  // @todo display errors
});

创建控制器非常简单：

class CoursesController
  def create
    @course = Course.new(course_params)
    respond_to do |format|
      if @course.save(course_params)
        format.json { head :created, location: @course }
      else
        format.json do
          render json: {
            errors: @course.errors.full_messages
          }
        end
      end
    end
  end

  def update
    @course = Course.find(params[:id])
    respond_to do |format|
      if @course.update(course_params)
        format.json { head :ok }
      else
        render json: {
          errors: @course.errors.full_messages
        }
      end
    end
  end
end
于 2017-11-07T18:56:09.960 回答
1

保留你的表单，将强大的参数更改为：

params.require(:courses).permit(
  courses: [
    :id,
    :title,
    :start_date, 
    :end_date
  ]
)

使用此代码参数应该没有索引键，@courses只是一个数组：

# CoursesController
def new
  @courses = []
  # creating 3 items for example
  3.times do
    @courses << Course.new
  end
end

def create
  errors = false
  @courses= []
  # keep courses in the array for showing errors
  courses_params[:courses].each do |params|
    course = Course.new(params)
    @courses << course 
    unless course.valid?
      errors = true
    end
  end
  if errors
    render :new
  else
    # if no errors save and redirect
    @courses.each(&:save)
    redirect_to courses_path, notice: 'courses created'
  end
end
于 2017-11-07T21:40:15.080 回答
0

事实证明，该f.simple_fields_for 'courses[]' ...方法仅在表单由现有记录填充时才为该字段集提供 ID，并且仅在这种情况下使用字符串 ID 映射到课程哈希的 params 结构。对于“新鲜”的记录，没有 ID，课程哈希被放置在一个普通数组中。

这段代码是在将课程从一年“滚动”到另一年的上下文中运行的——复制以前的课程并更改日期。这意味着每个字段集都有原始课程的 ID。

提交表单时，会创建一条新记录并使用新属性进行验证，正是这条没有 ID 的新记录重新填充了表单。“只有在提交一门课程时才会发生”这件事是一个红鲱鱼 - 测试场景的产物。

所以值得注意的是：f.simple_fields_for 'courses[]' ...为新记录创建一个数组，并为现有记录创建一个哈希映射 ID 到属性。

于 2017-11-08T15:59:17.860 回答