From c7f03c00a92289f743298cf9bd0b4a13b0ac3fae 2017-01-16 18:12:49 From: Jean-Philippe Lang Date: 2017-01-16 18:12:49 Subject: [PATCH] Use exists? instead of count (#24839). Patch by Vincent Robert and Go MAEDA. git-svn-id: http://svn.redmine.org/redmine/trunk@16211 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- diff --git a/app/models/repository/git.rb b/app/models/repository/git.rb index 26e0d39..195c878 100644 --- a/app/models/repository/git.rb +++ b/app/models/repository/git.rb @@ -143,7 +143,7 @@ class Repository::Git < Repository return if prev_db_heads.sort == repo_heads.sort h["db_consistent"] ||= {} - if changesets.count == 0 + if ! changesets.exists? h["db_consistent"]["ordering"] = 1 merge_extra_info(h) self.save diff --git a/app/views/trackers/index.html.erb b/app/views/trackers/index.html.erb index 5f40cf0..7ee6387 100644 --- a/app/views/trackers/index.html.erb +++ b/app/views/trackers/index.html.erb @@ -16,7 +16,7 @@ "> <%= link_to tracker.name, edit_tracker_path(tracker) %> - <% unless tracker.workflow_rules.count > 0 %> + <% unless tracker.workflow_rules.exists? %> <%= l(:text_tracker_no_workflow) %> (<%= link_to l(:button_edit), workflows_edit_path(:tracker_id => tracker) %>)