Download as
Browse Code »

Commit 2d20892070afba8c59258c1dd73c69d88e556d6c

Authored by Antonio Terceiro
1 parent 762b32cb
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

ActionItem1165: being resilient to profile removal. 

Don't try to expire cache if either side of the friendship does not exist
anymore.
Showing 1 changed file with 5 additions and 2 deletions   Show diff stats
app/sweepers/friendship_sweeper.rb
  Diff comments   View file @2d20892
... ... @@ -13,8 +13,11 @@ class FriendshipSweeper < ActiveRecord::Observer
13 13 protected
14 14  
15 15 def expire_caches(friendship)
16   - expire_cache(friendship.person)
17   - expire_cache(friendship.friend)
  16 + [friendship.person, friendship.friend].each do |profile|
  17 + if profile
  18 + expire_cache(profile)
  19 + end
  20 + end
18 21 end
19 22  
20 23 def expire_cache(profile)
... ...