maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Cache bust banished users' posts after the posts are deleted (#3044)

Browse source 
* Bust article caches after they've been deleted

* Use less instances of cachebuster class
This commit is contained in:
Andy Zhao 2019-06-04 15:04:30 -04:00 • committed by Ben Halpern
parent 40832602ce
commit a55bce99ac
1 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
app/services/moderator/manage_activity_and_roles.rb
View file

@ -15,9 +15,10 @@ module Moderator
def delete_comments
return unless user.comments.any?
cachebuster = CacheBuster.new
user.comments.find_each do |comment|
comment.reactions.delete_all
CacheBuster.new.bust_comment(comment.commentable, user.username)
cachebuster.bust_comment(comment.commentable, user.username)
comment.delete
comment.remove_notifications
end
@ -26,17 +27,21 @@ module Moderator
def delete_articles
return unless user.articles.any?
cachebuster = CacheBuster.new
virtual_articles = user.articles.map { |article| Article.new(article.attributes) }
user.articles.find_each do |article|
article.reactions.delete_all
article.comments.includes(:user).find_each do |comment|
comment.reactions.delete_all
CacheBuster.new.bust_comment(comment.commentable, comment.user.username)
cachebuster.bust_comment(comment.commentable, comment.user.username)
comment.delete
end
CacheBuster.new.bust_article(article)
article.remove_algolia_index
article.delete
end
virtual_articles.each do |article|
cachebuster.bust_article(article)
end
end
def delete_user_activity