Ruby Critic

Updated 2025-08-27 21:12:05 -0300

app/controllers/api/v1/accounts / deal_assignees_controller.rb

31 lines of codes

4 methods

5.1 complexity/method

4 churn

20.34 complexity

42 duplications

class Api::V1::Accounts::DealAssigneesController < Api::V1::InternalController                                        InstanceVariableAssumption              
      Api::V1::Accounts::DealAssigneesController assumes too much for instance variable '@deal_assignee'          
  
                                        IrresponsibleModule              
      Api::V1::Accounts::DealAssigneesController has no descriptive comment          
  
  before_action :set_deal_assignee, only: %i[destroy]

  def destroy                                        DuplicateCode              
      Similar code found in 2 nodes              Locations:                  0                  1                  
  
    if @deal_assignee.destroy
      head :no_content
    else
      render json: { errors: @deal_assignee.errors.full_messages }, status: :unprocessable_entity
    end
  end

  def create                                        DuplicateCode              
      Similar code found in 5 nodes              Locations:                  0                  1                  2                  3                  4                  
  
    @deal_assignee = DealAssignee.new(deal_assignees_params)

    if @deal_assignee.save
      render json: @deal_assignee, status: :created
    else
      render json: { errors: @deal_assignee.errors.full_messages }, status: :unprocessable_entity
    end
  end

  private

  def deal_assignees_params
    params.permit(:user_id, :deal_id)
  end

  def set_deal_assignee
    @deal_assignee = DealAssignee.find(params[:id])
  end
end