suzanne.soy/travis-api
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Assign variable before if statement

Browse Source
This commit is contained in:
Ana Rosas 2016-05-30 18:10:11 +02:00
parent 8655fd19ce
commit 0038197aab
1 changed files with 13 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
lib/travis/api/app/endpoint/builds.rb
View File

@ -49,24 +49,24 @@ class Travis::Api::App
post '/:id/restart' do post '/:id/restart' do
Metriks.meter("api.request.restart_build").mark Metriks.meter("api.request.restart_build").mark
if Travis::Features.owner_active?(:enqueue_to_hub, current_user) service = if Travis::Features.owner_active?(:enqueue_to_hub, current_user)
service = Travis::Enqueue::Services::RestartModel.new(current_user, { build_id: params[:id] }) Travis::Enqueue::Services::RestartModel.new(current_user, build_id: params[:id])
else else
service = self.service(:reset_model, build_id: params[:id]) self.service(:reset_model, build_id: params[:id])
end end
if !service.accept? result = if !service.accept?
status 400 status 400
result = false false
else elsif service.respond_to?(:push)
if service.respond_to?(:push) payload = { id: params[:id], user_id: current_user.id }
payload = {id: params[:id], user_id: current_user.id} service.push("build:restart", payload)
service.push("build:restart", payload)
else
Travis::Sidekiq::BuildRestart.perform_async(id: params[:id], user_id: current_user.id)
end
status 202 status 202
result = true true
else
Travis::Sidekiq::BuildRestart.perform_async(id: params[:id], user_id: current_user.id)
status 202
true
end end
respond_with(result: result, flash: service.messages) respond_with(result: result, flash: service.messages)