PERF: don't count all likes twice on like

discourse · Jun 25, 2015 · 1edb591 · 1edb591
1 parent 8252f4e
commit 1edb591
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/app/models/post_action.rb b/app/models/post_action.rb
@@ -280,9 +280,12 @@ def self.act(user, post, post_action_type_id, opts = {})
     end
 
     # agree with other flags
-    PostAction.agree_flags!(post, user) if staff_took_action
-    # update counters
-    post_action.try(:update_counters)
+    if staff_took_action
+      PostAction.agree_flags!(post, user)
+
+      # update counters
+      post_action.try(:update_counters)
+    end
 
     post_action
   rescue ActiveRecord::RecordNotUnique