From 58d8b9ae62813cb69821a4e0e41c914aee7e323a Mon Sep 17 00:00:00 2001
From: Shinya Maeda <shinya@gitlab.com>
Date: Fri, 23 Jun 2017 03:57:13 +0900
Subject: [PATCH] Controller logic. Delete before modification. Halfway.

---
 .../projects/pipeline_schedules_controller.rb |  3 +-
 app/models/ci/pipeline_schedule.rb            | 14 +++++++-
 app/models/ci/pipeline_schedule_variable.rb   |  1 +
 .../pipeline_schedules/_form.html.haml        | 32 +++++++++++++++++++
 4 files changed, 48 insertions(+), 2 deletions(-)

diff --git a/app/controllers/projects/pipeline_schedules_controller.rb b/app/controllers/projects/pipeline_schedules_controller.rb
index 60db179277b..fdbf930a5ef 100644
--- a/app/controllers/projects/pipeline_schedules_controller.rb
+++ b/app/controllers/projects/pipeline_schedules_controller.rb
@@ -66,6 +66,7 @@ class Projects::PipelineSchedulesController < Projects::ApplicationController
 
   def schedule_params
     params.require(:schedule)
-      .permit(:description, :cron, :cron_timezone, :ref, :active)
+      .permit(:description, :cron, :cron_timezone, :ref, :active,
+        variables_attributes: [:key, :value] )
   end
 end
diff --git a/app/models/ci/pipeline_schedule.rb b/app/models/ci/pipeline_schedule.rb
index 31b73248947..61215b1fb46 100644
--- a/app/models/ci/pipeline_schedule.rb
+++ b/app/models/ci/pipeline_schedule.rb
@@ -9,18 +9,30 @@ module Ci
     belongs_to :owner, class_name: 'User'
     has_one :last_pipeline, -> { order(id: :desc) }, class_name: 'Ci::Pipeline'
     has_many :pipelines
-    has_many :variables, class_name: 'Ci::PipelineScheduleVariable'
+    has_many :variables, :dependent => :destroy, class_name: 'Ci::PipelineScheduleVariable'
 
     validates :cron, unless: :importing?, cron: true, presence: { unless: :importing? }
     validates :cron_timezone, cron_timezone: true, presence: { unless: :importing? }
     validates :ref, presence: { unless: :importing? }
     validates :description, presence: true
+    validates_associated :variables
 
     before_save :set_next_run_at
 
     scope :active, -> { where(active: true) }
     scope :inactive, -> { where(active: false) }
 
+    accepts_nested_attributes_for :variables, allow_destroy: true
+
+    before_validation(on: :update) do
+      # TODO: if validation failed, restore the deleted_obj
+      deleted_obj = Ci::PipelineScheduleVariable.where(pipeline_schedule_id: self).destroy_all
+    end
+
+    after_validation(on: :update) do
+      # TODO: if validation failed, restore the deleted_obj
+    end
+
     def owned_by?(current_user)
       owner == current_user
     end
diff --git a/app/models/ci/pipeline_schedule_variable.rb b/app/models/ci/pipeline_schedule_variable.rb
index 2d681446d00..ee5b8733fac 100644
--- a/app/models/ci/pipeline_schedule_variable.rb
+++ b/app/models/ci/pipeline_schedule_variable.rb
@@ -4,6 +4,7 @@ module Ci
     include HasVariable
 
     belongs_to :pipeline_schedule
+
     validates :key, uniqueness: { scope: :pipeline_schedule_id }
   end
 end
diff --git a/app/views/projects/pipeline_schedules/_form.html.haml b/app/views/projects/pipeline_schedules/_form.html.haml
index fc7fa5c1876..c079d5712e3 100644
--- a/app/views/projects/pipeline_schedules/_form.html.haml
+++ b/app/views/projects/pipeline_schedules/_form.html.haml
@@ -22,6 +22,38 @@
       = f.label :ref, _('Target Branch'), class: 'label-light'
       = dropdown_tag(_("Select target branch"), options: { toggle_class: 'btn js-target-branch-dropdown', dropdown_class: 'git-revision-dropdown', title: _("Select target branch"), filter: true, placeholder: s_("OfSearchInADropdown|Filter"), data: { data: @project.repository.branch_names, default_branch: @project.default_branch } } )
       = f.text_field :ref, value: @schedule.ref, id: 'schedule_ref', class: 'hidden', name: 'schedule[ref]', required: true
+  -# TODO: Test code
+  = @schedule.variables.inspect
+  - if @schedule.variables.present?
+    - @schedule.variables.each_with_index do |variable, i|
+      .form-group
+        .col-md-9
+          %label.label-light Key
+          %input.form-control{:name => "schedule[variables_attributes][#{i}][key]", :type => "text", :value => variable.key}/
+          %p.gl-field-error.hide This field is required.
+          %label.label-light Value
+          %input.form-control{:name => "schedule[variables_attributes][#{i}][value]", :type => "text", :value => variable.value}/
+          %p.gl-field-error.hide This field is required.
+    - if @schedule.variables.count == 1
+      - (1..1).each do |i|
+        .form-group
+          .col-md-9
+            %label.label-light Key
+            %input.form-control{:name => "schedule[variables_attributes][#{i}][key]", :type => "text"}/
+            %p.gl-field-error.hide This field is required.
+            %label.label-light Value
+            %input.form-control{:name => "schedule[variables_attributes][#{i}][value]", :type => "text"}/
+            %p.gl-field-error.hide This field is required.
+  - else
+    - (0..0).each do |i|
+      .form-group
+        .col-md-9
+          %label.label-light Key
+          %input.form-control{:name => "schedule[variables_attributes][#{i}][key]", :type => "text"}/
+          %p.gl-field-error.hide This field is required.
+          %label.label-light Value
+          %input.form-control{:name => "schedule[variables_attributes][#{i}][value]", :type => "text"}/
+          %p.gl-field-error.hide This field is required.
   .form-group
     .col-md-9
       = f.label  :active, s_('PipelineSchedules|Activated'), class: 'label-light'