diff --git a/app/models/favorite.rb b/app/models/favorite.rb index 65eb9f158..06bd08bb8 100644 --- a/app/models/favorite.rb +++ b/app/models/favorite.rb @@ -14,13 +14,10 @@ class Favorite < ActiveRecord::Base def self.add(post, user) Favorite.transaction do - return if Favorite.for_user(user.id).where(:user_id => user.id, :post_id => post.id).lock("FOR UPDATE NOWAIT").exists? + return if Favorite.for_user(user.id).where(:user_id => user.id, :post_id => post.id).exists? Favorite.create!(:user_id => user.id, :post_id => post.id) - changes = "fav_count = fav_count + 1" - if user.is_gold? - changes = "#{changes}, score = score + 1" - end - Post.where(:id => post.id).update_all(changes) + Post.where(:id => post.id).update_all("fav_count = fav_count + 1") + Post.where(:id => post.id).update_all("score = score + 1") if user.is_gold? post.append_user_to_fav_string(user.id) User.where(:id => user.id).update_all("favorite_count = favorite_count + 1") user.favorite_count += 1 @@ -31,7 +28,7 @@ class Favorite < ActiveRecord::Base def self.remove(post, user) Favorite.transaction do - return unless Favorite.for_user(user.id).where(:user_id => user.id, :post_id => post.id).lock("FOR UPDATE NOWAIT").exists? + return unless Favorite.for_user(user.id).where(:user_id => user.id, :post_id => post.id).exists? Favorite.destroy_all(:user_id => user.id, :post_id => post.id) Post.where(:id => post.id).update_all("fav_count = fav_count - 1") Post.where(:id => post.id).update_all("Score = score - 1") if user.is_gold?