moonleerecords/moonlee-website

View on GitHub

Showing 211 of 211 total issues

Prefer annotated tokens (like %<foo>s</foo>) over unannotated tokens (like %s).
Open

    social_post.url = format('https://www.youtube.com/watch?v=%s', video.id)
Severity: Minor
Found in lib/tasks/social.rake by rubocop

Use a consistent style for named format string tokens.

Note: unannotated style cop only works for strings which are passed as arguments to those methods: sprintf, format, %. The reason is that unannotated format is very similar to encoded URLs or Date/Time formatting strings.

Example: EnforcedStyle: annotated (default)

# bad
format('%{greeting}', greeting: 'Hello')
format('%s', 'Hello')

# good
format('%<greeting>s', greeting: 'Hello')</greeting>

Example: EnforcedStyle: template

# bad
format('%<greeting>s', greeting: 'Hello')
format('%s', 'Hello')

# good
format('%{greeting}', greeting: 'Hello')</greeting>

Example: EnforcedStyle: unannotated

# bad
format('%<greeting>s', greeting: 'Hello')
format('%{greeting}', 'Hello')

# good
format('%s', 'Hello')</greeting>

Specify a :dependent option.
Open

  has_many :events
Severity: Minor
Found in app/models/venue.rb by rubocop

This cop looks for has_many or has_one associations that don't specify a :dependent option. It doesn't register an offense if :through option was specified.

Example:

# bad
class User < ActiveRecord::Base
  has_many :comments
  has_one :avatar
end

# good
class User < ActiveRecord::Base
  has_many :comments, dependent: :restrict_with_exception
  has_one :avatar, dependent: :destroy
  has_many :patients, through: :appointments
end

Use mappings[release_format].presence || release_format instead of mappings[release_format].present? ? mappings[release_format] : release_format.
Open

    mappings[release_format].present? ? mappings[release_format] : release_format
Severity: Minor
Found in app/helpers/releases_helper.rb by rubocop

This cop checks code that can be written more easily using Object#presence defined by Active Support.

Example:

# bad
a.present? ? a : nil

# bad
!a.present? ? nil : a

# bad
a.blank? ? nil : a

# bad
!a.blank? ? a : nil

# good
a.presence

Example:

# bad
a.present? ? a : b

# bad
!a.present? ? b : a

# bad
a.blank? ? b : a

# bad
!a.blank? ? a : b

# good
a.presence || b

Prefer annotated tokens (like %<foo>s</foo>) over unannotated tokens (like %s).
Open

    MEDIA_ENDPOINT = 'https://api.instagram.com/v1/users/self/media/recent?access_token=%s'.freeze
Severity: Minor
Found in app/lib/instagram/media.rb by rubocop

Use a consistent style for named format string tokens.

Note: unannotated style cop only works for strings which are passed as arguments to those methods: sprintf, format, %. The reason is that unannotated format is very similar to encoded URLs or Date/Time formatting strings.

Example: EnforcedStyle: annotated (default)

# bad
format('%{greeting}', greeting: 'Hello')
format('%s', 'Hello')

# good
format('%<greeting>s', greeting: 'Hello')</greeting>

Example: EnforcedStyle: template

# bad
format('%<greeting>s', greeting: 'Hello')
format('%s', 'Hello')

# good
format('%{greeting}', greeting: 'Hello')</greeting>

Example: EnforcedStyle: unannotated

# bad
format('%<greeting>s', greeting: 'Hello')
format('%{greeting}', 'Hello')

# good
format('%s', 'Hello')</greeting>

Missing magic comment # frozen_string_literal: true.
Open

class Release < ApplicationRecord
Severity: Minor
Found in app/models/release.rb by rubocop

This cop is designed to help upgrade to Ruby 3.0. It will add the comment # frozen_string_literal: true to the top of files to enable frozen string literals. Frozen string literals may be default in Ruby 3.0. The comment will be added below a shebang and encoding comment. The frozen string literal comment is only valid in Ruby 2.3+.

Example: EnforcedStyle: when_needed (default)

# The `when_needed` style will add the frozen string literal comment
# to files only when the `TargetRubyVersion` is set to 2.3+.
# bad
module Foo
  # ...
end

# good
# frozen_string_literal: true

module Foo
  # ...
end

Example: EnforcedStyle: always

# The `always` style will always add the frozen string literal comment
# to a file, regardless of the Ruby version or if `freeze` or `<<` are
# called on a string literal.
# bad
module Bar
  # ...
end

# good
# frozen_string_literal: true

module Bar
  # ...
end

Example: EnforcedStyle: never

# The `never` will enforce that the frozen string literal comment does
# not exist in a file.
# bad
# frozen_string_literal: true

module Baz
  # ...
end

# good
module Baz
  # ...
end

Use if bandcamp_url.blank? instead of unless bandcamp_url.present?.
Open

    return nil unless bandcamp_url.present?
Severity: Minor
Found in app/models/release.rb by rubocop

This cops checks for code that can be changed to blank?. Settings: NilOrEmpty: Convert checks for nil or empty? to blank? NotPresent: Convert usages of not present? to blank? UnlessPresent: Convert usages of unless present? to blank?

Example:

# NilOrEmpty: true
  # bad
  foo.nil? || foo.empty?
  foo == nil || foo.empty?

  # good
  foo.blank?

# NotPresent: true
  # bad
  !foo.present?

  # good
  foo.blank?

# UnlessPresent: true
  # bad
  something unless foo.present?
  unless foo.present?
    something
  end

  # good
  something if foo.blank?
  if foo.blank?
    something
  end

Missing magic comment # frozen_string_literal: true.
Open

module ApplicationHelper
Severity: Minor
Found in app/helpers/application_helper.rb by rubocop

This cop is designed to help upgrade to Ruby 3.0. It will add the comment # frozen_string_literal: true to the top of files to enable frozen string literals. Frozen string literals may be default in Ruby 3.0. The comment will be added below a shebang and encoding comment. The frozen string literal comment is only valid in Ruby 2.3+.

Example: EnforcedStyle: when_needed (default)

# The `when_needed` style will add the frozen string literal comment
# to files only when the `TargetRubyVersion` is set to 2.3+.
# bad
module Foo
  # ...
end

# good
# frozen_string_literal: true

module Foo
  # ...
end

Example: EnforcedStyle: always

# The `always` style will always add the frozen string literal comment
# to a file, regardless of the Ruby version or if `freeze` or `<<` are
# called on a string literal.
# bad
module Bar
  # ...
end

# good
# frozen_string_literal: true

module Bar
  # ...
end

Example: EnforcedStyle: never

# The `never` will enforce that the frozen string literal comment does
# not exist in a file.
# bad
# frozen_string_literal: true

module Baz
  # ...
end

# good
module Baz
  # ...
end

TODO found
Open

  # TODO: compressions
Severity: Minor
Found in app/models/artist.rb by fixme

TODO found
Open

  # TODO: add attachment - pdf???
Severity: Minor
Found in app/models/release.rb by fixme

TODO found
Open

  # TODO: get list of countries and country codes, for admin
Severity: Minor
Found in app/models/venue.rb by fixme

TODO found
Open

/TODO: slider touch/mobile

TODO found
Open

/TODO: links to other shops and digital releases

TODO found
Open

  # TODO: https://github.com/benmanns/tinypng
Severity: Minor
Found in app/models/slide.rb by fixme

TODO found
Open

  # TODO: automatically find lng/lat for self added events
Severity: Minor
Found in app/models/venue.rb by fixme

TODO found
Open

  # TODO: configure proper CSV, XML and JSON export
Severity: Minor
Found in app/admin/release.rb by fixme

TODO found
Open

/TODO: contact form

TODO found
Open

    // TODO: on big screens, 1px gets lost

TODO found
Open

  # TODO: http://stackoverflow.com/questions/28412310/rails-paperclip-tinypng-gem
Severity: Minor
Found in app/models/slide.rb by fixme

TODO found
Open

    = "TODO: we don't have content for this"

TODO found
Open

  # TODO: change image sizes?
Severity: Minor
Found in app/models/artist.rb by fixme
Severity
Category
Status
Source
Language