Redmine

The easiest and safest way to achieve this is to render the column content only when the spent time or total spent time is > 0:

diff --git a/app/helpers/queries_helper.rb b/app/helpers/queries_helper.rb
index 9b223f84a..401b8d776 100644
--- a/app/helpers/queries_helper.rb
+++ b/app/helpers/queries_helper.rb
@@ -268,9 +268,13 @@ module QueriesHelper
       when :hours, :estimated_hours, :total_estimated_hours
         format_hours(value)
       when :spent_hours
-        link_to_if(value > 0, format_hours(value), project_time_entries_path(item.project, :issue_id => "#{item.id}"))
+        if value > 0
+          link_to(format_hours(value), project_time_entries_path(item.project, :issue_id => "#{item.id}"))
+        end
       when :total_spent_hours
-        link_to_if(value > 0, format_hours(value), project_time_entries_path(item.project, :issue_id => "~#{item.id}"))
+        if value > 0
+          link_to(format_hours(value), project_time_entries_path(item.project, :issue_id => "~#{item.id}"))
+        end
       when :attachments
         value.to_a.map {|a| format_object(a)}.join(" ").html_safe

The current issue where an issue with spent time added with 0 hours cannot be distinguish from an issue with no spent times will remain.

I'm in favour of this change because the view will be cleaner and consistent with estimated time columns.

Here is a patch based on the one Marius posted in #note-2, with only minor changes.

Changes from the original code:

• Changed the column value to an empty string instead of nil when the value is 0, to be consistent with the other branches in the same case statement. The rendered output is the same.
• Updated the tests to verify that "0:00" is no longer displayed in the Spent time and Total spent time columns.

As Marius noted, an issue that has only 0-hour time entries cannot be distinguished from one with no time entries. I think this is acceptable because the issue page already shows the "Spent time" row only when the total is greater than 0, and such entries require the "Accept time logs with 0 hours" setting.