Sign upLogin

iZettle/ninetails

View on GitHub
Star
app/models/ninetails/revision.rb

Summary

Maintainability
A
0 mins
Test Coverage

Place the . on the next line, together with the method name.
Open

          where.not(id: container.id).
Severity: Minor
Found in app/models/ninetails/revision.rb by rubocop

This cop checks the . position in multi-line method calls.

Example: EnforcedStyle: leading (default)

# bad
something.
  mehod

# good
something
  .method

Example: EnforcedStyle: trailing

# bad
something
  .method

# good
something.
  mehod

Favor modifier unless usage when having a single-line body. Another good alternative is the usage of control flow &&/||.
Open

        unless section.valid?
Severity: Minor
Found in app/models/ninetails/revision.rb by rubocop

Checks for if and unless statements that would fit on one line if written as a modifier if/unless. The maximum line length is configured in the Metrics/LineLength cop.

Example:

# bad
if condition
  do_stuff(bar)
end

unless qux.empty?
  Foo.do_something
end

# good
do_stuff(bar) if condition
Foo.do_something unless qux.empty?

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

      if container.is_a?(Page) && url.present?
Severity: Minor
Found in app/models/ninetails/revision.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

Place the . on the next line, together with the method name.
Open

          includes(:current_revision).
Severity: Minor
Found in app/models/ninetails/revision.rb by rubocop

This cop checks the . position in multi-line method calls.

Example: EnforcedStyle: leading (default)

# bad
something.
  mehod

# good
something
  .method

Example: EnforcedStyle: trailing

# bad
something
  .method

# good
something.
  mehod

Place the . on the next line, together with the method name.
Open

        url_exists = Ninetails::Container.
Severity: Minor
Found in app/models/ninetails/revision.rb by rubocop

This cop checks the . position in multi-line method calls.

Example: EnforcedStyle: leading (default)

# bad
something.
  mehod

# good
something
  .method

Example: EnforcedStyle: trailing

# bad
something
  .method

# good
something.
  mehod

Place the . on the next line, together with the method name.
Open

          where(ninetails_revisions: { url: url }).
Severity: Minor
Found in app/models/ninetails/revision.rb by rubocop

This cop checks the . position in multi-line method calls.

Example: EnforcedStyle: leading (default)

# bad
something.
  mehod

# good
something
  .method

Example: EnforcedStyle: trailing

# bad
something
  .method

# good
something.
  mehod

There are no issues that match your filters.

Category
Status