Rails 4更新/删除具有嵌套字段的模型中的字段的问题

时间:2015-01-09 06:53:31

标签: ruby-on-rails ruby-on-rails-4 associations crud nested-forms

我一直致力于在Railscast第196集http://railscasts.com/episodes/196-nested-model-form-revised之后在我的应用中引入嵌套表单,并为rails 4 https://github.com/dnewkerk/nested-model-form重新制作版本。

我们假设收据文章之间存在1对多的关联。

以下是他们模特的样子:

receipt.rb:

class Receipt < ActiveRecord::Base
  has_many :articles, dependent: :destroy
  accepts_nested_attributes_for :articles, allow_destroy: true, reject_if: :all_blank

  belongs_to :shop
  belongs_to :user

  def display_name
    self.name
  end
end

article.rb:

class Article < ActiveRecord::Base
  belongs_to :receipt

  def name_with_brand
    "#{name} #{brand}"
  end
end

以下是receipts_controller.rb的外观:

class ReceiptsController < ApplicationController
  before_action :set_shop, only: [:show, :edit, :update, :destroy]

  respond_to :html, :xml, :json

  def index
    @receipts = current_user.receipts
    respond_with(@receipts)
  end

  def show
    respond_with(@receipt)
  end

  def new
   @receipt = Receipt.new
   2.times do
     @receipt.articles.build
   end
   respond_with(@receipt)
  end

  def edit
  end

  def create
    @receipt = Receipt.new(receipt_params)
    user_id = current_user.id

    @receipt.articles.each do |article|
      warranty_time = article.warranty_time
      article.warranty_expires = @receipt.shopping_date.advance(months: warranty_time)
    end

    @receipt.user_id = user_id
    @receipt.save
    respond_with(@receipt)
  end

  def update
    if @receipt.update(receipt_params)
      redirect_to @receipt, notice: "Successfully updated receipt."
    else
      render :edit
    end
  end

  def destroy
    @receipt.destroy
    respond_with(@receipt)
  end

  private

  def set_shop
    @receipt = Receipt.find(params[:id])
  end

  def receipt_params
    params.require(:receipt).permit(:name, :shopping_date, :shop_id, :file, 
    articles_attributes: [:id, :name, :brand, :warranty_time, :warranty_expires, 
                          :receipt_id,  :_destroy])
  end
end

这是我的receiptts.js.coffee的样子:

jQuery ->
  $('#receipt_shopping_date').datepicker(dateFormat: 'yy-mm-dd')
  $.datepicker.setDefaults($.datepicker.regional['PL']);


  $('form').on 'click', '.remove_fields', (event) ->
  $(this).prev('input[type=hidden]').val('1')
  $(this).closest('fieldset').hide()
  event.preventDefault()

  $('form').on 'click', '.add_fields', (event) ->
  time = new Date().getTime()
  regexp = new RegExp($(this).data('id'), 'g')
  $(this).before($(this).data('fields').replace(regexp, time))
  event.preventDefault()


$(document).ready(jQuery)
$(document).on('page:load', jQuery)

最后,我的观点是添加新收据并向其中添加文章:

(other fields...)

<div class="large-12 columns">
<p>Add articles on the receipt:</p>
</div>

<div class="field">
  <div class="large-12 columns">


  <%= f.fields_for :articles do |builder| %>
        <div class="article_fields">
    <%= render "article_fields", :f => builder %>
        </div>
        <% end %>

    <%= link_to_add_fields "Add another article", f, :articles %>

  </div>
</div>


<div class="actions">
<div class="large-12 columns">
    <%= f.submit "Sumbit Receipt" %>
</div>
</div>


<% end %>

正如您所看到的,我使用的是 link_to_add_fields 辅助方法,这就是它的样子:

def link_to_add_fields(name, f, association)
new_object = f.object.send(association).klass.new
id = new_object.object_id
fields = f.fields_for(association, new_object, child_index: id) do |builder|
  render(association.to_s.singularize + "_fields", f: builder)
end
link_to(name, '#', class: "add_fields small button", data: {id: id, fields: fields.gsub("\n", "")}) 
end

最后,你可以看到我生成一个名为_article_fields.html.erb的部分,这里是它的样子:

<fieldset style="width:1400px">
<legend>new article</legend>

<div class="large-2 columns">
<%= f.text_field :name%>
</div>

<div class="large-2 columns">
<%= f.text_field :brand%>
</div>

<div class="large-2 columns">
<%= f.text_field :warranty_time, class: "warranty" %>
</div>

<div class="large-12 columns">
<%= link_to "delete article", '#', class: "remove_fields button small alert" %>
</div>

</fieldset>

现在让我们来解决我的问题。在第一次创建收据时一切都很好 - 我在展示视图中看到收据中的文章数量,并在每篇文章中看到warranty_expires。

当我通过收据/编辑更新或删除article_fields时,事情变得混乱:

1)当我编辑收据并想要删除任何文章时(虽然在我的编辑视图中可视化它们消失了 - JS似乎可以工作),但是我的数据库中的字段不会被删除,因此显示视图仍然是准确的就像以前一样。

简单示例:

编辑前:我的收据有6篇文章

在编辑过程中:按下3次&#39;删除文章&#39;按钮,所以收据应该有3篇文章

编辑后:收据仍有6篇文章

2)当我编辑收据并想要添加另一个文章字段时,值warranty_expires始终为零 - 如何使其与我的收据控制器中的更新操作一起使用?我尝试使用与我的创建操作中相同的代码:

@receipt.articles.each do |article|
warranty_time = article.warranty_time
article.warranty_expires = @receipt.shopping_date.advance(months: warranty_time)
end

但它不会奏效。知道为什么吗?

简单示例:

收据已有2篇文章。当我添加第三个时,我得到以下结果:

3篇文章 - 所有这些文章都有name和warranty_time字段,但其中只有2个具有warranty_expires值。

非常感谢您的所有帮助。提前谢谢。

4 个答案:

答案 0 :(得分:1)

我认为您可以在文章模型中使用一些回调来解决第二个问题,

开始删除它,尽量让控制器尽可能简单并处理模型中的操作。

 @receipt.articles.each do |article|
  warranty_time = article.warranty_time
  article.warranty_expires = @receipt.shopping_date.advance(months: warranty_time)
end

在文章模型中添加一些回调

class Article < ActiveRecord::Base
  belongs_to :receipt

  def name_with_brand
    "#{name} #{brand}"
  end

  before_update :set_warranty_expires
  before_create :set_warranty_expires

  def set_warranty_expires
    self.warranty_expires = self.receipt.shopping_date.advance(months: self.warranty_time)
  end

end

代码未经测试,但其想法。希望它有所帮助。

检查这两个宝石simple_formnested_form这对于编写大型表单有很大帮助,并且它们可以很好地相互配合。

答案 1 :(得分:0)

更新:我设法修复了第一期。

第一个解决方案的修复方法如下:

隐藏字段:删除文章时缺少_destroy。

所以我需要更改以下代码:

<div class="large-12 columns">
<%= link_to "delete article", '#', class: "remove_fields button small alert" %>
</div>

为:

<div class="large-12 columns">
<%= f.hidden_field :_destroy %>
<%= link_to "delete article", '#', class: "remove_fields button small alert" %>
</div>

但仍然不知道如何解决第二个问题。

答案 2 :(得分:0)

首先我注意到,你的reciepts_controller新动作中有一个循环

2.times do 
  @receipt.articles.build 
end

这意味着,该文章将仅为该收据创建2次。

最好删除循环,以便添加所需数量的文章。 对于问题编号2,添加波纹管线以编辑控制器的操作

@receipt.articles.build

我想这会对你有帮助。

此外,nested_form是管理此类任务的绝佳宝石。

 https://github.com/ryanb/nested_form

检查出来。

答案 3 :(得分:0)

这是smooth.spline中调用.hide()的问题。我能想到的最简单的解决方法是将receipts.js.coffee替换为.hide()