diff options
| author | Shinya Maeda <shinya@gitlab.com> | 2017-06-25 20:59:27 +0300 |
|---|---|---|
| committer | Shinya Maeda <shinya@gitlab.com> | 2017-07-05 12:38:10 +0300 |
| commit | e364c118a05e4d11677b997a5e31b2f1aa8a4a92 (patch) | |
| tree | 7368867777be0469c3e9d3128a1d5d171d396971 /app/controllers/projects/pipeline_schedules_controller.rb | |
| parent | 507fedf3c39dbe6d2efff789d98b7b29ff9428a3 (diff) | |
Implement variables_attributes create/update cases
Diffstat (limited to 'app/controllers/projects/pipeline_schedules_controller.rb')
| -rw-r--r-- | app/controllers/projects/pipeline_schedules_controller.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/app/controllers/projects/pipeline_schedules_controller.rb b/app/controllers/projects/pipeline_schedules_controller.rb index fdbf930a5ef..2ee6229cf68 100644 --- a/app/controllers/projects/pipeline_schedules_controller.rb +++ b/app/controllers/projects/pipeline_schedules_controller.rb @@ -33,7 +33,8 @@ class Projects::PipelineSchedulesController < Projects::ApplicationController end def update - if schedule.update(schedule_params) + if Ci::CreatePipelineScheduleService + .new(@project, current_user, schedule_params).update(schedule) redirect_to namespace_project_pipeline_schedules_path(@project.namespace.becomes(Namespace), @project) else render :edit @@ -67,6 +68,6 @@ class Projects::PipelineSchedulesController < Projects::ApplicationController def schedule_params params.require(:schedule) .permit(:description, :cron, :cron_timezone, :ref, :active, - variables_attributes: [:key, :value] ) + variables_attributes: [:id, :key, :value, :_destroy] ) end end |