diff options
author | Kamil Trzciński <ayufan@ayufan.eu> | 2017-02-17 13:03:12 +0000 |
---|---|---|
committer | Kamil Trzciński <ayufan@ayufan.eu> | 2017-02-17 13:03:12 +0000 |
commit | f662bc5a866dc24580caa48952f25061bae60d1a (patch) | |
tree | 1cd3c637b0581056aa9ce608e08b974e870a4607 /app/controllers/admin | |
parent | 5cc9ebbe14ce35de1ec45fc7abce3aa9f408ddf2 (diff) | |
parent | 60288d6c62d7e65ed5a93a72ba047ccaa2daa22b (diff) | |
download | gitlab-ce-f662bc5a866dc24580caa48952f25061bae60d1a.tar.gz |
Merge branch 'use-update-runner-service' into 'master'
Prefer service object over after_save hook
Closes #26921
See merge request !8664
Diffstat (limited to 'app/controllers/admin')
-rw-r--r-- | app/controllers/admin/runners_controller.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/app/controllers/admin/runners_controller.rb b/app/controllers/admin/runners_controller.rb index 7345c91f67d..348641e5ecb 100644 --- a/app/controllers/admin/runners_controller.rb +++ b/app/controllers/admin/runners_controller.rb @@ -13,7 +13,7 @@ class Admin::RunnersController < Admin::ApplicationController end def update - if @runner.update_attributes(runner_params) + if Ci::UpdateRunnerService.new(@runner).update(runner_params) respond_to do |format| format.js format.html { redirect_to admin_runner_path(@runner) } @@ -31,7 +31,7 @@ class Admin::RunnersController < Admin::ApplicationController end def resume - if @runner.update_attributes(active: true) + if Ci::UpdateRunnerService.new(@runner).update(active: true) redirect_to admin_runners_path, notice: 'Runner was successfully updated.' else redirect_to admin_runners_path, alert: 'Runner was not updated.' @@ -39,7 +39,7 @@ class Admin::RunnersController < Admin::ApplicationController end def pause - if @runner.update_attributes(active: false) + if Ci::UpdateRunnerService.new(@runner).update(active: false) redirect_to admin_runners_path, notice: 'Runner was successfully updated.' else redirect_to admin_runners_path, alert: 'Runner was not updated.' |