diff options
author | Oswaldo Ferreira <oswluizf@gmail.com> | 2016-10-13 12:26:44 -0300 |
---|---|---|
committer | Oswaldo Ferreira <oswluizf@gmail.com> | 2016-11-11 22:54:11 -0200 |
commit | d48d879ef5e0b1517c43bef27f584655535259c8 (patch) | |
tree | 0d0477d0b4f35232fceb8ba83548c29187603caa /app/workers | |
parent | 6eeff67c6e03233d4480a55d05d4e0f1a88aef4c (diff) | |
download | gitlab-ce-d48d879ef5e0b1517c43bef27f584655535259c8.tar.gz |
Does not raise error when Note not found when processing NewNoteWorker
- Also remove unnecessary param
Diffstat (limited to 'app/workers')
-rw-r--r-- | app/workers/new_note_worker.rb | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/app/workers/new_note_worker.rb b/app/workers/new_note_worker.rb index c3e62bb88c0..66574d0fd01 100644 --- a/app/workers/new_note_worker.rb +++ b/app/workers/new_note_worker.rb @@ -2,10 +2,12 @@ class NewNoteWorker include Sidekiq::Worker include DedicatedSidekiqQueue - def perform(note_id, note_params) - note = Note.find(note_id) - - NotificationService.new.new_note(note) - Notes::PostProcessService.new(note).execute + def perform(note_id) + if note = Note.find_by(id: note_id) + NotificationService.new.new_note(note) + Notes::PostProcessService.new(note).execute + else + Rails.logger.error("NewNoteWorker: couldn't find note with ID=#{note_id}, skipping job") + end end end |