Sign upLogin

infusionvlc/infusion

View on GitHub
Star
app/controllers/application_controller.rb

Summary

Maintainability
A
0 mins
Test Coverage

Line is too long. [87/80]
Open

    added_attrs = [:username, :email, :bio, :locale, :password, :password_confirmation,
Severity: Minor
Found in app/controllers/application_controller.rb by rubocop

Line is too long. [85/80]
Open

    @notifications = current_user.notifications.where(read: false) if user_signed_in?
Severity: Minor
Found in app/controllers/application_controller.rb by rubocop

Line is too long. [95/80]
Open

                   :remember_me, :notify_meetups, :notify_proposals, :notify_comments, :avatar]
Severity: Minor
Found in app/controllers/application_controller.rb by rubocop

Do not prefix writer method names with set_.
Wontfix

  def set_locale_cookie(locale)
Severity: Minor
Found in app/controllers/application_controller.rb by rubocop

This cop makes sure that accessor methods are named properly.

Example:

# bad
def set_attribute(value)
end

# good
def attribute=(value)
end

# bad
def get_attribute
end

# good
def attribute
end

Unused method argument - resource. If it's necessary, use _ or _resource as an argument name to indicate that it won't be used. You can also write as after_sign_in_path_for(*) if you want the method to accept any arguments but don't care about them.
Open

  def after_sign_in_path_for(resource)
Severity: Minor
Found in app/controllers/application_controller.rb by rubocop

This cop checks for unused method arguments.

Example:

# bad

def some_method(used, unused, _unused_but_allowed)
  puts used
end

Example:

# good

def some_method(used, _unused, _unused_but_allowed)
  puts used
end

There are no issues that match your filters.

Category
Status