Project

General

Profile

Actions

Patch #13400

open

Calculate %done based on estimated and logged time

Added by Holger Just about 11 years ago. Updated over 5 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Issues planning
Start date:
Due date:
% Done:

0%

Estimated time:

Description

One of our customers at Planio requested to have the %done field calculated based on estimated and logged time. This patch is extracted from the Planio code base.

If enabled, it calculated the percent done according to these rules:

  • if estimated time is set: logged_time / estimated_time
    • the number is never higher than 100%, even if logged_time > estimated_time
  • if estimated_time is not set: 0

Logged and estimated times of child issues are taken into account, similarly to the way it is done when using the issue status. If a user logs/modifies/deletes time, it creates a journal entry of the changed %done on the affected issue. It will not create journals on parent issues that might also be updated.

The patch is attached.


Files


Related issues

Related to Redmine - Feature #12762: Add option to calculate done ratio with the data from time trackingNew

Actions
Related to Redmine - Feature #24277: Introducing Remaining Time field as method to track the remaining time to complete an issueNew

Actions
Actions #2

Updated by Mischa The Evil about 11 years ago

Jan Niggemann wrote:

Dupe of #12762?

I'd say just related since this is a patch which implements feature #12762.

Actions #3

Updated by Etienne Massip about 11 years ago

Could you use rather use Issue model time_entries association after_add and after_remove callbacks rather than TimeEntry model ones?

Actions #4

Updated by Etienne Massip about 11 years ago

  • Category set to Issues planning
Actions #5

Updated by Holger Just about 11 years ago

Etienne Massip wrote:

Could you use rather use Issue model time_entries association after_add and after_remove callbacks rather than TimeEntry model ones?

Well, you can add/edit/delete time entries all without even touching an issue. I'm not sure how that is supposed to work as in this case, the issues isn't even loaded.

Also one tiny remark which I forgot above: the patch is against Redmine 1.4.7. If this is something that should go into the core, I could probably port that to 2.2 (haven't tested it there).

Actions #6

Updated by Etienne Massip about 11 years ago

Holger Just wrote:

Also one tiny remark which I forgot above: the patch is against Redmine 1.4.7. If this is something that should go into the core, I could probably port that to 2.2 (haven't tested it there).

I think it would have its place in core so it would be great to have this patch against a recent trunk revision.

Actions #7

Updated by Etienne Massip about 11 years ago

Holger Just wrote:

Etienne Massip wrote:

Could you use rather use Issue model time_entries association after_add and after_remove callbacks rather than TimeEntry model ones?

Well, you can add/edit/delete time entries all without even touching an issue. I'm not sure how that is supposed to work as in this case, the issues isn't even loaded.

I didn't think that the actual TimeEntry model was this much tied to the issue the time entry belongs to, looking a bit more deeply in its code, it's very OK like you did.

Actions #8

Updated by Mischa The Evil about 11 years ago

FTR: related and duplicating issues are listed in DoneRatio_issues_overview.

Actions #9

Updated by Toshi MARUYAMA over 10 years ago

0001-patch.diff seems for trunk r9702.
I removed languages except en and de.

Actions #10

Updated by Toshi MARUYAMA over 10 years ago

This is a patch rebased to trunk r12075.

Actions #11

Updated by Vincent Adelé almost 9 years ago

I applied the patch issue-13400-trunk-r12075.diff on Redmine 2.4.2.stable with success but I get that error when submitting an issue:

NameError (undefined local variable or method `update_done_ratio_from_issue_status' for #<Issue:0x007f6a2ca42408>):
  app/models/issue.rb:165:in `create_or_update'
  app/controllers/issues_controller.rb:470:in `block in save_issue_with_child_records'
  app/controllers/issues_controller.rb:458:in `save_issue_with_child_records'
  app/controllers/issues_controller.rb:184:in `update'

Is there any other solution to get this feature on 2.4.2 ?

Actions #12

Updated by Peter Drábik almost 9 years ago

Vincent Adelé wrote:

I applied the patch issue-13400-trunk-r12075.diff on Redmine 2.4.2.stable with success but I get that error when submitting an issue:
[...]

Is there any other solution to get this feature on 2.4.2 ?

I have this problem on 3.0.2 and I need this feature

Actions #13

Updated by Gregor Schmidt over 5 years ago

I recently came across this issue while working at Planio. I've rebased Holger's patch and added some tests.

It would be great, if this feature suggestion could be reconsidered.

Actions #14

Updated by Holger Just over 5 years ago

  • Target version set to Candidate for next minor release

Thank you Gregor for working on this. I think this is a great addition.

I'm setting the version to Candidate for next minor release, especially in the light of DoneRatio issues overview.

Actions #16

Updated by Go MAEDA over 5 years ago

There are many similar requests: #952, #3719, #7545, #11999, #12762, #27965

Actions #17

Updated by Go MAEDA over 5 years ago

  • Related to Feature #24277: Introducing Remaining Time field as method to track the remaining time to complete an issue added
Actions #18

Updated by Marius BĂLTEANU over 5 years ago

I would like to discuss the implementation of #24277 instead of this feature because I found it more complete and flexible for the users and also, it incorporates this functionality by default.

Actions #19

Updated by Go MAEDA over 5 years ago

I think the proposed feature is appropriate to say "% spent" rather than "% done". And it shows strange "% done" value for closed issues in some circumstances. Suppose that a person finished an issue in 5 hours, and the issue was estimated to take 10 hours. The person finished the issue ahead of schedule but the closed issue shows that "% done" is 50%.

Actions #20

Updated by Gregor Schmidt over 5 years ago

Thank you for your feedback:

Marius Ionescu BALTEANU: Thank you for pointing out #24277. I think it's a nice addition. My main goal was to keep this issue here alive, so that people are able to use the feature eventually.

@Go MAEDA: I think, one may argue both ways. Yes it's weird, that the issue is less than 100 % done, but closed already. On the other hand, this may be seen as a simple indicator, that you've stayed below budget.

In the end, I think Redmine should be consistent. Since the fields are also not set to 100 % when closing an issue with the "use the issue field" setting, I think it's expected to handle it the same way with this new setting. The status is only taken into account, when selecting "Use the issue status".

Actions

Also available in: Atom PDF