From 6cc48afcb17e6b0f15fb8c1b33343bca5398b720 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Victor=20Geonizeli?= Date: Sun, 27 Feb 2022 23:08:47 -0300 Subject: [PATCH] remove unused controller actions --- app/controllers/posts_controller.rb | 31 +++----------- app/controllers/user_follows_controller.rb | 37 +++-------------- app/controllers/users_controller.rb | 48 +--------------------- 3 files changed, 11 insertions(+), 105 deletions(-) diff --git a/app/controllers/posts_controller.rb b/app/controllers/posts_controller.rb index 9b33ff2..e1fe816 100644 --- a/app/controllers/posts_controller.rb +++ b/app/controllers/posts_controller.rb @@ -1,6 +1,4 @@ class PostsController < ApplicationController - before_action :set_post, only: %i[ show update destroy ] - # GET /posts # GET /posts.json def index @@ -10,6 +8,7 @@ class PostsController < ApplicationController # GET /posts/1 # GET /posts/1.json def show + @post = Post.find(params[:id]) end # POST /posts @@ -24,30 +23,10 @@ class PostsController < ApplicationController end end - # PATCH/PUT /posts/1 - # PATCH/PUT /posts/1.json - def update - if @post.update(post_params) - render :show, status: :ok, location: @post - else - render json: @post.errors, status: :unprocessable_entity - end - end - - # DELETE /posts/1 - # DELETE /posts/1.json - def destroy - @post.destroy - end - private - # Use callbacks to share common setup or constraints between actions. - def set_post - @post = Post.find(params[:id]) - end - # Only allow a list of trusted parameters through. - def post_params - params.require(:post).permit(:content, :user_id, :quoted_post_id) - end + # Only allow a list of trusted parameters through. + def post_params + params.require(:post).permit(:content, :user_id, :quoted_post_id) + end end diff --git a/app/controllers/user_follows_controller.rb b/app/controllers/user_follows_controller.rb index c683931..3ddf0ea 100644 --- a/app/controllers/user_follows_controller.rb +++ b/app/controllers/user_follows_controller.rb @@ -1,17 +1,4 @@ class UserFollowsController < ApplicationController - before_action :set_user_follow, only: %i[ show update destroy ] - - # GET /user_follows - # GET /user_follows.json - def index - @user_follows = UserFollow.all - end - - # GET /user_follows/1 - # GET /user_follows/1.json - def show - end - # POST /user_follows # POST /user_follows.json def create @@ -24,30 +11,16 @@ class UserFollowsController < ApplicationController end end - # PATCH/PUT /user_follows/1 - # PATCH/PUT /user_follows/1.json - def update - if @user_follow.update(user_follow_params) - render :show, status: :ok, location: @user_follow - else - render json: @user_follow.errors, status: :unprocessable_entity - end - end - # DELETE /user_follows/1 # DELETE /user_follows/1.json def destroy - @user_follow.destroy + UserFollow.find(params[:id]).destroy end private - # Use callbacks to share common setup or constraints between actions. - def set_user_follow - @user_follow = UserFollow.find(params[:id]) - end - # Only allow a list of trusted parameters through. - def user_follow_params - params.require(:user_follow).permit(:follower_id, :followed_id) - end + # Only allow a list of trusted parameters through. + def user_follow_params + params.require(:user_follow).permit(:follower_id, :followed_id) + end end diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb index 2a7816c..6847b50 100644 --- a/app/controllers/users_controller.rb +++ b/app/controllers/users_controller.rb @@ -1,53 +1,7 @@ class UsersController < ApplicationController - before_action :set_user, only: %i[ show update destroy ] - - # GET /users - # GET /users.json - def index - @users = User.all - end - # GET /users/1 # GET /users/1.json def show + @user = User.find(params[:id]) end - - # POST /users - # POST /users.json - def create - @user = User.new(user_params) - - if @user.save - render :show, status: :created, location: @user - else - render json: @user.errors, status: :unprocessable_entity - end - end - - # PATCH/PUT /users/1 - # PATCH/PUT /users/1.json - def update - if @user.update(user_params) - render :show, status: :ok, location: @user - else - render json: @user.errors, status: :unprocessable_entity - end - end - - # DELETE /users/1 - # DELETE /users/1.json - def destroy - @user.destroy - end - - private - # Use callbacks to share common setup or constraints between actions. - def set_user - @user = User.find(params[:id]) - end - - # Only allow a list of trusted parameters through. - def user_params - params.require(:user).permit(:username) - end end