github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-27 02:13:38 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

FIX: Don't stop the badge grant after the first failure

Browse Source
This commit is contained in:
riking 2015-03-28 18:36:05 -07:00
parent 260b94e7e1
commit 6ce26ab4d8
2 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
app/jobs/scheduled/badge_grant.rb
View File

@ -11,7 +11,12 @@ module Jobs
return unless SiteSetting.enable_badges return unless SiteSetting.enable_badges
Badge.all.each do |b| Badge.all.each do |b|
BadgeGranter.backfill(b) begin
BadgeGranter.backfill(b)
rescue => ex
# TODO - expose errors in UI
Discourse.handle_job_exception(ex, error_context({}, code_desc: 'Exception granting badges', extra: {badge_id: b.id}))
end
end end
BadgeGranter.revoke_ungranted_titles! BadgeGranter.revoke_ungranted_titles!

4
app/services/badge_granter.rb
View File

@ -120,7 +120,9 @@ class BadgeGranter
user_ids = list.map{|i| i["user_ids"]}.flatten.compact.uniq user_ids = list.map{|i| i["user_ids"]}.flatten.compact.uniq
next unless post_ids.present? || user_ids.present? next unless post_ids.present? || user_ids.present?
find_by_type(type).each{|badge| backfill(badge, post_ids: post_ids, user_ids: user_ids)} find_by_type(type).each{ |badge|
backfill(badge, post_ids: post_ids, user_ids: user_ids)
}
end end
end end