Блог

У меня есть эта модель

 class User < ActiveRecord::Base
  attr_accessible :type, :school_name, :school_grade

  validates :type, :inclusion =>{:in =>['1','2']}, :allow_blank => true
  validates :school_name,  :presence =>{:if => :student?}
  validates :school_grade, :numericality =>{:only_integer =>true}, :allow_blank => {:unless => :student?}

  def student?
    type.to_s == '1'
  end
end
  

Я попробовал следующие шаблоны.

 User.create(:type =>'2', :school_name =>nil, :school_grade =>nil)
  

=> OK. Ошибки не произошло.

 User.create(:type =>'1', :school_name =>nil, :school_grade =>1)
  

=> OK. Произошла ошибка ActiveRecord::RecordInvalid «имя_школы пусто».

 User.create(:type =>'1', :school_name =>'foo', :school_grade =>'string is not a number')
  

=> OK. Произошла ошибка проверки «school_grade не является числом».

 User.create(:type =>'1', :school_name =>'foo', :school_grade =>nil)
  

=> NG. Я ожидаю ошибку «school_grade — это не число», но ошибки не произошло.

Я тоже пробовал этот шаблон, но получил тот же результат.

   validates :school_grade, :numericality =>{:only_integer =>true}, :allow_blank => {:if => :not_student?}

  def not_student?
    type.to_s != '1'
  end
  

Я думаю, проверка: allow_blank всегда активна, но почему? И как это исправить?

Спасибо.

allow_blank не принимает Hash , только логическое значение, поэтому вы не можете написать что-то подобное :allow_blank => {:unless => :student?} , как вы можете видеть в исходном activemodel-4.1.2/lib/active_model/validator.rb

 def validate(record)
  attributes.each do |attribute|
    value = record.read_attribute_for_validation(attribute)
    next if (value.nil? amp;amp; options[:allow_nil]) || (value.blank? amp;amp; options[:allow_blank])
    validate_each(record, attribute, value)
  end
end
  

Однако то, чего вы хотите достичь, может быть сделано в двух строках:

 class User < ActiveRecord::Base
  validates_presence_of :school_grade, :if => :student?
  validates :school_grade, :numericality =>{:only_integer =>true}, :allow_blank => true
end
  

Вы хотите, чтобы allow_blank был условным или чтобы проверка всей численности была полной?

Если последнее:

 validates :school_grade, :numericality =>{:only_integer =>true}, :allow_blank => true, :unless => :student?
validates :school_grade, :numericality =>{:only_integer =>true}, :allow_blank => false, :if => :student?