ActiveRecord :: Base before_validation mit bedingten nicht ausgelöst

Question

Ich verwende before_validation, um den Zustand meiner User Modell zu gewährleisten.

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

Die Bedingung ist wahr, wenn ich mit einem neuen Rekord versuchte.

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

Und der Rückruf funktioniert perfekt ohne die bedingte.

Die Verwendung des Rückrufs mit der Bedingung funktioniert jedoch nicht.

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 

Nur um zu klären, die renter Methode führt Folgendes aus:

def renter 
    self.resident_type = :renter 
    self 
end 

def renter! 
    renter.save 
end 

Gibt es etwas, dass ich fehle?

Ich vermute, zwei Dinge, 1) die bedingte und 2) die Rückkehr von renter und mir missverstanden, wie before_validation funktioniert.

Answer 1

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

Mein zweiter Verdacht richtig war. before_validation erwartet truthy oder falsch.

Answer 2

Sie dies versuchen kann ...

before_validation :renter, if: :check_renter? 

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

Answer 3

IMO Ihr Rückruf und Ihre renter Methode sind schwer zu lesen und zu verstehen. Ich denke es lohnt sich, diesen Code in eine eigene Methode zu verschieben:

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