Forgot that /history can also be called without a bbox (if one can't be found)
authorÆvar Arnfjörð Bjarmason <avarab@gmail.com>
Sun, 7 Jun 2009 02:53:24 +0000 (02:53 +0000)
committerÆvar Arnfjörð Bjarmason <avarab@gmail.com>
Sun, 7 Jun 2009 02:53:24 +0000 (02:53 +0000)
I think the best thing to do here is to simply remove the RSS link, if
someone wants a RSS feed without a bbox they can click the Recent
Changes link and get a feed there.

app/views/changeset/list_bbox.rhtml

index 354f77727e80a88d415236d04634f9d3bb6119e7..ef774e37819c223167d41f0d5d7f321be66b1a94 100644 (file)
@@ -39,8 +39,13 @@ end
 %>
 <br>
 
-<%= rss_link_to :action => 'list_bbox_rss', :bbox => @bbox.join(",") %>
+<% if @bbox %>
+  <%= rss_link_to :action => 'list_bbox_rss', :bbox => @bbox.join(",") %>
+<% end %>
+
 
-<% content_for :head do %>
-<%= auto_discovery_link_tag :atom, :controller => 'changeset', :action => 'list_bbox_rss', :bbox => @bbox.join(",") %>
+<% if @bbox %>
+  <% content_for :head do %>
+    <%= auto_discovery_link_tag :atom, :controller => 'changeset', :action => 'list_bbox_rss', :bbox => @bbox.join(",") %>
+  <% end %>
 <% end %>