diff options
author | Robert Speicher <rspeicher@gmail.com> | 2019-06-05 00:54:31 +0300 |
---|---|---|
committer | Robert Speicher <rspeicher@gmail.com> | 2019-06-05 00:54:31 +0300 |
commit | e794649b525fb8b5cb4f49d3d053e80efaf0682b (patch) | |
tree | 7f468ebe2c2c9d269f3f0c15734a2b14e53f7d10 /app/models | |
parent | 2e32e39350671e9d059592f91827a58ad04f0269 (diff) | |
parent | 9797b27e1a096f684b870e55aab0b5d1250e73f8 (diff) |
Merge branch 'ce-9578-adjust-opened-issues-count-and-completion-rate' into 'master'
Port: EE adjust burndown chart opened issues count
See merge request gitlab-org/gitlab-ce!28777
Diffstat (limited to 'app/models')
-rw-r--r-- | app/models/concerns/milestoneish.rb | 20 |
1 files changed, 6 insertions, 14 deletions
diff --git a/app/models/concerns/milestoneish.rb b/app/models/concerns/milestoneish.rb index e65bbb8ca07..3deb86da6cf 100644 --- a/app/models/concerns/milestoneish.rb +++ b/app/models/concerns/milestoneish.rb @@ -1,28 +1,20 @@ # frozen_string_literal: true module Milestoneish - def closed_items_count(user) - memoize_per_user(user, :closed_items_count) do - (count_issues_by_state(user)['closed'] || 0) + merge_requests.closed_and_merged.size - end - end - - def total_items_count(user) - memoize_per_user(user, :total_items_count) do - total_issues_count(user) + merge_requests.size - end - end - def total_issues_count(user) count_issues_by_state(user).values.sum end + def closed_issues_count(user) + count_issues_by_state(user)['closed'].to_i + end + def complete?(user) - total_items_count(user) > 0 && total_items_count(user) == closed_items_count(user) + total_issues_count(user) > 0 && total_issues_count(user) == closed_issues_count(user) end def percent_complete(user) - ((closed_items_count(user) * 100) / total_items_count(user)).abs + closed_issues_count(user) * 100 / total_issues_count(user) rescue ZeroDivisionError 0 end |