Uniqify counts for containing ways and relations on browse pages

Fixes #3519
This commit is contained in:
Tom Hughes 2022-03-30 22:26:39 +01:00
parent 8364fa6eb6
commit f2656f925e
3 changed files with 4 additions and 4 deletions

View file

@ -14,7 +14,7 @@
<h4><%= t "browse.part_of" %></h4>
<% unless node.ways.empty? %>
<details <%= "open" if node.ways.count < 10 %>>
<summary><%= t "browse.part_of_ways", :count => node.ways.count %></summary>
<summary><%= t "browse.part_of_ways", :count => node.ways.uniq.count %></summary>
<ul class="list-unstyled">
<% node.ways.uniq.each do |way| %>
<li><%= link_to printable_name(way), { :action => "way", :id => way.id.to_s }, { :class => link_class("way", way), :title => link_title(way) } %></li>
@ -24,7 +24,7 @@
<% end %>
<% unless node.containing_relation_members.empty? %>
<details <%= "open" if node.containing_relation_members.count < 10 %>>
<summary><%= t "browse.part_of_relations", :count => node.containing_relation_members.count %></summary>
<summary><%= t "browse.part_of_relations", :count => node.containing_relation_members.uniq.count %></summary>
<ul class="list-unstyled">
<%= render :partial => "containing_relation", :collection => node.containing_relation_members.uniq %>
</ul>

View file

@ -13,7 +13,7 @@
<% unless relation.containing_relation_members.empty? %>
<h4><%= t "browse.part_of" %></h4>
<details <%= "open" if relation.containing_relation_members.count < 10 %>>
<summary><%= t "browse.part_of_relations", :count => relation.containing_relation_members.count %></summary>
<summary><%= t "browse.part_of_relations", :count => relation.containing_relation_members.uniq.count %></summary>
<ul class="list-unstyled">
<%= render :partial => "containing_relation", :collection => relation.containing_relation_members.uniq %>
</ul>

View file

@ -13,7 +13,7 @@
<% unless way.containing_relation_members.empty? %>
<h4><%= t "browse.part_of" %></h4>
<details <%= "open" if way.containing_relation_members.count < 10 %>>
<summary><%= t "browse.part_of_relations", :count => way.containing_relation_members.count %></summary>
<summary><%= t "browse.part_of_relations", :count => way.containing_relation_members.uniq.count %></summary>
<ul class="list-unstyled">
<%= render :partial => "containing_relation", :collection => way.containing_relation_members.uniq %>
</ul>