Ruby Critic

Updated 2025-10-21 10:25:37 +1000
app/controllers/accounts/settings/deals / deal_lost_reasons_controller.rb

70 lines of codes
8 methods
9.0 complexity/method
1 churn
71.66 complexity
55 duplications
class Accounts::Settings::Deals::DealLostReasonsController < InternalController                                        InstanceVariableAssumption              
      Accounts::Settings::Deals::DealLostReasonsController assumes too much for instance variable '@deal_lost_reason'          
  
                                        IrresponsibleModule              
      Accounts::Settings::Deals::DealLostReasonsController has no descriptive comment          
  
  before_action :set_deal_lost_reason, only: %i[edit update destroy]

  def index
    @deal_lost_reasons = DealLostReason.all
  end

  def edit; end

  def update
    if @deal_lost_reason.update(deal_lost_reason_params)
      respond_to do |format|                                        DuplicateCode              
      Similar code found in 2 nodes              Locations:                  0                  1                  
  
        format.html do
          redirect_to edit_account_settings_deals_deal_lost_reason_path(Current.account, @deal_lost_reason),
                      notice: t('flash_messages.updated', model: DealLostReason.model_name.human)
        end
        format.turbo_stream
      end
    else
      render :edit, status: :unprocessable_entity
    end
  end

  def new
    @deal_lost_reason = DealLostReason.new
  end

  def create                                        TooManyStatements              
      Accounts::Settings::Deals::DealLostReasonsController#create has approx 6 statements          
  
    @deal_lost_reason = DealLostReason.new(deal_lost_reason_params)
    if @deal_lost_reason.save
      respond_to do |format|
        format.html do                                        DuplicateCode              
      Similar code found in 2 nodes              Locations:                  0                  1                  
  
          redirect_to account_settings_deals_deal_lost_reasons_path(Current.account),
                      notice: t('flash_messages.created', model: DealLostReason.model_name.human)
        end
        format.turbo_stream
      end
    else
      render :new, status: :unprocessable_entity
    end
  end

  def destroy                                        HighComplexity              
      Accounts::Settings::Deals::DealLostReasonsController#destroy has a flog score of 25          
  
                                        TooManyStatements              
      Accounts::Settings::Deals::DealLostReasonsController#destroy has approx 6 statements          
  
    if @deal_lost_reason.destroy
      respond_to do |format|
        format.html do                                        DuplicateCode              
      Similar code found in 2 nodes              Locations:                  0                  1                  
  
                                        DuplicateMethodCall              
      Accounts::Settings::Deals::DealLostReasonsController#destroy calls 'format.html' 2 times              Locations:                  0                  1                  
  
          redirect_to account_settings_deals_deal_lost_reasons_path(Current.account),                                        DuplicateMethodCall              
      Accounts::Settings::Deals::DealLostReasonsController#destroy calls 'Current.account' 2 times              Locations:                  0                  1                  
  
                                        DuplicateMethodCall              
      Accounts::Settings::Deals::DealLostReasonsController#destroy calls 'account_settings_deals_deal_lost_reasons_path(Current.account)' 2 times              Locations:                  0                  1                  
  
                      notice: t('flash_messages.deleted', model: DealLostReason.model_name.human)
        end
      end
    else
      respond_to do |format|
        format.html do                                        DuplicateMethodCall              
      Accounts::Settings::Deals::DealLostReasonsController#destroy calls 'format.html' 2 times              Locations:                  0                  1                  
  
          redirect_to account_settings_deals_deal_lost_reasons_path(Current.account),                                        DuplicateMethodCall              
      Accounts::Settings::Deals::DealLostReasonsController#destroy calls 'Current.account' 2 times              Locations:                  0                  1                  
  
                                        DuplicateMethodCall              
      Accounts::Settings::Deals::DealLostReasonsController#destroy calls 'account_settings_deals_deal_lost_reasons_path(Current.account)' 2 times              Locations:                  0                  1                  
  
                      flash: { error: @deal_lost_reason.errors.full_messages.to_sentence }
        end
      end
    end
  end

  private

  def set_deal_lost_reason
    @deal_lost_reason = DealLostReason.find(params[:id])
  end

  def deal_lost_reason_params
    params.require(:deal_lost_reason).permit(:name)
  end
end