ruby-on-rails - 如何为 after_save 创建的对象返回验证错误？

Question

我正在编写一个预订系统，它使用 ice_cube 处理定期预订。 gem 。一个 Booking has_many BookingItem s，重复规则中的每次出现一个，这些是在由 Booking 调用的方法中创建的。的 after_save 回调。

在我向 BookingItem 添加验证之前，一切正常。通过检查是否还没有 BookingItem 来避免重复预订在给定的时间。此验证引发了一个错误，我想在预订表单上显示该错误，但目前它只是默默地阻止了 Booking从被保存 - 因为错误是由 BookingItem 引发的它没有被传递回 Booking 的表单.

应用程序/模型/booking.rb

class Booking < ActiveRecord::Base
  include IceCube

  has_many :booking_items, :dependent => :destroy

  after_save :recreate_booking_items!

  # snip

  private

  def recreate_booking_items!
    schedule.all_occurrences.each do |date|
      booking_items.create!(space: self.requested_space, 
                            booking_date: date.to_date,
                            start_time: Time.parse("#{date.to_date.to_default_s} #{self.start_time.strftime('%H:%M:00')}"),
                            end_time: Time.parse("#{date.to_date.to_default_s} #{self.end_time.strftime('%H:%M:00')}"))
    end
  end
end

应用程序/模型/booking_item.rb

class BookingItem < ActiveRecord::Base
  belongs_to :booking

  validate :availability_of_space

  # snip

  private

    def availability_of_space
      unless space.available_between? DateTime.parse("#{booking_date}##{start_time}"), DateTime.parse("#{booking_date}##{end_time}")
        errors[:base] << "The selected space is not available between those times."
      end
    end
end

app/views/booking/_form.html.erb

<% if @booking.errors.any? %>
  <div id="error_explanation">
    <p><%= pluralize(@booking.errors.count, "error") %> prohibited this booking from being saved:</p>
    <ul>
      <% @booking.errors.full_messages.each do |msg| %>
        <li><%= msg %></li>
      <% end %>
    </ul>
  </div>
<% end %>

<%= form_for(@booking, :html => { :class => "nice custom"}) do |f| %>
  ...
<% end %>

Answer 1

如果您使用 after_save，您的选择会受到一定限制。回调创建 BookingItem对象。

而不是使用 after_save , 我会使用 before_validation并进行一些调整以适应这种情况。

1) 构建BookingItem before_validation 中的对象打回来

before_validation :recreate_booking_items!

和

def recreate_booking_items!
  schedule.all_occurrences.each do |date|
    booking_items.build(......
  end
end

请注意，我使用的是 build而不是 create!
当您验证 Booking对象，新的 BookingItem booking_items 中的对象收集也将被验证。任何错误都将包含在主 Booking 中对象的错误集合，您可以像往常一样在 View 中显示它们，因为 Booking对象将无法保存。

笔记

1) BookingItem Booking 时自动验证对象。对象被验证，因为它们是新记录并且属于 has_many协会。如果它们被持久化(即已经在数据库中)，它们将不会被自动验证。

2) before_validation回调可以在对象的生命周期中多次调用，具体取决于您的代码。在这种情况下， BookingItem每次调用回调时都会构建对象，这将导致重复。为了防止这种情况，您可以在 recreate_booking_items! 的开头添加以下行:

booking_items.delete_all

当然，如果你坚持了 BookingItem，你可能不想这样做。数据库中的对象(见下文)。

3) 此代码明确设计用于创建 Booking对象。如果您正在编辑现有的 Booking已经持久化的对象 BookingItem对象，可能需要进行某些修改，具体取决于您所需的功能。

更新:

在下面的评论中解决@Simon 的后续问题。

我可以想到您可能想要这样做的两种方法:

1) 将验证保存在 BookingItem正如你所拥有的。

然后，我将在 Booking 中有一个自定义验证器像这样:

validate :validate_booking_items

def validate_booking_items
  booking_items.each do |bi|
    if bi.invalid?
      errors[:base] << "Booking item #{bi.<some property>} is invalid for <some reason>"
    end
  end
end

这会在 Booking 中添加一个很好的自定义消息对于每个无效的 BookingItem , 但它也给每个 BookingItem它自己的错误集合，您可以使用它来识别 booking_items无效。可以引用无效的 booking_items像这样:

@booking.booking_items.select {|bi| bi.errors.present?}

那么如果要显示无效的 booking_items在你看来:

f.fields_for :booking_items, f.object.booking_items.select {|bi| bi.errors.present? } do |bi|
end

这种方法的问题是 BookingItem可能由于多种原因而无效，并尝试将所有这些原因添加到基础 Booking 中错误收集可能会变得困惑。

因此，另一种方法:

2) 忘记 Booking 中的自定义验证器.依靠 Rails 对 has_many 的非持久成员的自动验证为每个 BookingItem 运行验证检查的集合目的。这将为他们每个人提供一个错误集合。

然后，在您看来，您可以遍历无效的 booking_items并显示他们各自的错误。

<ul>
  <% @booking.booking_items.select {|bi| bi.errors.present? }.each do |bi| %>
    <li>
      Booking item <%= bi.name %> could not be saved because:
      <ul>
        <% bi.errors.full_messages.each do |msg| %>
          <li><%= msg %></li>
        <% end %>
      </ul> 
    </li>
  <% end %>
</ul>

如果您使用这种方法，您的 Booking 中将出现一般的“预订项目无效”错误。对象错误集合，因此您可能希望以某种方式忽略它们，以免它们显示。

注意:我对 IceCube 不熟悉，但如果你显示的是 BookingItem表格中的对象通过 nested_attributes_for ，这可能与构建 BookingItem 发生冲突 before_validation 中的对象打回来。