From 5d36f8c9ed04451717d8483be14a33f7c5eb017b Mon Sep 17 00:00:00 2001
From: cxt <853663049@qq.com>
Date: Thu, 14 Nov 2019 19:49:11 +0800
Subject: [PATCH] =?UTF-8?q?=E8=AF=BE=E5=A0=82=E6=9B=B4=E6=96=B0=E6=8E=A5?=
 =?UTF-8?q?=E5=8F=A3?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/controllers/weapps/courses_controller.rb | 14 +++++++--
 app/forms/weapps/update_course_form.rb       | 15 ++++++++++
 app/services/weapps/create_course_service.rb |  7 +++++
 app/services/weapps/update_course_service.rb | 31 ++++++++++++++++++++
 app/views/weapps/courses/edit.json.jbuilder  |  2 ++
 5 files changed, 67 insertions(+), 2 deletions(-)
 create mode 100644 app/forms/weapps/update_course_form.rb
 create mode 100644 app/services/weapps/update_course_service.rb
 create mode 100644 app/views/weapps/courses/edit.json.jbuilder

diff --git a/app/controllers/weapps/courses_controller.rb b/app/controllers/weapps/courses_controller.rb
index c7ecb89c9..38f1b99f9 100644
--- a/app/controllers/weapps/courses_controller.rb
+++ b/app/controllers/weapps/courses_controller.rb
@@ -1,5 +1,6 @@
 class Weapps::CoursesController < Weapps::BaseController
   # before_action :require_wechat_login!
+  before_action :teacher_allowed, except: [:create]
 
   def create
     return render_error("只有老师身份才能创建课堂") unless current_user.is_teacher?
@@ -12,11 +13,12 @@ class Weapps::CoursesController < Weapps::BaseController
   end
 
   def edit
-
+    @course = current_course
   end
 
   def update
-
+    Weapps::UpdateCourseService.call(current_course, course_params)
+    render_ok
   end
 
   private
@@ -24,4 +26,12 @@ class Weapps::CoursesController < Weapps::BaseController
   def course_params
     params.permit(:name, :course_list_name, :credit, course_module_types: [])
   end
+
+  def current_course
+    @_current_course = Course.find params[:id]
+  end
+
+  def teacher_allowed
+    return render_forbidden unless current_user.course_identity(current_course) < Course::STUDENT
+  end
 end
\ No newline at end of file
diff --git a/app/forms/weapps/update_course_form.rb b/app/forms/weapps/update_course_form.rb
new file mode 100644
index 000000000..6fb1d81d9
--- /dev/null
+++ b/app/forms/weapps/update_course_form.rb
@@ -0,0 +1,15 @@
+class Weapps::UpdateCourseForm
+  include ActiveModel::Model
+
+  attr_accessor :course
+  attr_accessor :name, :course_list_name, :credit
+
+  validates :name, presence: true
+  validates :course_list_name, presence: true
+
+  validate :course_name_prefix
+
+  def course_name_prefix
+    raise '课堂名称应以课程名称开头' unless name.index(course_list_name) && name.index(course_list_name) == 0
+  end
+end
\ No newline at end of file
diff --git a/app/services/weapps/create_course_service.rb b/app/services/weapps/create_course_service.rb
index d7dd81ffe..196533bb6 100644
--- a/app/services/weapps/create_course_service.rb
+++ b/app/services/weapps/create_course_service.rb
@@ -14,6 +14,13 @@ class Weapps::CreateCourseService < ApplicationService
       course.school_id = course.teacher&.school_id
       course.is_public = 0
       course.credit = params[:credit].blank? ? nil : params[:credit]
+      course_list = CourseList.find_by(name: params[:course_list_name].to_s.strip)
+      if course_list
+        course.course_list_id = course_list.id
+      else
+        new_course_list = CourseList.create!(name: params[:course_list_name].to_s.strip, user_id: course.tea_id, is_admin: 0)
+        course.course_list_id = new_course_list.id
+      end
       course.save!
 
       course.generate_invite_code
diff --git a/app/services/weapps/update_course_service.rb b/app/services/weapps/update_course_service.rb
new file mode 100644
index 000000000..4f2fa4676
--- /dev/null
+++ b/app/services/weapps/update_course_service.rb
@@ -0,0 +1,31 @@
+class Weapps::UpdateCourseService < ApplicationService
+  attr_reader :course, :params
+
+  def initialize(course, params)
+    @course = course
+    @params = params
+  end
+
+  def call
+    Weapps::UpdateCourseForm.new(form_params).validate!
+
+    ActiveRecord::Base.transaction do
+      course.name = params[:name].to_s.strip
+      course.credit = params[:credit].blank? ? nil : params[:credit]
+      course_list = CourseList.find_by(name: params[:course_list_name].to_s.strip)
+      if course_list
+        course.course_list_id = course_list.id
+      else
+        new_course_list = CourseList.create!(name: params[:course_list_name].to_s.strip, user_id: course.tea_id, is_admin: 0)
+        course.course_list_id = new_course_list.id
+      end
+      course.save!
+    end
+  end
+
+  private
+
+  def form_params
+    params.merge(course: course)
+  end
+end
\ No newline at end of file
diff --git a/app/views/weapps/courses/edit.json.jbuilder b/app/views/weapps/courses/edit.json.jbuilder
new file mode 100644
index 000000000..ef7a28a42
--- /dev/null
+++ b/app/views/weapps/courses/edit.json.jbuilder
@@ -0,0 +1,2 @@
+json.(@course, :id, :name, :credit)
+json.course_list_name @course.course_list&.name
\ No newline at end of file