Download as
Browse Code »

Commit 9ef7a07492b1ae51a906ca3341e1056e710629bc

Authored by Daniela Feitosa
Committed by Antonio Terceiro
1 parent c54ef437
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

ActionItem1197: expiring statistics block cache 

When a new profile is created, statistics block should be updated
Showing 2 changed files with 14 additions and 0 deletions   Show diff stats
app/models/profile.rb
  Diff comments   View file @9ef7a07
... ... @@ -638,4 +638,9 @@ class Profile < ActiveRecord::Base
638 638 ProfileSweeper.new().after_update(profile)
639 639 end
640 640  
  641 + # FIXME: horrible workaround to circular dependancy in environment.rb
  642 + after_create do |profile|
  643 + ProfileSweeper.new().after_create(profile)
  644 + end
  645 +
641 646 end
... ...
app/sweepers/profile_sweeper.rb
  Diff comments   View file @9ef7a07
... ... @@ -7,6 +7,10 @@ class ProfileSweeper # < ActiveRecord::Observer
7 7 expire_caches(profile)
8 8 end
9 9  
  10 + def after_create(profile)
  11 + expire_statistics_block_cache(profile)
  12 + end
  13 +
10 14 protected
11 15  
12 16 def expire_caches(profile)
... ... @@ -19,4 +23,9 @@ protected
19 23 expire_timeout_fragment(block.cache_keys)
20 24 end
21 25 end
  26 +
  27 + def expire_statistics_block_cache(profile)
  28 + blocks = profile.environment.blocks.select { |b| b.kind_of?(EnvironmentStatisticsBlock) }
  29 + blocks.map(&:cache_keys).each{|ck|expire_timeout_fragment(ck)}
  30 + end
22 31 end
... ...