Merge pull request #69 from joshsoftware/issue-17

Fixed unschedule(nil) bug - fixes #17

README.md

@@ -120,8 +120,14 @@ Sidekiq::Status::pct_complete job_id #=> 5
 ```ruby
 scheduled_job_id = MyJob.perform_in 3600
 Sidekiq::Status.cancel scheduled_job_id #=> true
-#doesn't cancel running jobs, this is more like unscheduling, therefore an alias:
+# doesn't cancel running jobs, this is more like unscheduling, therefore an alias:
 Sidekiq::Status.unschedule scheduled_job_id #=> true
+
+# returns false if invalid or wrong scheduled_job_id is provided 
+Sidekiq::Status.unschedule some_other_unschedule_job_id #=> false
+Sidekiq::Status.unschedule nil #=> false
+Sidekiq::Status.unschedule '' #=> false
+# Note: cancel and unschedule are alias methods.
 ```
 
 ### Sidekiq web integration

lib/sidekiq-status/storage.rb

@@ -91,8 +91,7 @@ def scan_scheduled_jobs_for_jid(scheduled_jobs, job_id)
     # A Little skecthy, I know, but the structure of these internal JSON
     # is predefined in such a way where this will not catch unintentional elements,
     # and this is notably faster than performing JSON.parse() for every listing:
-    scheduled_jobs.each { |job_listing| (return job_listing) if job_listing.include?("\"jid\":\"#{job_id}") }
-    nil
+    scheduled_jobs.select { |job_listing| job_listing.match(/\"jid\":\"#{job_id}\"/) }[0]
   end
 
   # Yields redis connection. Uses redis pool if available.