Railsify the box link on the browse changeset page

This removes the whitespace in the url, and makes the root path a
named route so that it can be nicely referenced.

diff --git a/app/views/browse/_changeset_details.html.erb b/app/views/browse/_changeset_details.html.erb
index 2d7808137bd8de0102c227d84981e2633e16f5f5..eb90fad1044665cd877c0c417aba7d782a419e88 100644 (file)
--- a/app/views/browse/_changeset_details.html.erb
+++ b/app/views/browse/_changeset_details.html.erb
@@ -33,8 +33,7 @@
           </tr>
           <tr>
             <td><%=bbox.min_lon -%></td>
           </tr>
           <tr>
             <td><%=bbox.min_lon -%></td>

-            <td>(<a href='/?minlon=<%= bbox.min_lon %>&minlat=<%= bbox.min_lat %>&maxlon=<%= bbox.max_lon %>&maxlat=<%= bbox.max_lat %>
-                  &box=yes' title='<%= t 'browse.changeset_details.show_area_box' %>'><%= t 'browse.changeset_details.box' %></a>)</td>
+            <td>(<%= link_to t('browse.changeset_details.box'), root_path(:minlon => bbox.min_lon, :minlat => bbox.min_lat, :maxlon => bbox.max_lon, :maxlat => bbox.max_lat, :box => 'yes'), :title => t('browse.changeset_details.show_area_box') %>)</td>

             <td><%=bbox.max_lon -%></td>
           </tr>
           <tr>
             <td><%=bbox.max_lon -%></td>
           </tr>
           <tr>

diff --git a/config/routes.rb b/config/routes.rb
index fa7dd5ba04faca6ec0406a0c1fb6c04d5945c7bc..267de7956add038ad4b1d88e105f229626de0200 100644 (file)
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -93,7 +93,7 @@ OpenStreetMap::Application.routes.draw do
   match '/browse' => 'changeset#list'
 
   # web site
   match '/browse' => 'changeset#list'
 
   # web site

-  match '/' => 'site#index'
+  root :to => 'site#index'

   match '/edit' => 'site#edit'
   match '/copyright' => 'site#copyright'
   match '/copyright/:copyright_locale' => 'site#copyright'
   match '/edit' => 'site#edit'
   match '/copyright' => 'site#copyright'
   match '/copyright/:copyright_locale' => 'site#copyright'