Commits

Anonymous committed 523b182

Fixed that link to the assignee is escaped twice on the issue view (#11352).

  • Participants
  • Parent commits 6954f4f

Comments (0)

Files changed (2)

File app/views/issues/show.html.erb

   rows.left l(:field_priority), h(@issue.priority.name), :class => 'priority'
 
   unless @issue.disabled_core_fields.include?('assigned_to_id')
-    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'
+    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'
   end
   unless @issue.disabled_core_fields.include?('category_id')
     rows.left l(:field_category), h(@issue.category ? @issue.category.name : "-"), :class => 'category'

File test/functional/issues_controller_test.rb

     assert_no_tag 'a', :content => /Next/
   end
 
+  def test_show_should_display_link_to_the_assignee
+    get :show, :id => 2
+    assert_response :success
+    assert_select '.assigned-to' do
+      assert_select 'a[href=/users/3]'
+    end
+  end
+
   def test_show_should_display_visible_changesets_from_other_projects
     project = Project.find(2)
     issue = project.issues.first