Question

所以我有一个非常简单的模型，其中设置了嵌套属性：

class Property < ActiveRecord::Base
  has_many :trees
  accepts_nested_attributes for :trees
end

然后我在我的控制器中构建它们：

class PropertiesController < ApplicationController

  def new
    @property = Properties.new
    @trees = @property.trees.build
  end
end

然后在我看来（使用苗条）：

= form_for @property |p|
  = p.text_field :name

  .tree
    = p.fields_for :trees do |t|
      = t.text_field :fruit
      = t.select :quantity, (1..1000).to_a

    = link_to 'Add Another Tree', new_properties_path, id: 'new-tree'

然后在我的js（coffeescript）文件中，绑定到'＃new-tree'锚点的click事件：

event.preventDefault()
tree = $(event.target).parents('.tree')
tree.after tree.clone(true)

这可以按照您的预期工作，但是当我提交表单时，只提交最后一个树参数。这是因为在form_for生成的html中我得到了这样的东西：

<input type="text" id="property_trees_attributes_0_fruit" name="property[trees_attributes][0][fruit]">
<select id="property_trees_attributes_0_quantity" name="property[trees_attributes][0][quantity]" class="valid">
  <option value="1">1</option>
  <option value="2">2</option>
  <option value="3">3</option>
  ...
</select>

当我克隆这个html时，我得到了0中的1的名称和id，但我认为它应该是event.preventDefault() tree = $(event.target).parents('.tree') clone = tree.clone() number = parseInt(clone.find('input[type=text]').attr('id').match /[0-9]/) next = number + 1 clone = clone.html().replace /_(#{number})_|\[(#{number})\]/, next section.after clone来添加额外的树参数等。

我尝试通过做这样的事情补偿js：

_0_

正则表达式的目的是替换两个下划线或两个括号之间的任何数字，例如[0]或_0_

我的正则表达式似乎不起作用。它只用1替换{{1}}，这个解决方案对我来说似乎很麻烦。我会使正则表达式更简单，但是我不得不担心它会改变选择菜单选项的值。

有关清理的建议吗？我错过了一些非常明显的东西吗？

Answer 1

你遇到的问题是你只是复制了DOM元素 - 这是IMO的一个黑客攻击，因为你之前没有构建你需要的对象（不会保留数据等）

我们之前使用Ajax实现了“向f.fields_for添加额外字段” - 有great tutorial here和RailsCast here

<强> CHILD_INDEX

您的问题的答案是使用child_index帮助中的fields_for选项，并使用integer时间戳：

<% index = Time.now.to_i %>
<%= f.fields_for :trees, child_index: index do |fields| %>
    #your fields here
<% end %>

<强>代码

以下是我们的工作：

#app/views/properties/new.html.erb
<%= form_for @property do |f| %>
   <%= render partial: "trees_fields", locals: { f: f, child_index: Time.now.to_i } %>
   <%= link_to "New Field", new_field_property_path %>
<% end %>

#app/views/properties/_trees_fields.html.erb
<%= f.fields_for :trees, child_index: child_index do |trees| %>
   <%= trees.text_field :your_attr %>
<% end %>

#app/controllers/properties_controller.rb
def new_field
    @property = Property.new
    @property.trees.build
    render "add_tree", layout: false
end

#app/views/properties/add_tree.html.erb
<%= form_for @property do |f| %>
      <%= render partial: "trees_fields", locals: {f: f, child_index: Time.now.to_i} %>
<% end %>

 #app/assets/javascripts/application.js.coffee
$ ->
  $(document).on "click", "#add_tree", (e) ->
     e.preventDefault();
     $.ajax
       url: '/messages/add_subscriber'
       success: (data) ->
              el_to_add = $(data).html()
          $('#trees_form').append(el_to_add)
       error: (data) ->
          alert "Sorry, There Was An Error!"

Answer 2

您是对的，嵌套的.tree元素不应具有相同的ID。所以你不应该克隆它们。而是使用帮助器构建新项目，例如，基于当前时间生成id，然后粘贴新的嵌套元素。这是一个复杂的场景，所以我认为观看相关的railscast会更好：

Rails嵌套属性form_for for has_many，使用javascript无限制嵌套模型

2 个答案: