tasafo/certifico

Assignment Branch Condition size for with_attachment is too high. [<11, 29, 1> 31.03/17]
Open

  def with_attachment
    @subscriber = Subscriber.find(params[:subscriber_id])
    @user = @subscriber.user
    certificate = @subscriber.certificate
    user_name = @user.full_name.parameterize

Found in app/mailers/certificate_mailer.rb by rubocop

Read up
Read up
Exclude checks
- Disable engine
- Disable check

This cop checks that the ABC size of methods is not higher than the configured maximum. The ABC size is based on assignments, branches (method calls), and conditions. See http://c2.com/cgi/wiki?AbcMetric and https://en.wikipedia.org/wiki/ABC_Software_Metric.

Interpreting ABC size:

<= 17 satisfactory
18..30 unsatisfactory
> 30 dangerous

You can have repeated "attributes" calls count as a single "branch". For this purpose, attributes are any method with no argument; no attempt is meant to distinguish actual attr_reader from other methods.

Example: CountRepeatedAttributes: false (default is true)

# `model` and `current_user`, refenced 3 times each,
 # are each counted as only 1 branch each if
 # `CountRepeatedAttributes` is set to 'false'

 def search
   @posts = model.active.visible_by(current_user)
             .search(params[:q])
   @posts = model.some_process(@posts, current_user)
   @posts = model.another_process(@posts, current_user)

   render 'pages/search/page'
 end

This cop also takes into account IgnoredMethods (defaults to [])