Defect #31779

Total estimated time column shown even when estimated time field is deactivated

Added by Felix Schäfer about 1 month ago. Updated about 12 hours ago.

Status:NewStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:Issues list
Target version:3.4.12
Resolution: 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)
     }


31779_add_test.patch Magnifier (1.12 KB) Yuichi HARADA, 2019-09-18 06:58

History

#1 Updated by Kevin Fischer about 1 month ago

Since it's not a "core field" in the original sense, fixing it here seems good to me
+1

#2 Updated by Go MAEDA 30 days ago

  • Target version set to 4.0.5

#3 Updated by Yuichi HARADA about 20 hours ago

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.

#4 Updated by Go MAEDA about 12 hours ago

  • Target version changed from 4.0.5 to 3.4.12

Also available in: Atom PDF