Rails counter_cache 未正确更新

Question

使用 Rails 3.1.3，我试图找出为什么在通过 update_attributes 更改父记录 id 时我们的计数器缓存没有正确更新。

class ExhibitorRegistration < ActiveRecord::Base
  belongs_to :event, :counter_cache => true
end

class Event < ActiveRecord::Base
  has_many :exhibitor_registrations, :dependent => :destroy
end

describe ExhibitorRegistration do
  it 'correctly maintains the counter cache on events' do
    event = Factory(:event)
    other_event = Factory(:event)
    registration = Factory(:exhibitor_registration, :event => event)

    event.reload
    event.exhibitor_registrations_count.should == 1

    registration.update_attributes(:event_id => other_event.id)

    event.reload
    event.exhibitor_registrations_count.should == 0

    other_event.reload
    other_event.exhibitor_registrations_count.should == 1
  end
end

此规范失败，表明事件上的计数器缓存没有减少。

1) ExhibitorRegistration correctly maintains the counter cache on events
   Failure/Error: event.exhibitor_registrations_count.should == 0
     expected: 0
          got: 1 (using ==)

我是否应该期望这能起作用，或者我是否需要手动跟踪更改并自己更新计数器？

Answer 1

来自精细手册：

：counter_cache

通过使用 increment_counter 和 decrement_counter 缓存关联类上所属对象的数量。当此类的对象被创建时，计数器缓存会递增；当该类的对象被销毁时，计数器缓存会递减。

没有提到当对象从一个所有者移动到另一个所有者时更新缓存。当然，Rails 文档通常不完整，因此我们必须查看源代码进行确认。当你说 :counter_cache =>; true，您触发对私人的呼叫add_counter_cache_callbacks 方法 和 add_counter_cache_callbacks 执行此操作：

1. 添加一个 after_create 回调，该回调调用 increment_counter。
2. 添加一个 before_destroy 回调，调用 减量计数器。
3. 调用 attr_readonly使计数器列只读。

我不认为您期望太多，您只是期望 ActiveRecord 比实际情况更完整。

不过，一切并没有丢失，您可以自己填补缺失的部分，而不需要太多的努力。如果您想允许重新设置父级并更新计数器，您可以向您的 ExhibitorRegistration 添加一个 before_save 回调来调整计数器本身，如下所示（未经测试的演示代码）：

class ExhibitorRegistration < ActiveRecord::Base
  belongs_to :event, :counter_cache => true
  before_save :fix_counter_cache, :if => ->(er) { !er.new_record? && er.event_id_changed? }

private

  def fix_counter_cache
    Event.decrement_counter(:exhibitor_registration_count, self.event_id_was)
    Event.increment_counter(:exhibitor_registration_count, self.event_id)
  end

end

如果您喜欢冒险，您可以将类似的内容修补到 ActiveRecord::Associations::Builder#add_counter_cache_callbacks 中并提交补丁。您所期望的行为是合理的，我认为 ActiveRecord 支持它是有意义的。

Answer 2

如果您的计数器已损坏或者您直接通过 SQL 对其进行了修改，则可以修复它。

使用：

ModelName.reset_counters(id_of_the_object_having_corrupted_count, one_or_many_counters)

示例 1：重新计算 id = 17 的帖子的缓存计数。

Post.reset_counters(17, :comments)

来源

示例 2：重新计算所有文章的缓存计数。

Article.ids.each { |id| Article.reset_counters(id, :comments) }

Answer 3

我最近遇到了同样的问题（Rails 3.2.3）。看起来它还没有修复，所以我必须继续进行修复。下面是我如何修改 ActiveRecord::Base 并利用 after_update 回调来保持 counter_cache 同步。

扩展 ActiveRecord::Base

使用以下内容创建一个新文件 lib/fix_counters_update.rb：

module FixUpdateCounters

  def fix_updated_counters
    self.changes.each {|key, value|
      # key should match /master_files_id/ or /bibls_id/
      # value should be an array ['old value', 'new value']
      if key =~ /_id/
        changed_class = key.sub(/_id/, '')
        changed_class.camelcase.constantize.decrement_counter(:"#{self.class.name.underscore.pluralize}_count", value[0]) unless value[0] == nil
        changed_class.camelcase.constantize.increment_counter(:"#{self.class.name.underscore.pluralize}_count", value[1]) unless value[1] == nil
      end
    }
  end 
end

ActiveRecord::Base.send(:include, FixUpdateCounters)

上面的代码使用 ActiveModel::Dirty 方法 changes 返回包含已更改属性的哈希以及包含旧值和新值的数组。通过测试该属性是否是关系（即以 /_id/ 结尾），您可以有条件地确定是否需要运行 decrement_counter 和/或 increment_counter。测试数组中是否存在 nil 至关重要，否则会导致错误。

添加到初始值设定项

创建一个新文件 config/initializers/active_record_extensions.rb，其中包含以下内容：

require 'fix_update_counters'

添加到模型< /strong>

对于您想要更新计数器缓存的每个模型，添加回调：

class Comment < ActiveRecord::Base
  after_update :fix_updated_counters
  ....
end

Answer 4

此问题的修复已合并到活动记录主

https://github.com/rails/rails /问题/9722

Answer 5

counter_cache 函数旨在通过关联名称而不是基础 id 列进行工作。在您的测试中，而不是：

registration.update_attributes(:event_id => other_event.id)

尝试

registration.update_attributes(:event => other_event)

更多信息可以在此处找到：http:// api.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html