Project

General

Profile

Actions

Patch #26728

closed

count > 0 vs exists?

Added by jwjw yy over 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Code cleanup/refactoring
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

redmine/app/models/project.rb:671:      if versions.count > 0
redmine/app/views/imports/show.html.erb:3:<% if @import.saved_items.count > 0 %>
redmine/app/views/imports/show.html.erb:15:<% if @import.unsaved_items.count > 0 %>

could be

redmine/app/models/project.rb:671:      if versions.exists?
redmine/app/views/imports/show.html.erb:3:<% if @import.saved_items.exists? %>
redmine/app/views/imports/show.html.erb:15:<% if @import.unsaved_items.exists?%>

Related issues

Related to Redmine - Patch #24839: Minor performance improvement - Replace count by exists?ClosedJean-Philippe Lang

Actions
Actions #1

Updated by Toshi MARUYAMA over 6 years ago

  • Category set to Code cleanup/refactoring
  • Status changed from New to Needs feedback

Please post by patch.

Actions #2

Updated by Toshi MARUYAMA over 6 years ago

And please describe why this change is effective.

Actions #3

Updated by Toshi MARUYAMA over 6 years ago

Toshi MARUYAMA wrote:

Please post by patch.

Please post by attachment patch file.

Actions #4

Updated by Go MAEDA over 5 years ago

  • Related to Patch #24839: Minor performance improvement - Replace count by exists? added
Actions #5

Updated by Go MAEDA over 5 years ago

  • Status changed from Needs feedback to Closed

We have decided not to use exists? there. Please see #24839#note-1 and #24839#note-3.

Actions

Also available in: Atom PDF