ActiveRecord :: Base before_validation with conditional not triggered

Question

我使用before_validation來確保我的User模型的狀態。

before_validation :renter, if: 'resident? && active? && unit.present? && units.empty?' 

當我嘗試使用新記錄時，條件爲真。

user.resident? && user.active? && user.unit.present? && user.units.empty? 
=> true 

而且回調在沒有條件的情況下完美運作。

但是，使用具有條件的回調不起作用。

user = User.new(resident_attr_except_resident_type) 
user.save 
=> false 
user.errors.full_messages 
=> ["Resident type can't be blank"] 

user.renter 
=> #<User ..., resident_type: 0> # resident_type 0 because is a enum 
user.save 
=> true 

只是爲了澄清，該renter方法執行以下操作：

def renter 
    self.resident_type = :renter 
    self 
end 

def renter! 
    renter.save 
end 

這有什麼，我很想念？

我懷疑兩件事1）有條件的和2）renter的回報和我誤解before_validation如何工作。

Answer 1

http://api.rubyonrails.org/classes/ActiveRecord/Callbacks.html#module-ActiveRecord::Callbacks-label-before_validation-2A+returning+statements

我的第二個懷疑是正確的。 before_validation預計真摯或虛假。

Answer 2

，你可以嘗試這個...

before_validation :renter, if: :check_renter? 

def check_renter? 
    resident? && active? && unit.present? && units.empty? 
end 

Answer 3

IMO你的回調和你的renter方法很難閱讀和理解。我認爲將代碼轉移到自己的方法是值得的：

before_validation :determine_renter_type 

private 
def detemine_renter_type 
    if resident? && active? && unit.present? && units.empty? 
    self.renter_type = User.renter_types[:renter] 
    end 

    self # avoid aborting the save process by returning a truthy value 
end