From: Tom Hughes <tom@compton.nu>
Date: Thu, 5 May 2011 18:42:18 +0000 (+0100)
Subject: Don't show the "Nodes" heading for a way with no nodes
X-Git-Tag: live~6859^2~19
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/79ad65895bf92943c592e187987b5a60e81c0d53

Don't show the "Nodes" heading for a way with no nodes
---

diff --git a/app/views/browse/_way_details.html.erb b/app/views/browse/_way_details.html.erb
index 2b284cf3c..ce5c86dda 100644
--- a/app/views/browse/_way_details.html.erb
+++ b/app/views/browse/_way_details.html.erb
@@ -2,22 +2,24 @@
 
   <%= render :partial => "common_details", :object => way_details %>
 
-  <tr valign="top">
-    <th><%= t'browse.way_details.nodes' %></th>
-    <td>
-      <table cellpadding="0">
-        <% way_details.way_nodes.each do |wn| %>
-          <tr><td>
-            <%= link_to h(printable_name(wn.node)), { :action => "node", :id => wn.node_id.to_s }, :class => link_class('node', wn.node), :title => link_title(wn.node) %>
-            <% related_ways = wn.node.ways.reject { |w| w.id == way_details.id } %>
-	    <% if related_ways.size > 0 then %>
-              (<%= t 'browse.way_details.also_part_of', :count => related_ways.size, :related_ways => related_ways.map { |w| link_to(h(printable_name(w)), { :action => "way", :id => w.id.to_s }, :class => link_class('way', w), :title => link_title(w) ) }.to_sentence %>)
-            <% end %>
-	  </td></tr>
-        <% end %>
-      </table>
-    </td>
-  </tr>
+  <% unless way_details.way_nodes.empty? %>
+    <tr valign="top">
+      <th><%= t'browse.way_details.nodes' %></th>
+      <td>
+        <table cellpadding="0">
+          <% way_details.way_nodes.each do |wn| %>
+            <tr><td>
+              <%= link_to h(printable_name(wn.node)), { :action => "node", :id => wn.node_id.to_s }, :class => link_class('node', wn.node), :title => link_title(wn.node) %>
+              <% related_ways = wn.node.ways.reject { |w| w.id == way_details.id } %>
+              <% if related_ways.size > 0 then %>
+                (<%= t 'browse.way_details.also_part_of', :count => related_ways.size, :related_ways => related_ways.map { |w| link_to(h(printable_name(w)), { :action => "way", :id => w.id.to_s }, :class => link_class('way', w), :title => link_title(w) ) }.to_sentence %>)
+              <% end %>
+            </td></tr>
+          <% end %>
+        </table>
+      </td>
+    </tr>
+  <% end %>
 
   <% unless way_details.containing_relation_members.empty? %>
     <tr valign="top">