Skip to content
This repository
Browse code

1.9 Fix for exception views.

  • Loading branch information...
commit c253d7b2db53dc1049615f470b5ede9b5385fa38 1 parent 2f1ded3
Michael Koziarski NZKoz authored
4 actionpack/lib/action_controller/templates/rescues/_trace.erb
@@ -15,12 +15,12 @@
15 15 show = "document.getElementById('#{name.gsub /\s/, '-'}').style.display='block';"
16 16 hide = (names - [name]).collect {|hide_name| "document.getElementById('#{hide_name.gsub /\s/, '-'}').style.display='none';"}
17 17 %>
18   - <a href="#" onclick="<%= hide %><%= show %>; return false;"><%= name %></a> <%= '|' unless names.last == name %>
  18 + <a href="#" onclick="<%= hide.join %><%= show %>; return false;"><%= name %></a> <%= '|' unless names.last == name %>
19 19 <% end %>
20 20
21 21 <% traces.each do |name, trace| %>
22 22 <div id="<%= name.gsub /\s/, '-' %>" style="display: <%= name == "Application Trace" ? 'block' : 'none' %>;">
23   - <pre><code><%= trace.join "\n" %></code></pre>
  23 + <pre><code><%=h trace.join "\n" %></code></pre>
24 24 </div>
25 25 <% end %>
26 26 </div>

0 comments on commit c253d7b

Maxim Chernyak aka hakunin

Did anyone else have their backtraces link to their editor (e.g. txmt:// links), and had them escaped due to this 'h'? Reply here if you had such problem.

Please sign in to comment.
Something went wrong with that request. Please try again.