Browse Source

Fix Rubocop offences (#2630)

* disable Bundler/OrderedGems

* fix rubocop Lint/UselessAssignment

* fix rubocop Style/BlockDelimiters

* fix rubocop Style/AlignHash

* fix rubocop Style/AlignParameters, Style/EachWithObject

* fix rubocop Style/SpaceInLambdaLiteral
custom
yhirano 5 years ago
committed by Eugen Rochko
parent
commit
3988f2dade
  1. 3
      .rubocop.yml
  2. 2
      app/controllers/auth/sessions_controller.rb
  3. 4
      app/controllers/concerns/localized.rb
  4. 10
      app/mailers/notification_mailer.rb
  5. 14
      app/models/account.rb
  6. 2
      app/models/concerns/paginable.rb

3
.rubocop.yml

@ -77,6 +77,9 @@ Style/Lambda:
Rails/HasAndBelongsToMany:
Enabled: false
Bundler/OrderedGems:
Enabled: false
AllCops:
TargetRubyVersion: 2.3
Exclude:

2
app/controllers/auth/sessions_controller.rb

@ -51,7 +51,7 @@ class Auth::SessionsController < Devise::SessionsController
def valid_otp_attempt?(user)
user.validate_and_consume_otp!(user_params[:otp_attempt]) ||
user.invalidate_otp_backup_code!(user_params[:otp_attempt])
rescue OpenSSL::Cipher::CipherError => error
rescue OpenSSL::Cipher::CipherError => _error
false
end

4
app/controllers/concerns/localized.rb

@ -17,9 +17,9 @@ module Localized
end
def default_locale
ENV.fetch('DEFAULT_LOCALE') {
ENV.fetch('DEFAULT_LOCALE') do
user_supplied_locale || I18n.default_locale
}
end
end
def user_supplied_locale

10
app/mailers/notification_mailer.rb

@ -60,11 +60,11 @@ class NotificationMailer < ApplicationMailer
I18n.with_locale(@me.user.locale || I18n.default_locale) do
mail to: @me.user.email,
subject: I18n.t(
:subject,
scope: [:notification_mailer, :digest],
count: @notifications.size
)
subject: I18n.t(
:subject,
scope: [:notification_mailer, :digest],
count: @notifications.size
)
end
end
end

14
app/models/account.rb

@ -75,12 +75,12 @@ class Account < ApplicationRecord
scope :by_domain_accounts, -> { group(:domain).select(:domain, 'COUNT(*) AS accounts_count').order('accounts_count desc') }
delegate :email,
:current_sign_in_ip,
:current_sign_in_at,
:confirmed?,
to: :user,
prefix: true,
allow_nil: true
:current_sign_in_ip,
:current_sign_in_at,
:confirmed?,
to: :user,
prefix: true,
allow_nil: true
def follow!(other_account)
active_relationships.where(target_account: other_account).first_or_create!(target_account: other_account)
@ -329,7 +329,7 @@ class Account < ApplicationRecord
private
def follow_mapping(query, field)
query.pluck(field).inject({}) { |mapping, id| mapping[id] = true; mapping }
query.pluck(field).each_with_object({}) { |id, mapping| mapping[id] = true }
end
def avatar_styles(file)

2
app/models/concerns/paginable.rb

@ -4,7 +4,7 @@ module Paginable
extend ActiveSupport::Concern
included do
scope :paginate_by_max_id, -> (limit, max_id = nil, since_id = nil) {
scope :paginate_by_max_id, ->(limit, max_id = nil, since_id = nil) {
query = order(arel_table[:id].desc).limit(limit)
query = query.where(arel_table[:id].lt(max_id)) unless max_id.blank?
query = query.where(arel_table[:id].gt(since_id)) unless since_id.blank?

Loading…
Cancel
Save