Project

General

Profile

Actions

Defect #12803

closed

Cross-year date range in time tracking report misses some hours

Added by Murray Melvin about 11 years ago. Updated about 11 years ago.

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

0%

Estimated time:
Resolution:
Duplicate
Affected version:

Description

How to reproduce:

  1. Have some tickets that have time tracked in both 2012 and 2013.
  2. Select "last 2 weeks" date preset (or manually enter 2012-12-24 to 2013-01-06)
  3. Select the Report tab
  4. Add Details | Week- Project
  5. Add Details | Week- Issue
  6. Note that the total at the top of the report does not equal the total in the bottom right corner of the report. Times from 2013 have been omitted, although the projects and issues are included.
  7. Also note that the weeks shown in the column headers are 2012-52 and 2012-1

This has also been reported to be a problem in 1.4.4.stable.9879.


Related issues

Is duplicate of Redmine - Defect #5329: Time entries report broken on first week of yearClosedJean-Philippe Lang2010-04-15

Actions
Actions #1

Updated by Uwe Koloska about 11 years ago

Here is a patch against 1.4.4.stable.9879 (the lines that have to be changed are the same in trunk)

--- lib/redmine/helpers/time_report.rb  (Revision 9879)
+++ lib/redmine/helpers/time_report.rb  (Arbeitskopie)
@@ -67,7 +67,7 @@
             when 'month'
               row['month'] = "#{row['tyear']}-#{row['tmonth']}" 
             when 'week'
-              row['week'] = "#{row['tyear']}-#{row['tweek']}" 
+              row['week'] = Date.parse("#{row['spent_on']}").cwyear.to_s + "-#{row['tweek']}" 
             when 'day'
               row['day'] = "#{row['spent_on']}" 
             end
@@ -98,7 +98,7 @@
               @periods << "#{date_from.year}-#{date_from.month}" 
               date_from = (date_from + 1.month).at_beginning_of_month
             when 'week'
-              @periods << "#{date_from.year}-#{date_from.to_date.cweek}" 
+              @periods << "#{date_from.to_date.cwyear}-#{date_from.to_date.cweek}" 
               date_from = (date_from + 7.day).at_beginning_of_week
             when 'day'
               @periods << "#{date_from.to_date}" 

Reason is the difference between the year for the week and the year from the date. The first ISO week of this year looks like this in the time_entries table:

tyear tmonth tweek spent_on
2012 12 1 2012-12-31
2013 1 1 2013-01-01
2013 1 1 2013-01-02
2013 1 1 2013-01-03
Maybe someone could help me with the following questions, as I'm only a noob with redmine and rails development:
  • is there a more elegant way to get the year for the ISO week than parsing spent_on?
  • How can I create a test for this bug?
  • How can I use the logger in this function? logger.debug gives undefined local variable or method `logger'
Actions #2

Updated by Filou Centrinov about 11 years ago

Duplicate of #5329

Actions #3

Updated by Jean-Philippe Lang about 11 years ago

  • Status changed from New to Closed
  • Resolution set to Duplicate
Actions

Also available in: Atom PDF