diff --git a/app/helpers/queries_helper.rb b/app/helpers/queries_helper.rb index 3aef7083a..c88d589b3 100644 --- a/app/helpers/queries_helper.rb +++ b/app/helpers/queries_helper.rb @@ -227,7 +227,7 @@ module QueriesHelper end content = link_to( - sprite_icon(icon, column.caption), + icon ? sprite_icon(icon, column.caption) : column.caption, {:params => request.query_parameters.deep_merge(sort_param)}, link_options ) diff --git a/test/functional/issues_controller_test.rb b/test/functional/issues_controller_test.rb index 5227efff5..5269fbcb4 100644 --- a/test/functional/issues_controller_test.rb +++ b/test/functional/issues_controller_test.rb @@ -1242,6 +1242,12 @@ class IssuesControllerTest < Redmine::ControllerTest issues_in_list ) assert_select 'table.issues.sort-by-tracker.sort-asc' + + # Ensure that only the primary sort key column has a sort order icon + assert_select 'th.tracker svg', 1 + css_select('table.issues th:not(.tracker)').each do |th| + assert_select th, 'svg', 0 + end end def test_index_sort_by_field_not_included_in_columns