cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use ActionView's safe_join to avoid using raw when joining arrays of text

Browse source
This commit is contained in:
Andy Allan 2020-01-02 18:07:23 +01:00
parent 5aa255e13f
commit 53f2c36f1d
2 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
app/views/traces/_trace.html.erb
View file

@ -24,7 +24,7 @@
<%= t ".by" %> <%= link_to h(trace.user.display_name), user_path(trace.user) %>
<% if !trace.tags.empty? %>
<%= t ".in" %>
<%= raw(trace.tags.collect { |tag| link_to_tag tag.tag }.join(", ")) %>
<%= safe_join(trace.tags.collect { |tag| link_to_tag tag.tag }, ", ") %>
<% end %>
</td>
</tr>

2
app/views/traces/show.html.erb
View file

@ -40,7 +40,7 @@
<td><%= t ".tags" %></td>
<td>
<% unless @trace.tags.empty? %>
<%= raw(@trace.tags.collect { |tag| link_to tag.tag, :controller => "traces", :action => "index", :tag => tag.tag, :id => nil }.join(", ")) %>
<%= safe_join(@trace.tags.collect { |tag| link_to tag.tag, :controller => "traces", :action => "index", :tag => tag.tag, :id => nil }, ", ") %>
<% else %>
<i><%= t ".none" %></i>
<% end %>