Ruby Critic

Updated 2025-04-10 19:04:14 -0300
app/controllers/accounts / deal_assignees_controller.rb

46 lines of codes
6 methods
7.7 complexity/method
6 churn
46.4 complexity
24 duplications
class Accounts::DealAssigneesController < InternalController                                        InstanceVariableAssumption              
      Accounts::DealAssigneesController assumes too much for instance variable '@deal'          
  
                                        InstanceVariableAssumption              
      Accounts::DealAssigneesController assumes too much for instance variable '@deal_assignee'          
  
                                        IrresponsibleModule              
      Accounts::DealAssigneesController has no descriptive comment          
  
  before_action :set_deal_assignee, only: %i[destroy]
  before_action :set_deal, only: %i[new]

  def destroy
    return unless @deal_assignee.destroy

    respond_to do |format|                                        DuplicateCode              
      Similar code found in 2 nodes              Locations:                  0                  1                  
  
      format.html do
        redirect_to account_deal_path(current_user.account, @deal_assignee.deal),
                    notice: t('flash_messages.deleted', model: DealAssignee.model_name.human)
      end
      format.turbo_stream
    end
  end

  def new
    @deal_assignee = @deal.deal_assignees.new
  end

  def create                                        TooManyStatements              
      Accounts::DealAssigneesController#create has approx 6 statements          
  
    @deal_assignee = DealAssignee.new(deal_assignees_params)
    if @deal_assignee.save
      respond_to do |format|
        format.html { redirect_to account_deal_path(@deal_assignee.account, @deal_assignee.deal) }
        format.turbo_stream
      end
    else
      render :new, status: :unprocessable_entity
    end
  end

  private

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

  def set_deal
    @deal = Deal.find(params[:deal_id])
  end

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