diff --git a/app/services/badge_granter.rb b/app/services/badge_granter.rb index b132c7af184..2bca12c518f 100644 --- a/app/services/badge_granter.rb +++ b/app/services/badge_granter.rb @@ -15,7 +15,7 @@ class BadgeGranter user_badge = UserBadge.find_by(badge_id: @badge.id, user_id: @user.id, post_id: @post_id) - if user_badge.nil? || @badge.multiple_grant? + if user_badge.nil? || (@badge.multiple_grant? && @post_id.nil?) UserBadge.transaction do user_badge = UserBadge.create!(badge: @badge, user: @user, granted_by: @granted_by, diff --git a/spec/services/badge_granter_spec.rb b/spec/services/badge_granter_spec.rb index f893bd42cf6..a8f8499ff20 100644 --- a/spec/services/badge_granter_spec.rb +++ b/spec/services/badge_granter_spec.rb @@ -90,7 +90,9 @@ describe BadgeGranter do # Nice post badge post.update_attributes like_count: 10 BadgeGranter.update_badges(action: :post_like, post_id: post.id) + BadgeGranter.update_badges(action: :post_like, post_id: post.id) UserBadge.find_by(user_id: user.id, badge_id: 6).should_not be_nil + UserBadge.where(user_id: user.id, badge_id: 6).count.should == 1 # Good post badge post.update_attributes like_count: 25 BadgeGranter.update_badges(action: :post_like, post_id: post.id)