Refactor student removal so it's common between #destroy and #destroy_batch without coupling those methods.

Author: fspeirs

app/controllers/api/school_students_controller.rb

@@ -110,8 +110,7 @@ def update
     end
 
     def destroy
-      params[:student_ids] = [params[:id]]
-      destroy_batch
+      remove_students([params[:id]])
     end
 
     def destroy_batch
@@ -126,6 +125,12 @@ def destroy_batch
         return
       end
 
+      remove_students(student_ids)
+    end
+
+    private
+
+    def remove_students(student_ids)
       # Invoke StudentRemovalService
       service = StudentRemovalService.new(
         students: student_ids,
@@ -148,8 +153,6 @@ def destroy_batch
       end
     end
 
-    private
-
     def school_student_params
       params.require(:school_student).permit(:username, :password, :name)
     end