Merge pull request #2207 from TracksApp/exists-instead-of-count

Use exists? instead of count > 0
This commit is contained in:
Matt Rogers 2019-05-04 14:54:21 -05:00 committed by GitHub
commit 996395cc80
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -12,8 +12,8 @@ class RecurringTodosController < ApplicationController
@recurring_todos = current_user.recurring_todos.active.includes(:tags, :taggings)
@completed_recurring_todos = current_user.recurring_todos.completed.limit(10).includes(:tags, :taggings)
@no_recurring_todos = @recurring_todos.count == 0
@no_completed_recurring_todos = @completed_recurring_todos.count == 0
@no_recurring_todos = @recurring_todos.exists?
@no_completed_recurring_todos = @completed_recurring_todos.exists?
@count = @recurring_todos.count
@new_recurring_todo = RecurringTodo.new