has_many 关联的验证错误不会阻止保存

Question

我有一个模特 eclub，有很多邀请，我想限制可以添加的邀请数量。我的模型是这样的

class Eclub < ActiveRecord::Base
    has_many :invites, dependent: :destroy
    validates :invites, length: {maximum: 50, message: 'The maximum number of invites have been sent' }

根据这个SO answer 应该可以工作，前提是我不关心标记为销毁的邀请。我的 rspec 测试首先创建了一个 eclub 并成功添加了 50 个邀请。那么

  invite = Invite.new(name: 'Too Many', email: 'extra@gmail.com')
  eclub.invites << invite
  expect(eclub).to be_invalid
  expect(eclub.errors[:invites].first).to include 'maximum number'
  expect(eclub.reload.invites.size).to eq 50

前两个期望通过，但最后一个失败了

Failure/Error: expect(eclub.reload.invites.size).to eq 50

       expected: 50
            got: 51

如何防止额外的邀请被添加到集合中？

Answer 1

添加此验证的最佳位置不是Eclub，而是Invite 模型。试试这个：

class Invite < ActiveRecord::Base
  belongs_to :eclub
  validate :check_invite_count!, on: :create

  private

  def check_invite_count!
    return if eclub.nil?
    if eclub.invites.count > 50
      errors.add(:base, 'Cannot add more than 50 invites for an Eclub')
    end
  end
end

此外，此代码 validates :invites, length: {maximum: 50, message: 'The maximum number of invites have been sent' } 将不起作用，因为它假定 invites 是一个字符串。

Answer 2

validates length 用于测量字符串的长度，可能不适用于整数。

你可能想试试validates :numericality => {:maximum => 50}

编辑 是的，我在这里可能不会工作，因为它是我们需要验证的邀请计数（而不是邀请关联本身），所以这里有一个自定义验证器，应该可以满足您的需求：

validate :invite_count_is_ok

private

def invite_count_is_ok
  return unless invites.present?
  errors.add(:base, "You may only have up to 50 invites at a time") unless invites.count <= 50
end

Answer 3

validates(
  :invites,
  numericality: {
    greater_than_or_equal_to: 0,
    less_than_or_equal_to: 50
  }
)