Actions
Defect #31779
closedTotal estimated time column shown even when estimated time field is deactivated
Start date:
Due date:
% Done:
0%
Estimated time:
Resolution:
Fixed
Affected version:
Description
Even when the estimated time field is deactivated in the trackers available in the issue list the total estimated time column is available. This diff patches this issue, though maybe fixing it in Tracker.disabled_core_fields
would be better.
--- a/app/models/issue_query.rb +++ b/app/models/issue_query.rb @@ -288,6 +288,7 @@ class IssueQuery < Query end disabled_fields = Tracker.disabled_core_fields(trackers).map {|field| field.sub(/_id$/, '')} + disabled_fields << "total_estimated_hours" if disabled_fields.include?("estimated_hours") @available_columns.reject! {|column| disabled_fields.include?(column.name.to_s) }
Files
Updated by Kevin Fischer over 5 years ago
Since it's not a "core field" in the original sense, fixing it here seems good to me
+1
Updated by Yuichi HARADA over 5 years ago
- File 31779_add_test.patch 31779_add_test.patch added
Felix Schäfer wrote:
Even when the estimated time field is deactivated in the trackers available in the issue list the total estimated time column is available. This diff patches this issue, though maybe fixing it in
Tracker.disabled_core_fields
would be better.[...]
I created a test. I attached a patch.
Updated by Go MAEDA over 5 years ago
- Target version changed from 4.0.5 to 3.4.12
Updated by Jean-Philippe Lang about 5 years ago
- Status changed from New to Closed
- Assignee set to Jean-Philippe Lang
- Resolution set to Fixed
Committed, thanks.
Actions