unschedule(nil) - unschedules the first scheduled job about sidekiq-status HOT 5 CLOSED

Got you, thanks

from sidekiq-status.

+1

from sidekiq-status.

Lost a while hunting this bug, +1 on a fix.

from sidekiq-status.

Temporary workaround

Sidekiq::Status.unschedule(job_id) if job_id

Literally came here to see if anyone had already submitted this bug. Nice one

from sidekiq-status.

@utgarda - Fixed this issue in PR #69.
Problem was with following line

scheduled_jobs.each { |job_listing| (return job_listing) if job_listing.include?("\"jid\":\"#{job_id}") }

Above line was checking patterns inclusion only, so following was getting true if unschedule(nil)
Ex: job_listing

"{\"retry\":false,\"queue\":\"default\",\"backtrace\":true,\"expires_in\":86400,\"class\":\"MyWorker\",\"args\":[10],\"jid\":\"a28ba4cc9b0396da3fb2c410\",\"enqueued_at\":1449306022.4224896}" 

job_listing.include?("\"jid\":\"#{nil}")
=> \"jid\":\" 
# resulting in true as \"jid\":\"a28ba4cc9b0396da3fb2c410\" has inclusion of \"jid\":\"

Replaced inclusion check with this, Additional I have tested this change and it will handle all cases like unschedule(nil), unschedule('') and unschedule(other_jid).

from sidekiq-status.