From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Tue, 23 Jun 2009 00:02:35 +0000 (+0000)
Subject: 'trace.tags.none' was never displayed because the empty list ([]) is true in ruby... 
X-Git-Tag: live~6958
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/9e504a0b33032635c3dc5801b22ca3e7566d6b0b

'trace.tags.none' was never displayed because the empty list ([]) is true in ruby. Use trace.tags.empty?

Wrap 'trace.tags.none' in <i> to make it obvious that "none" is a system-generated value
---

diff --git a/app/views/trace/view.html.erb b/app/views/trace/view.html.erb
index 80733844d..048d92d35 100644
--- a/app/views/trace/view.html.erb
+++ b/app/views/trace/view.html.erb
@@ -35,12 +35,12 @@
   <tr>
     <td><%= t'trace.view.tags' %></td>
     <td>
-    <% if @trace.tags %>
+    <% unless @trace.tags.empty? %>
       <% @trace.tags.each do |tag| %>
         <%= link_to tag.tag, { :controller => 'trace', :action => 'list', :tag => tag.tag, :id => nil } %>
       <% end %>
     <% else %>
-      <%= t'trace.view.none' %>
+      <i><%= t'trace.view.none' %></i>
     <% end %>
     </td>
   </tr>