From: Tom Hughes <tom@compton.nu>
Date: Mon, 27 Aug 2012 20:18:07 +0000 (+0100)
Subject: Use start.html.erb views instead of _sidebar.html.erb partials
X-Git-Tag: live~5515
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/3d874a5d61b332cefe944b77cfa38a1061425d3f?hp=07c997d1d296fb7e3d74ddcc618fc7bd093c7725

Use start.html.erb views instead of _sidebar.html.erb partials
---

diff --git a/app/controllers/browse_controller.rb b/app/controllers/browse_controller.rb
index e3f20c803..e3833a7d1 100644
--- a/app/controllers/browse_controller.rb
+++ b/app/controllers/browse_controller.rb
@@ -1,5 +1,5 @@
 class BrowseController < ApplicationController
-  layout 'site'
+  layout 'site', :except => [ :start ]
 
   before_filter :authorize_web  
   before_filter :set_locale 
@@ -7,7 +7,6 @@ class BrowseController < ApplicationController
   around_filter :web_timeout, :except => [:start]
 
   def start 
-    render :partial => "sidebar"
   end
   
   def relation
diff --git a/app/controllers/export_controller.rb b/app/controllers/export_controller.rb
index 736efec19..2102a5704 100644
--- a/app/controllers/export_controller.rb
+++ b/app/controllers/export_controller.rb
@@ -4,7 +4,6 @@ class ExportController < ApplicationController
   before_filter :set_locale
 
   def start
-    render :partial => "sidebar"
   end
 
   #When the user clicks 'Export' we redirect to a URL which generates the export download
diff --git a/app/views/browse/_sidebar.html.erb b/app/views/browse/start.html.erb
similarity index 100%
rename from app/views/browse/_sidebar.html.erb
rename to app/views/browse/start.html.erb
diff --git a/app/views/export/_sidebar.html.erb b/app/views/export/start.html.erb
similarity index 100%
rename from app/views/export/_sidebar.html.erb
rename to app/views/export/start.html.erb
diff --git a/test/functional/export_controller_test.rb b/test/functional/export_controller_test.rb
index 8e4282b8f..af9d28183 100644
--- a/test/functional/export_controller_test.rb
+++ b/test/functional/export_controller_test.rb
@@ -18,7 +18,7 @@ class ExportControllerTest < ActionController::TestCase
   def test_start
     xhr :get, :start
     assert_response :success
-    assert_template 'export/_sidebar'
+    assert_template 'export/start'
   end
   
   def test_finish_osm