diff options
author | Sean McGivern <sean@mcgivern.me.uk> | 2016-11-22 09:55:53 +0000 |
---|---|---|
committer | Alejandro RodrÃguez <alejorro70@gmail.com> | 2016-11-22 16:34:20 +0000 |
commit | cf7239ab4d23f257fd07fa9d1900b3372eacbc3f (patch) | |
tree | ea7248b78d598d3f7e8c5204a85440ba811423b2 | |
parent | e084cbd978f22704484edb89f0663c1d56cbf338 (diff) | |
download | gitlab-ce-cf7239ab4d23f257fd07fa9d1900b3372eacbc3f.tar.gz |
Merge branch 'preserve-second-parameter-new-note-worker' into 'master'
Preserve optional second parameter in NewNoteWorker jobs
Closes #24678
See merge request !7602
-rw-r--r-- | app/workers/new_note_worker.rb | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/app/workers/new_note_worker.rb b/app/workers/new_note_worker.rb index 66574d0fd01..926162b8c53 100644 --- a/app/workers/new_note_worker.rb +++ b/app/workers/new_note_worker.rb @@ -2,7 +2,9 @@ class NewNoteWorker include Sidekiq::Worker include DedicatedSidekiqQueue - def perform(note_id) + # Keep extra parameter to preserve backwards compatibility with + # old `NewNoteWorker` jobs (can remove later) + def perform(note_id, _params = {}) if note = Note.find_by(id: note_id) NotificationService.new.new_note(note) Notes::PostProcessService.new(note).execute |