refactor(soft-delete-student-class): soft-delete when removing studen…

…t from class

Gemfile

@@ -9,6 +9,9 @@ gem 'tzinfo-data', platforms: [:mswin, :mswin64]
 # Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
 gem 'rails', '~> 7.0.8.4'
 
+# Soft delete
+gem 'paranoia', '~> 2.4'
+
 # Use PostgreSQL for the backend
 gem 'pg'
 

Makefile

@@ -0,0 +1,11 @@
+SHELL := /bin/zsh
+
+# Loading all context of ~/.zshrc
+.SHELLFLAGS := -c '. ~/.zshrc; exec "$$0"'
+CURRENT_BRANCH := $(shell git rev-parse --abbrev-ref HEAD)
+REMOTE := origin
+
+clean:
+	echo "Preparing to setup soft-delete"
+	rails generate migration AddDeletedAtToCourseUsers deleted_at:datetime:index
+	rails db:migrate

app/controllers/users_controller.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 class UsersController < ApplicationController
   load_resource :user
+  before_action :set_course
+  before_action :set_course_user, only: [:destroy]
 
   def show
     if @user.built_in?
@@ -19,4 +21,29 @@ def other_instances
     tenant = current_tenant
     ActsAsTenant.without_tenant { Instance.containing_user(@user) - [tenant] }
   end
+
+  def destroy
+    if @course_user.student?
+      @course_user.destroy
+      respond_to do |format|
+        format.html { redirect_to course_users_path(@course), notice: 'Student was successfully removed from the course.' }
+        format.json { head :no_content }
+      end
+    else
+      respond_to do |format|
+        format.html { redirect_to course_users_path(@course), alert: 'Only students can be removed.' }
+        format.json { render json: { error: 'Only students can be removed' }, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  private
+
+  def set_course
+    @course = Course.find(params[:course_id])
+  end
+
+  def set_course_user
+    @course_user = @course.course_users.find(params[:id])
+  end
 end

app/models/course_user.rb

@@ -3,6 +3,7 @@ class CourseUser < ApplicationRecord
   include CourseUser::StaffConcern
   include CourseUser::LevelProgressConcern
   include CourseUser::TodoConcern
+  acts_as_paranoid
 
   after_initialize :set_defaults, if: :new_record?
   before_validation :set_defaults, if: :new_record?

config/routes.rb

@@ -83,6 +83,10 @@
     post 'mark_as_read'
   end
 
+  resources :courses do
+    resources :course_users, only: [:destroy]
+  end
+
   resources :jobs, only: [:show]
 
   resources :instance_user_role_requests, path: 'role_requests' do