Merge pull request #115 from sebastianludwig/master

Fixed variable naming inconsistency
arangamani · Nov 28, 2013 · 9bcbd73 · 9bcbd73
2 parents 98d8f27 + c3e3afd
commit 9bcbd73
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/jenkins_api_client/job.rb b/lib/jenkins_api_client/job.rb
@@ -916,7 +916,7 @@ def get_build_id_the_old_way(expected_build_id, opts)
               # Jenkins will return 404's until the job starts
               begin
                 get_build_details(job_name, expected_build_id)
-                completion_proc.call(expected_build_number, false) if completion_proc
+                completion_proc.call(expected_build_id, false) if completion_proc
 
                 return expected_build_id
               rescue JenkinsApi::Exceptions::NotFound => e
@@ -940,7 +940,7 @@ def get_build_id_the_old_way(expected_build_id, opts)
         rescue JenkinsApi::Exceptions::ApiException => e
           completion_proc.call(nil, false) if completion_proc
           # Jenkins Api threw an error at us
-          @logger.warn "Problem while waiting for '#{job_name}' build ##{expected_build_number} to start.  #{e.class} #{e}"
+          @logger.warn "Problem while waiting for '#{job_name}' build ##{expected_build_id} to start.  #{e.class} #{e}"
           raise
         end
       end