Re-add and apply Rails/UniqBeforePluck rule

We removed it in commit d639cd58 because it recommended using `uniq`
where `distinct` was more appropriate. This has been fixed in
rubocop-rails 2.6.0.
This commit is contained in:
Javi Martín
2020-10-11 20:47:51 +02:00
parent d7e6a5c6bf
commit 4658e18db5
2 changed files with 4 additions and 1 deletions

View File

@@ -300,6 +300,9 @@ Rails/SkipsModelValidations:
Rails/TimeZone:
Enabled: true
Rails/UniqBeforePluck:
Enabled: true
Rails/UniqueValidationWithoutIndex:
Enabled: true
Severity: refactor

View File

@@ -1,6 +1,6 @@
class Admin::StatsController < Admin::BaseController
def show
@event_types = Ahoy::Event.pluck(:name).uniq.sort
@event_types = Ahoy::Event.distinct.order(:name).pluck(:name)
@visits = Visit.count
@debates = Debate.with_hidden.count