Sign upLogin

infusionvlc/ConexionFelina

View on GitHub
Star
app/validators/sponsor_validator.rb

Summary

Maintainability
A
0 mins
Test Coverage

Line is too long. [82/80]
Open

      record.errors[:last_payment_date] << "Last payment date must be in the past"
Severity: Minor
Found in app/validators/sponsor_validator.rb by rubocop

Use a guard clause instead of wrapping the code inside a conditional expression.
Open

    if !record.last_payment_date.nil? && record.last_payment_date > Date.today
Severity: Minor
Found in app/validators/sponsor_validator.rb by rubocop

Use a guard clause instead of wrapping the code inside a conditional expression

Example:

# bad
def test
  if something
    work
  end
end

# good
def test
  return unless something
  work
end

# also good
def test
  work if something
end

# bad
if something
  raise 'exception'
else
  ok
end

# good
raise 'exception' if something
ok

Prefer single-quoted strings when you don't need string interpolation or special symbols.
Open

      record.errors[:last_payment_date] << "Last payment date must be in the past"
Severity: Minor
Found in app/validators/sponsor_validator.rb by rubocop

Checks if uses of quotes match the configured preference.

Example: EnforcedStyle: single_quotes (default)

# bad
"No special symbols"
"No string interpolation"
"Just text"

# good
'No special symbols'
'No string interpolation'
'Just text'
"Wait! What's #{this}!"

Example: EnforcedStyle: double_quotes

# bad
'Just some text'
'No special chars or interpolation'

# good
"Just some text"
"No special chars or interpolation"
"Every string in #{project} uses double_quotes"

There are no issues that match your filters.

Category
Status