ist: on =>: gültig für einen Rückruf vor dem Speichern in Rails 3.2.3

Question

Wie Sie wissen, werden before_save Rückrufe vor before_create Rückrufe ausgeführt.

Daher haben einige Leute vorgeschlagen, dass in besser wäre before_save :method, :on => :create anstelle von before_create zu verwenden, so dass die Callback-Methode zur richtigen Zeit in Bezug auf andere Rückrufe (wie Autosave-Rückrufe) ausgeführt wird. Siehe beispielsweise Pivotal Labs blog post und this StackOverflow answer.

Allerdings, so weit ich sagen kann, erreicht die :on => :create Option nicht den gewünschten Effekt auf einen Rückruf before_save. Mit anderen Worten, der Rückruf wird für jeden Speichervorgang ausgeführt, unabhängig davon, ob er erstellt wurde oder nicht.

Die :on => :create Option tut für before_validation Rückrufe als gültig erscheinen, though.

Könnte jemand bestätigen, ob die :on => :create für eine before_save arbeiten soll? Hat es in früheren Versionen von Rails funktioniert und ist jetzt defekt, oder sind die oben genannten Links einfach falsch?

Angenommen, :on => :create ist nicht gültig, ist das folgende akzeptabel, und/oder gibt es einen besseren Weg?

before_save :callback_method, :if => :new_record? 

Vielen Dank.

Answer 1

Sie haben recht, es gibt keine :on Option für before_save Rückruf. Aber ich verstehe nicht, warum before_save anstelle von before_create verwenden. before_create Rückruf wird direkt nach before_save aufgerufen werden.

Natürlich können Sie before_save :callback_method, :if => :new_record? verwenden. Aber ich persönlich mag diese Lösung nicht - was, wenn ich Bedingungen in der :if Option hinzufügen muss?

Wenn eine Abhängigkeit zwischen before_save und before_create Callbacks haben, würde ich vorschlagen, 2 Callbacks zu kombinieren. Zum Beispiel (Pseudo-Code):

class MyModel < ActiveRecord::Base 
    before_create :prepare_x 
    before_save :do_something_with_x 

    def prepare_x 
    @x = 10 
    end 


    # will not work, because `prepare_x` called after `do_something_with_x` 
    def do_something_with_x 
    @a = 100/@x 
    end 
end 

# || 
# || 
# \/ 

class MyModel < ActiveRecord::Base 

    before_save :do_something_with_x 

    def do_something_with_x 
    @x = 10 if new_record? 
    @a = 100/@x 
    end 
end