diff options
author | Mark Longair <mhl@pobox.com> | 2013-08-19 10:34:05 +0100 |
---|---|---|
committer | Mark Longair <mhl@pobox.com> | 2013-08-20 12:11:45 +0100 |
commit | e5855f7fd2657574c5af99890c63d530ca3bb5d0 (patch) | |
tree | 338b64fd532af727027ada82b7a4d2042855e305 /lib/configuration.rb | |
parent | 32625f08b9c7e7bb65bd24099e64d7d17be5e45e (diff) |
Improve calculation of PublicBody statistics columns
On PublicBody, we don't need to update info_requests_count
because that's already done with :counter_cache. On the
other hand, info_requests_successful_count and
info_requests_not_held_count can't be updated easily with
counter_cache (since they need conditions to be attached).
Instead we update them in post_save and post_destroy,
as suggested here:
http://blog.douglasfshearer.com/post/17495285851/custom-counter-cache-with-conditions
This also adds tests to ensure that the
after_(save|destroy) callbacks are called and that they
modify the counts correctly.
Diffstat (limited to 'lib/configuration.rb')
0 files changed, 0 insertions, 0 deletions