AgileVentures/LocalSupport

View on GitHub

Showing 792 of 792 total issues

Pass &:capitalize as an argument to map instead of a block.
Open

    phrase.humanize.split(' ').map{|w| w.capitalize}.join(' ')

Use symbols as procs when possible.

Example:

# bad
something.map { |s| s.upcase }

# good
something.map(&:upcase)

Line is too long. [128/90] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)
Open

  has_and_belongs_to_many :base_organisations, :through => :category_organisations, :association_foreign_key => :organisation_id
Severity: Minor
Found in app/models/category.rb by rubocop

Adjoining classes: .bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-danger:active
Open

.bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-danger:active,

Adjoining classes: .bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-danger.active
Open

.bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-danger.active {

Adjoining classes: .bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-default:focus
Open

.bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-default:focus,

Adjoining classes: .bootstrap-switch .bootstrap-switch-handle-on.bootstrap-switch-default.disabled
Open

.bootstrap-switch .bootstrap-switch-handle-on.bootstrap-switch-default.disabled,

Adjoining classes: .bootstrap-switch .bootstrap-switch-handle-on.bootstrap-switch-default:active
Open

.bootstrap-switch .bootstrap-switch-handle-on.bootstrap-switch-default:active,

Adjoining classes: .bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-default:active
Open

.bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-default:active,

Adjoining classes: .bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-default:active
Open

.bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-default:active,

Adjoining classes: .bootstrap-switch .bootstrap-switch-label.disabled
Open

.bootstrap-switch .bootstrap-switch-label.disabled,

Adjoining classes: .bootstrap-switch .bootstrap-switch-label.active
Open

.bootstrap-switch .bootstrap-switch-label.active {

Line exceeds maximum allowed length
Open

  $("input[type='checkbox'].publish").bootstrapSwitch(size: 'mini', onText: 'public', offText: 'private')

Line exceeds maximum allowed length
Open

# You can use CoffeeScript in this file: http://jashkenas.github.com/coffee-script/

Line is too long. [104/90] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)
Open

  validates_url :donation_info, preferred_scheme: 'http://', message: 'Donation url is not a valid URL',
Severity: Minor
Found in app/models/base_organisation.rb by rubocop

Rename is_organisation_removed? to organisation_removed?. (https://github.com/bbatsov/ruby-style-guide#bool-methods-qmark)
Open

  def is_organisation_removed?
Severity: Minor
Found in app/models/csv_organisation.rb by rubocop

This cop makes sure that predicates are named properly.

Example:

# bad
def is_even?(value)
end

# good
def even?(value)
end

# bad
def has_value?
end

# good
def value?
end

Unused method argument - org. If it's necessary, use _ or _org as an argument name to indicate that it won't be used. You can also write as can_delete?(*) if you want the method to accept any arguments but don't care about them. (https://github.com/bbatsov/ruby-style-guide#underscore-unused-vars)
Open

  def can_delete? org
Severity: Minor
Found in app/models/user.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

Adjoining classes: .bootstrap-switch .bootstrap-switch-handle-on.bootstrap-switch-danger.active
Open

.bootstrap-switch .bootstrap-switch-handle-on.bootstrap-switch-danger.active,

Adjoining classes: .bootstrap-switch .bootstrap-switch-handle-on.bootstrap-switch-danger.active
Open

.bootstrap-switch .bootstrap-switch-handle-on.bootstrap-switch-danger.active,

Adjoining classes: .bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-danger.disabled
Open

.bootstrap-switch .bootstrap-switch-handle-off.bootstrap-switch-danger.disabled,

Rule doesn't have all its properties in alphabetical order.
Open

.bootstrap-switch .bootstrap-switch-handle-on.bootstrap-switch-default:hover,
Severity
Category
Status
Source
Language