Refactor sidekiq adapter enqueue and enqueue_at methods

rails · May 8, 2015 · e744cfe · e744cfe
1 parent 0834d60
commit e744cfe
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/activejob/lib/active_job/queue_adapters/sidekiq_adapter.rb b/activejob/lib/active_job/queue_adapters/sidekiq_adapter.rb
@@ -17,24 +17,20 @@ module QueueAdapters
     class SidekiqAdapter
       def enqueue(job) #:nodoc:
         #Sidekiq::Client does not support symbols as keys
-        sidekiq_job_id = Sidekiq::Client.push \
+        job.provider_job_id = Sidekiq::Client.push \
           'class'   => JobWrapper,
           'wrapped' => job.class.to_s,
           'queue'   => job.queue_name,
           'args'    => [ job.serialize ]
-        job.provider_job_id = sidekiq_job_id
-        sidekiq_job_id
       end
 
       def enqueue_at(job, timestamp) #:nodoc:
-        sidekiq_job_id = Sidekiq::Client.push \
+        job.provider_job_id = Sidekiq::Client.push \
           'class'   => JobWrapper,
           'wrapped' => job.class.to_s,
           'queue'   => job.queue_name,
           'args'    => [ job.serialize ],
           'at'      => timestamp
-        job.provider_job_id = sidekiq_job_id
-        sidekiq_job_id
       end
 
       class JobWrapper #:nodoc: