Project

General

Profile

Actions

Defect #14737

closed

Gantt, completed % truncated instead of rounded

Added by Jérôme BATAILLE over 10 years ago. Updated almost 10 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Gantt
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Files

RM_14737_gantt_completed_pourcent_truncated.diff (834 Bytes) RM_14737_gantt_completed_pourcent_truncated.diff Jérôme BATAILLE, 2013-08-21 16:23
RM_14737_gantt_completed_pourcent_truncated.diff (778 Bytes) RM_14737_gantt_completed_pourcent_truncated.diff Jérôme BATAILLE, 2013-08-22 15:29
edit.png (6 KB) edit.png Toshi MARUYAMA, 2014-05-14 13:45
view.png (5.79 KB) view.png Toshi MARUYAMA, 2014-05-14 13:45

Related issues

Has duplicate Redmine - Feature #13442: Gantt : rounding error for version completed percentClosed

Actions
Actions #2

Updated by Daniel Felix over 10 years ago

+1 to the patch.
But English comments would be better. ;-)

Actions #4

Updated by Daniel Felix over 10 years ago

Np. Looks better. :-)

Actions #5

Updated by Jérôme BATAILLE almost 10 years ago

Any chance that this very simple patch could be integrated to the core ?

Actions #6

Updated by Toshi MARUYAMA almost 10 years ago

This problem is not only pdf but also web form.

diff --git a/app/views/issues/_attributes.html.erb b/app/views/issues/_attributes.html.erb
--- a/app/views/issues/_attributes.html.erb
+++ b/app/views/issues/_attributes.html.erb
@@ -63,7 +63,7 @@
 <% end %>

 <% if @issue.safe_attribute? 'estimated_hours' %>
-<p><%= f.text_field :estimated_hours, :size => 3, :disabled => !@issue.leaf?, :required => @issue.required_attribute?('estimated_hours') %> <%= l(:field_hours) %></p>
+<p><%= f.text_field :estimated_hours, :size => 10, :disabled => !@issue.leaf?, :required => @issue.required_attribute?('estimated_hours') %> <%= l(:field_hours) %></p>
 <% end %>

 <% if @issue.safe_attribute?('done_ratio') && @issue.leaf? && Issue.use_field_for_done_ratio? %>


Actions #7

Updated by Jérôme BATAILLE almost 10 years ago

Hi Toshi,

You are talking about estimated_hours, do you mean that this is linked to done_ratio ?
The only place I have found this issue is in the Gantt, on the versions display percent is not truncated.

Actions #8

Updated by Jérôme BATAILLE almost 10 years ago

Duplicated by #13442

Actions #9

Updated by Toshi MARUYAMA almost 10 years ago

Jérôme BATAILLE wrote:

Hi Toshi,

You are talking about estimated_hours, do you mean that this is linked to done_ratio ?
The only place I have found this issue is in the Gantt, on the versions display percent is not truncated.

Sorry, I confused with #12510#note-15.

Actions #10

Updated by Toshi MARUYAMA almost 10 years ago

  • Target version set to 2.5.2
Actions #11

Updated by Toshi MARUYAMA almost 10 years ago

  • Target version changed from 2.5.2 to 2.6.0
Actions #12

Updated by Toshi MARUYAMA almost 10 years ago

  • Related to Feature #13442: Gantt : rounding error for version completed percent added
Actions #13

Updated by Toshi MARUYAMA almost 10 years ago

  • Related to deleted (Feature #13442: Gantt : rounding error for version completed percent)
Actions #14

Updated by Toshi MARUYAMA almost 10 years ago

  • Has duplicate Feature #13442: Gantt : rounding error for version completed percent added
Actions #15

Updated by Toshi MARUYAMA almost 10 years ago

It is reasonable because issue form uses round.
source:tags/2.5.1/app/helpers/application_helper.rb#L1122

Actions #16

Updated by Toshi MARUYAMA almost 10 years ago

  • Status changed from New to Closed
  • Resolution set to Fixed

Committed in trunk r13137, thanks.

Actions #17

Updated by Jérôme BATAILLE almost 10 years ago

Thanks a lot ! this is Smile 13'th patch integrated into the core.

Actions

Also available in: Atom PDF