Project

General

Profile

Actions

Feature #20221

closed

Time entry query : column week

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

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

0%

Estimated time:
Resolution:
Fixed

Files


Related issues

Related to Redmine - Defect #26780: Translation for label_week in time report is not workingClosedJean-Philippe Lang

Actions
Actions #1

Updated by Jérôme BATAILLE over 8 years ago

Here is the patch working on any version of Redmine.

Please feel free to integrate this straightforward patch

Actions #3

Updated by Jérôme BATAILLE over 8 years ago

Here is a patch, only the unit test, there is no controller test about queries available columns

Actions #4

Updated by Toshi MARUYAMA over 8 years ago

  • Target version set to 3.2.0
Actions #5

Updated by Jean-Philippe Lang over 8 years ago

Does it make sense to display the week number without the year? Sorting on this column will result in time entries from different years appear with the same week.

Actions #6

Updated by Jérôme BATAILLE over 8 years ago

Jean-Philippe Lang wrote:

Does it make sense to display the week number without the year? Sorting on this column will result in time entries from different years appear with the same week.

I don't see a use case that makes this sort useful, but on the other hand displaying this column is useful to have the week number without duplicating informations in other columns (spent_on). The yyyy-ww sort can be done with spent_on.

Thank to have added this patch to 3.2.0

If you have some time to review my other pending patches I will be very pleased !

Actions #7

Updated by Jean-Philippe Lang over 8 years ago

Jérôme BATAILLE wrote:

I don't see a use case that makes this sort useful

Just asking... because the patch makes it sortable.

Actions #8

Updated by Jérôme BATAILLE over 8 years ago

Jean-Philippe Lang wrote:

Just asking... because the patch makes it sortable.

We are at the frontier of YAGNI :-) I can't see a use case, but perhaps some folks will, and adding the sort costs nothing.

Actions #9

Updated by Jean-Philippe Lang over 8 years ago

  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang
  • Resolution set to Fixed

Column added in r14723. But sort is done on spent_on, as sorting by week number does make no sense when we have time entries on several years.

Actions #10

Updated by Jérôme BATAILLE over 8 years ago

Thanks a lot !

Actions #11

Updated by Toshi MARUYAMA over 6 years ago

  • Related to Defect #26780: Translation for label_week in time report is not working added
Actions

Also available in: Atom PDF