Redmine

The view issue_edit.text.html.rhtml should use the HTML version of show_detail(). The current implementation looks like:

<ul>
<% for detail in @journal.details %>
    <li><%= show_detail(detail, true) %></li>
<% end %>
</ul>

But since this is for an HTML display, the following should be used:

<ul>
<% for detail in @journal.details %>
    <li><%= show_detail(detail) %></li>
<% end %>
</ul>

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.