From 1974db27b665956e043ac5926b7b415a7764b767 Mon Sep 17 00:00:00 2001 From: r888888888 Date: Fri, 12 Aug 2016 13:34:02 -0700 Subject: [PATCH] consolidate update_column calls on posts --- app/models/post.rb | 26 +++++++++++++++++--------- 1 file changed, 17 insertions(+), 9 deletions(-) diff --git a/app/models/post.rb b/app/models/post.rb index 4e9097e03..5d7466ff5 100644 --- a/app/models/post.rb +++ b/app/models/post.rb @@ -1022,11 +1022,13 @@ class Post < ActiveRecord::Base module CountMethods def fix_post_counts post.set_tag_counts - post.update_column(:tag_count, post.tag_count) - post.update_column(:tag_count_general, post.tag_count_general) - post.update_column(:tag_count_artist, post.tag_count_artist) - post.update_column(:tag_count_copyright, post.tag_count_copyright) - post.update_column(:tag_count_character, post.tag_count_character) + post.update_columns( + :tag_count => post.tag_count, + :tag_count_general => post.tag_count_general, + :tag_count_artist => post.tag_count_artist, + :tag_count_copyright => post.tag_count_copyright, + :tag_count_character => post.tag_count_character + ) end def get_count_from_cache(tags) @@ -1292,10 +1294,16 @@ class Post < ActiveRecord::Base end Post.transaction do - update_column(:is_deleted, true) - update_column(:is_pending, false) - update_column(:is_flagged, false) - update_column(:is_banned, true) if options[:ban] || has_tag?("banned_artist") + self.is_deleted = true + self.is_pending = false + self.is_flagged = false + self.is_banned = true if options[:ban] || has_tag?("banned_artist") + update_columns( + :is_deleted => is_deleted, + :is_pending => is_pending, + :is_flagged => is_flagged, + :is_banned => is_banned + ) give_favorites_to_parent if options[:move_favorites] update_parent_on_save