11

我有一个看起来像这样的 ActiveRecord 类。

class Foo
  belongs_to :bar, autosave: true
  before_save :modify_bar
  ...
end

如果我做一些记录,我会看到bar正在修改,但它的更改没有保存。怎么了?

4

2 回答 2

24

这里的问题是autosave: true简单地设置一个正常的before_save回调,并且before_save回调按照它们被创建的顺序运行。**

因此,它尝试保存bar没有更改的 ,然后调用modify_bar

解决方案是确保modify_bar回调在自动保存之前运行。

一种方法是使用prepend选项。

class Foo
  belongs_to :bar, autosave: true
  before_save :modify_bar, prepend: true
  ...
end

另一种方法是将before_save声明放在belongs_to.

另一种方法是bar在方法结束时显式保存modify_bar并且根本不使用该autosave选项。

感谢 Danny Burkes 的有用博客文章

** 此外,它们在所有after_validation回调之后和任何before_create回调之前运行 -请参阅文档


更新

这是检查此类回调顺序的一种方法。

  describe "sequence of callbacks" do

    let(:sequence_checker) { SequenceChecker.new }

    before :each do
      foo.stub(:bar).and_return(sequence_checker)
    end

    it "modifies bar before saving it" do
      # Run the before_save callbacks and halt before actually saving
      foo.run_callbacks(:save) { false }
      # Test one of the following
      #
      # If only these methods should have been called
      expect(sequence_checker.called_methods).to eq(%w[modify save])
      # If there may be other methods called in between
      expect(sequence_checker.received_in_order?('modify', 'save')).to be_true
    end

  end

使用这个支持类:

class SequenceChecker
  attr_accessor :called_methods

  def initialize
    self.called_methods = []
  end

  def method_missing(method_name, *args)
    called_methods << method_name.to_s
  end

  def received_in_order?(*expected_methods)
    expected_methods.map!(&:to_s)
    called_methods & expected_methods == expected_methods
  end

end
于 2013-02-21T21:24:02.337 回答
1

上述答案(显然)是您的解决方案。然而:

我很好用:autosave,但我不认为改变外部关联是回调的工作。我说的是你的:modify_bar。正如这篇文章中精彩解释的那样, 我更喜欢使用另一个对象一次保存多个模型。当事情变得更复杂时,它确实简化了您的生活,并使测试变得更加容易。

在这种情况下,这可以在控制器中或从服务对象中完成。

于 2015-11-02T12:44:51.267 回答