ecs: modify add course manager api

app/controllers/ecs/course_managers_controller.rb

@@ -3,7 +3,8 @@ class Ecs::CourseManagersController < Ecs::CourseBaseController
   before_action :check_major_manager_permission!, only: [:create, :destroy]
 
   def create
-    @user = Ecs::CreateCourseManagerService.call(current_course, params[:user_id])
+    Ecs::CreateCourseManagerService.call(current_course, params[:user_ids])
+    render_ok
   rescue Ecs::CreateCourseManagerService::Error => ex
     render_error(ex.message)
   end

app/services/ecs/create_course_manager_service.rb

@@ -3,27 +3,29 @@ class Ecs::CreateCourseManagerService < ApplicationService
 
   COURSE_MANAGER_COUNT_LIMIT = 2  # 课程管理员数量限制
 
-  attr_reader :ec_course, :user_id
+  attr_reader :ec_course, :user_ids
 
-  def initialize(ec_course, user_id)
+  def initialize(ec_course, user_ids)
     @ec_course = ec_course
-    @user_id   = user_id
+    @user_ids  = user_ids
   end
 
   def call
-    user = User.find_by(id: params[:user_id])
+    users_count = User.where(id: user_ids).count
-    raise Error, '该用户不存在' if user.blank?
+    raise Error, '用户不存在' if users_count != user_ids.size
 
-    if ec_course.ec_course_users.exists?(user_id: user.id)
+    if ec_course.ec_course_users.exists?(user_id: user_ids)
-      raise Error, '该用户已经是该课程的管理员了'
+      raise Error, '用户已经是该课程的管理员'
     end
 
-    if ec_course.ec_course_users.count >= COURSE_MANAGER_COUNT_LIMIT
+    if ec_course.ec_course_users.count + user_ids.size >= COURSE_MANAGER_COUNT_LIMIT
-      raise Error, '该课程管理员数量已达上限'
+      raise Error, "课程管理员数量过多（最多#{COURSE_MANAGER_COUNT_LIMIT}）"
     end
 
-    ec_course.ec_course_users.create!(user: user)
+    ActiveRecord::Base.transaction do
-
+      user_ids.each do |user_id|
-    user
+        ec_course.ec_course_users.create!(user_id: user_id)
+      end
+    end
   end
 end

app/views/ecs/course_managers/create.json.jbuilder

@@ -1 +0,0 @@
-json.partial! 'ecs/shared/user', user: @user