diff options
| author | Kamil Trzciński <ayufan@ayufan.eu> | 2018-05-28 13:49:08 +0300 |
|---|---|---|
| committer | Dylan Griffith <dyl.griffith@gmail.com> | 2018-05-31 11:56:41 +0300 |
| commit | c6e95b04405f1e07f76505b03c6c096f4c4d084b (patch) | |
| tree | c1d36122def40f8b0912b822bfcab764d646f39b /app/controllers/projects/runner_projects_controller.rb | |
| parent | d9251f2ea047d359d1d7d4799d1ba84da5896f64 (diff) | |
Improve `Ci::Runner#assign_to` to return a flag whether it succeeded or not
Diffstat (limited to 'app/controllers/projects/runner_projects_controller.rb')
| -rw-r--r-- | app/controllers/projects/runner_projects_controller.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/app/controllers/projects/runner_projects_controller.rb b/app/controllers/projects/runner_projects_controller.rb index 0ec2490655f..a080724634b 100644 --- a/app/controllers/projects/runner_projects_controller.rb +++ b/app/controllers/projects/runner_projects_controller.rb @@ -9,9 +9,8 @@ class Projects::RunnerProjectsController < Projects::ApplicationController return head(403) unless can?(current_user, :assign_runner, @runner) path = project_runners_path(project) - runner_project = @runner.assign_to(project, current_user) - if runner_project.persisted? + if @runner.assign_to(project, current_user) redirect_to path else redirect_to path, alert: 'Failed adding runner to project' |