Don't pass paths to element_single_current_link helper

This commit is contained in:
Anton Khorev 2024-03-16 04:51:42 +03:00
parent 065e9d217f
commit 927cc9f45e
3 changed files with 5 additions and 5 deletions

View file

@ -1,6 +1,6 @@
module BrowseHelper module BrowseHelper
def element_single_current_link(type, object, url) def element_single_current_link(type, object)
link_to url, { :class => element_class(type, object), :title => element_title(object), :rel => (link_follow(object) if type == "node") } do link_to object, { :class => element_class(type, object), :title => element_title(object), :rel => (link_follow(object) if type == "node") } do
element_strikethrough object do element_strikethrough object do
printable_element_name object printable_element_name object
end end

View file

@ -17,7 +17,7 @@
<summary><%= t "browse.part_of_ways", :count => node.ways.uniq.count %></summary> <summary><%= t "browse.part_of_ways", :count => node.ways.uniq.count %></summary>
<ul class="list-unstyled"> <ul class="list-unstyled">
<% node.ways.uniq.each do |way| %> <% node.ways.uniq.each do |way| %>
<li><%= element_single_current_link "way", way, way_path(way) %></li> <li><%= element_single_current_link "way", way %></li>
<% end %> <% end %>
</ul> </ul>
</details> </details>

View file

@ -27,12 +27,12 @@
<ul class="list-unstyled"> <ul class="list-unstyled">
<% way.way_nodes.each do |wn| %> <% way.way_nodes.each do |wn| %>
<li> <li>
<%= element_single_current_link "node", wn.node, node_path(wn.node) %> <%= element_single_current_link "node", wn.node %>
<% related_ways = wn.node.ways.reject { |w| w.id == wn.way_id } %> <% related_ways = wn.node.ways.reject { |w| w.id == wn.way_id } %>
<% if related_ways.size > 0 then %> <% if related_ways.size > 0 then %>
(<%= t ".also_part_of_html", (<%= t ".also_part_of_html",
:count => related_ways.size, :count => related_ways.size,
:related_ways => to_sentence(related_ways.map { |w| element_single_current_link "way", w, way_path(w) }) %>) :related_ways => to_sentence(related_ways.map { |w| element_single_current_link "way", w }) %>)
<% end %> <% end %>
</li> </li>
<% end %> <% end %>