##// END OF EJS Templates
Fixed that link to the assignee is escaped twice on the issue view (#11352)....
Jean-Philippe Lang -
r9784:165c2be523be
parent child
Show More
@@ -37,7 +37,7
37 rows.left l(:field_priority), h(@issue.priority.name), :class => 'priority'
37 rows.left l(:field_priority), h(@issue.priority.name), :class => 'priority'
38
38
39 unless @issue.disabled_core_fields.include?('assigned_to_id')
39 unless @issue.disabled_core_fields.include?('assigned_to_id')
40 rows.left l(:field_assigned_to), avatar(@issue.assigned_to, :size => "14").to_s + (@issue.assigned_to ? link_to_user(@issue.assigned_to) : "-"), :class => 'assigned-to'
40 rows.left l(:field_assigned_to), avatar(@issue.assigned_to, :size => "14").to_s.html_safe + (@issue.assigned_to ? link_to_user(@issue.assigned_to) : "-"), :class => 'assigned-to'
41 end
41 end
42 unless @issue.disabled_core_fields.include?('category_id')
42 unless @issue.disabled_core_fields.include?('category_id')
43 rows.left l(:field_category), h(@issue.category ? @issue.category.name : "-"), :class => 'category'
43 rows.left l(:field_category), h(@issue.category ? @issue.category.name : "-"), :class => 'category'
@@ -1114,6 +1114,14 class IssuesControllerTest < ActionController::TestCase
1114 assert_no_tag 'a', :content => /Next/
1114 assert_no_tag 'a', :content => /Next/
1115 end
1115 end
1116
1116
1117 def test_show_should_display_link_to_the_assignee
1118 get :show, :id => 2
1119 assert_response :success
1120 assert_select '.assigned-to' do
1121 assert_select 'a[href=/users/3]'
1122 end
1123 end
1124
1117 def test_show_should_display_visible_changesets_from_other_projects
1125 def test_show_should_display_visible_changesets_from_other_projects
1118 project = Project.find(2)
1126 project = Project.find(2)
1119 issue = project.issues.first
1127 issue = project.issues.first
General Comments 0
You need to be logged in to leave comments. Login now