Vote #68117
完了Issue mail should render the HTML version of the issue details
0%
説明
The view issue_edit.text.html.rhtml should use the HTML version of show_detail(). The current implementation looks like:
-
<% for detail in @journal.details %>
- <%= show_detail(detail, true) %> <% end %>
But since this is for an HTML display, the following should be used:
-
<% for detail in @journal.details %>
- <%= show_detail(detail) %> <% end %>
Unfortunately, when that is changed, an error is thrown when updating an issue if an email is sent out. For some reason, the following line from issues_helper.rb (line 147 in 1.0.1):
old_value = content_tag("strike", old_value) if detail.old_value and (!detail.value or detail.value.empty?)
Causes an exception. The exception is raised because detail.value is a Fixnum and Fixnum doesn't have a .empty? method.
journals
This has been done with r8721 for version:1.4.0; the above mentioned issue had already been fixed by r6323.
--------------------------------------------------------------------------------
related_issues
relates,Closed,10251,Description diff link in note details is relative when received by email