Feature #30346
closedAdd "Target Version" to the list of "Available columns" in "Spent time" Tab
Added by VD DV about 6 years ago. Updated about 5 years ago.
0%
Description
I am evaluating last Redmine version (v4.0.0), and I have discovered that "Target Version" is missing at "Spent time" Tab in the list of "Available columns" while selecting columns for timelogs. Therefore, CSV export from "Spent time" TAB is also missing that valuable informatin for further spent time analysis.
It is clear that versions are tied to projects, and same "Target Version" literal in different projects might not be the same version, therefore might lead to confusion.
But, "Target Version" together with "Project" information (that is already selectable at "Available columns") can avoid any unclearness.
Files
report.png (217 KB) report.png | Marius BĂLTEANU, 2019-01-07 05:22 | ||
filter.png (310 KB) filter.png | Marius BĂLTEANU, 2019-01-07 05:22 | ||
0001-Adds-Target-Version-column-to-the-list-of-Available-.patch (3.53 KB) 0001-Adds-Target-Version-column-to-the-list-of-Available-.patch | Marius BĂLTEANU, 2019-11-06 22:36 |
Related issues
Updated by Marius BĂLTEANU about 6 years ago
- File filter.png filter.png added
- File report.png report.png added
- Status changed from New to Needs feedback
- Assignee set to VD DV
In which version of Redmine did you see the "Target Version" (Version) as available column in "Spent time" tab? I've double checked now in Redmine 3.4 and Redmine 4.0 and is not available as column, it is available as filter and group criteria in the Reports tab (not Details). Please see below screenshots.
Updated by VD DV almost 6 years ago
You are right. There is no such column in previous versions. Therefore this issue might be evaluated not as deffect but as a new feature.
Updated by Marius BĂLTEANU almost 6 years ago
- Tracker changed from Defect to Feature
- Subject changed from "Target Version" is missing at "Spent time" Tab in the list of "Available columns" to Add "Target Version" to the list of "Available columns" in "Spent time" Tab
- Status changed from Needs feedback to New
- Assignee deleted (
VD DV)
Updated by Marius BĂLTEANU almost 6 years ago
- Related to Feature #14130: Spent Time Report Filter options added
Updated by Pablo Sanxiao over 5 years ago
What about this issue? Is it going to plan to be added for next redmine version?
I think it's quite easy to implement and very useful and demanded. I don't know too much about redmine source code but I think patch would be very similar to this one: https://www.redmine.org/issues/28391
Updated by Go MAEDA about 5 years ago
- Related to Feature #23401: Add tracker and status columns/filters to detailed timelog added
Updated by Go MAEDA about 5 years ago
- Related to Feature #28391: Add issue category filter and column to spent time queries added
Updated by Go MAEDA about 5 years ago
+1
Spent time tab has 4 types of filters for core fields of issues. They are Tracker, Status, Target version, and Category. However, only Target version is not included in available columns. I think it is quite odd that the field can be used as a filter is not included in available columns.
I think the target version field should have been added to available columns when #23401 was implemented.
Before r15738:Field | Filters | Available columns |
---|---|---|
Tracker | - | - |
Status | - | - |
Target version | ✓ | - |
Field | Filters | Available columns |
---|---|---|
Tracker | ✓ | ✓ |
Status | ✓ | ✓ |
Target version | ✓ | - |
Updated by Marius BĂLTEANU about 5 years ago
- File 0001-Adds-Target-Version-column-to-the-list-of-Available-.patch added
- Target version set to Candidate for next major release
Here is a patch for this feature. Test results here.
Updated by Marius BĂLTEANU about 5 years ago
- File deleted (
0001-Adds-Target-Version-column-to-the-list-of-Available-.patch)
Updated by Marius BĂLTEANU about 5 years ago
Marius BALTEANU wrote:
Here is a patch for this feature. Test results here.
There was an error in my proposed patch, I'll repost the patch soon.
Updated by Marius BĂLTEANU about 5 years ago
- File 0001-Adds-Target-Version-column-to-the-list-of-Available-.patch added
Here it is.
Updated by Marius BĂLTEANU about 5 years ago
- File deleted (
0001-Adds-Target-Version-column-to-the-list-of-Available-.patch)
Updated by Go MAEDA about 5 years ago
- Target version changed from Candidate for next major release to 4.1.0
Thank you for posting the patch. But I had to change it as follows. Without the change, the id of the version is shown in the column instead of the name.
Setting the target version to 4.1.0.
diff --git a/app/models/time_entry_query.rb b/app/models/time_entry_query.rb
index 77929e740..ed0d4dfbd 100644
--- a/app/models/time_entry_query.rb
+++ b/app/models/time_entry_query.rb
@@ -34,7 +34,7 @@ class TimeEntryQuery < Query
QueryAssociationColumn.new(:issue, :tracker, :caption => :field_tracker, :sortable => "#{Tracker.table_name}.position"),
QueryAssociationColumn.new(:issue, :status, :caption => :field_status, :sortable => "#{IssueStatus.table_name}.position"),
QueryAssociationColumn.new(:issue, :category, :caption => :field_category, :sortable => "#{IssueCategory.table_name}.name"),
- QueryAssociationColumn.new(:issue, :fixed_version_id, :caption => :field_fixed_version, :sortable => Version.fields_for_order_statement),
+ QueryAssociationColumn.new(:issue, :fixed_version, :caption => :field_fixed_version, :sortable => Version.fields_for_order_statement),
QueryColumn.new(:comments),
QueryColumn.new(:hours, :sortable => "#{TimeEntry.table_name}.hours", :totalable => true),
]
Updated by Marius BĂLTEANU about 5 years ago
Go MAEDA wrote:
Thank you for posting the patch. But I had to change it as follows. Without the change, the id of the version is shown in the column instead of the name.
Setting the target version to 4.1.0.
[...]
Indeed, :fixed_version
should be, I think I've mixed up some things before posting the patch.
Updated by Go MAEDA about 5 years ago
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Resolution set to Fixed
Committed the patch. Thank you.
Updated by Go MAEDA about 5 years ago
- Related to Patch #32478: Add missing fixture to TimelogControllerTest added