From 6416bb5f5150f3b3077ae39e088f6b9fdf63f43a Mon Sep 17 00:00:00 2001 From: Go MAEDA Date: Sun, 14 Jun 2020 06:56:05 +0000 Subject: [PATCH] Done ratio of a parent issue may be shown as 99% even though all subtasks are completed (#33576). Patch by Go MAEDA. git-svn-id: http://svn.redmine.org/redmine/trunk@19818 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- app/models/issue.rb | 6 +++--- test/unit/issue_subtasking_test.rb | 10 ++++++++++ 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/app/models/issue.rb b/app/models/issue.rb index e665a46cb..a89f23dd8 100644 --- a/app/models/issue.rb +++ b/app/models/issue.rb @@ -1731,12 +1731,12 @@ class Issue < ActiveRecord::Base if children.any? child_with_total_estimated_hours = children.select {|c| c.total_estimated_hours.to_f > 0.0} if child_with_total_estimated_hours.any? - average = child_with_total_estimated_hours.map(&:total_estimated_hours).sum.to_f / child_with_total_estimated_hours.count + average = child_with_total_estimated_hours.map(&:total_estimated_hours).sum.to_d / child_with_total_estimated_hours.count else - average = 1.0 + average = 1.0.to_d end done = children.map {|c| - estimated = c.total_estimated_hours.to_f + estimated = (c.total_estimated_hours || 0.0).to_d estimated = average unless estimated > 0.0 ratio = c.closed? ? 100 : (c.done_ratio || 0) estimated * ratio diff --git a/test/unit/issue_subtasking_test.rb b/test/unit/issue_subtasking_test.rb index 85772909b..2826162f6 100644 --- a/test/unit/issue_subtasking_test.rb +++ b/test/unit/issue_subtasking_test.rb @@ -241,6 +241,16 @@ class IssueSubtaskingTest < ActiveSupport::TestCase end end + def test_done_ratio_of_parent_with_completed_children_should_not_be_99 + with_settings :parent_issue_done_ratio => 'derived' do + parent = Issue.generate! + parent.generate_child!(:estimated_hours => 8.0, :done_ratio => 100) + parent.generate_child!(:estimated_hours => 8.1, :done_ratio => 100) + # (8.0 * 100 + 8.1 * 100) / (8.0 + 8.1) => 99.99999999999999 + assert_equal 100, parent.reload.done_ratio + end + end + def test_changing_parent_should_update_previous_parent_done_ratio with_settings :parent_issue_done_ratio => 'derived' do first_parent = Issue.generate! -- 2.39.5