Defect #28409

Total estimated time value not visible when tracker has estimate time field disabled

Added by Sebastian Paluch about 2 years ago. Updated 7 days ago.

Status:NewStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:Issues
Target version:Candidate for next minor release
Resolution: Affected version:3.4.4

Description

There is a tracker that does have estimate time field disabled. The issue of that tracker can have subtasks of different trackers that do have estimated time. The estiamted time for the parent issue is not shown at all, it should show total estimated time, in brackets. This works fine for spend time.

before@2x.png (27.1 KB) Go MAEDA, 2018-04-03 04:35

after@2x.png (29.2 KB) Go MAEDA, 2018-04-03 04:35

tests.patch Magnifier (1.18 KB) Tomomi Yuzuriha, 2018-05-28 07:07

History

#1 Updated by Go MAEDA about 2 years ago

I agree with Sebastian Paluch. Here is a patch without tests.

Index: app/views/issues/show.html.erb
===================================================================
--- app/views/issues/show.html.erb    (revision 17253)
+++ app/views/issues/show.html.erb    (working copy)
@@ -64,7 +64,7 @@
   unless @issue.disabled_core_fields.include?('done_ratio')
     rows.right l(:field_done_ratio), progress_bar(@issue.done_ratio, :legend => "#{@issue.done_ratio}%"), :class => 'progress'
   end
-  unless @issue.disabled_core_fields.include?('estimated_hours')
+  unless @issue.disabled_core_fields.include?('estimated_hours') && @issue.total_estimated_hours.blank?
     rows.right l(:field_estimated_hours), issue_estimated_hours_details(@issue), :class => 'estimated-hours'
   end
   if User.current.allowed_to?(:view_time_entries, @project) && @issue.total_spent_hours > 0

Before:

After:

#2 Updated by Go MAEDA about 2 years ago

  • Target version set to Candidate for next minor release

#3 Updated by Tomomi Yuzuriha about 2 years ago

I've added the test-code for Go MAEDA's patch(#28409-1).

#4 Updated by Sebastian Paluch 7 days ago

Any chance to get this minor thing in soon?

Also available in: Atom PDF