Merge branch 'preserve-second-parameter-new-note-worker' into 'master'

Preserve optional second parameter in NewNoteWorker jobs Closes #24678 See merge request !7602

Merge branch 'preserve-second-parameter-new-note-worker' into 'master'
d2340d88 · Sean McGivern · 0f590750 · 99432cbc · d2340d88
Commit d2340d88 authored Nov 22, 2016 by Sean McGivern
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

new_note_worker.rb app/workers/new_note_worker.rb +3 -1

No files found.
--- 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