From 17b23b075642d6bf2ba66a0448a7a7acfd2824b6 Mon Sep 17 00:00:00 2001
From: John Firebaugh <john.firebaugh@gmail.com>
Date: Sat, 25 Aug 2012 19:27:10 -0700
Subject: [PATCH] Extract browse.js static asset

---
 .../start.js.erb => assets/javascripts/browse.js}      |  2 --
 app/assets/javascripts/site.js                         |  2 ++
 app/controllers/browse_controller.rb                   |  1 +
 app/views/browse/_map.html.erb                         |  1 -
 app/views/changeset/_map.html.erb                      |  1 -
 app/views/diary_entry/edit.html.erb                    |  1 -
 app/views/site/index.html.erb                          | 10 ++++++----
 app/views/user/_map.html.erb                           |  1 -
 8 files changed, 9 insertions(+), 10 deletions(-)
 rename app/{views/browse/start.js.erb => assets/javascripts/browse.js} (99%)

diff --git a/app/views/browse/start.js.erb b/app/assets/javascripts/browse.js
similarity index 99%
rename from app/views/browse/start.js.erb
rename to app/assets/javascripts/browse.js
index 6478794da..fef09aada 100644
--- a/app/views/browse/start.js.erb
+++ b/app/assets/javascripts/browse.js
@@ -542,5 +542,3 @@ function clearStatus() {
   $("#browse_status").html("");
   $("#browse_status").hide();
 }
-
-startBrowse("<%=j render :partial => "sidebar" %>");
diff --git a/app/assets/javascripts/site.js b/app/assets/javascripts/site.js
index 978e84e3d..df56b95bc 100644
--- a/app/assets/javascripts/site.js
+++ b/app/assets/javascripts/site.js
@@ -2,8 +2,10 @@
 //= require jquery_ujs
 //= require jquery.autogrowtextarea
 //= require jquery.timers
+//= require openlayers
 //= require i18n/translations
 //= require globals
+//= require browse
 //= require export
 
 /*
diff --git a/app/controllers/browse_controller.rb b/app/controllers/browse_controller.rb
index 2a7e2c5f7..e3f20c803 100644
--- a/app/controllers/browse_controller.rb
+++ b/app/controllers/browse_controller.rb
@@ -7,6 +7,7 @@ class BrowseController < ApplicationController
   around_filter :web_timeout, :except => [:start]
 
   def start 
+    render :partial => "sidebar"
   end
   
   def relation
diff --git a/app/views/browse/_map.html.erb b/app/views/browse/_map.html.erb
index b8077e682..340e26132 100644
--- a/app/views/browse/_map.html.erb
+++ b/app/views/browse/_map.html.erb
@@ -1,4 +1,3 @@
-<%= javascript_include_tag 'openlayers.js' %>
 <%= javascript_include_tag 'map.js' %>
 
 <iframe id="linkloader" style="display: none">
diff --git a/app/views/changeset/_map.html.erb b/app/views/changeset/_map.html.erb
index 3357057b1..e055047a9 100644
--- a/app/views/changeset/_map.html.erb
+++ b/app/views/changeset/_map.html.erb
@@ -1,4 +1,3 @@
-<%= javascript_include_tag 'openlayers.js' %>
 <%= javascript_include_tag 'map.js' %>
 
 <div id="changeset_list_map">
diff --git a/app/views/diary_entry/edit.html.erb b/app/views/diary_entry/edit.html.erb
index 54ed0cc70..fc34842ad 100644
--- a/app/views/diary_entry/edit.html.erb
+++ b/app/views/diary_entry/edit.html.erb
@@ -49,7 +49,6 @@
   <% zoom = '12' %>
 <% end %>
 
-<%= javascript_include_tag 'openlayers.js' %>
 <%= javascript_include_tag 'map.js' %>
 
 <script type="text/javascript">
diff --git a/app/views/site/index.html.erb b/app/views/site/index.html.erb
index 5eb8ae228..9b4405feb 100644
--- a/app/views/site/index.html.erb
+++ b/app/views/site/index.html.erb
@@ -6,7 +6,7 @@
 
 <% unless STATUS == :api_offline or STATUS == :database_offline -%>
   <% content_for :editmenu do -%>
-    <li><%= link_to t("browse.start_rjs.data_layer_name"), "#", :id => "show_data" %></li>
+    <li><%= link_to t("browse.start_rjs.data_layer_name"), { :controller => :browse, :action => :start }, :id => "show_data" %></li>
   <% end -%>
 <% end -%>
 
@@ -125,7 +125,6 @@ else
 end
 %>
 
-<%= javascript_include_tag 'openlayers.js' %>
 <%= javascript_include_tag 'map.js' %>
 
 <%= render :partial => 'resize' %>
@@ -192,8 +191,11 @@ end
   }
 
   $(document).ready(function () {
-    $("#show_data").click(function () {
-      $.ajax({ url: "<%= url_for :controller => :browse, :action => :start %>" });
+    $("#show_data").click(function (e) {
+      $.ajax({ url: $(this).attr('href'), success: function (sidebarHtml) {
+        startBrowse(sidebarHtml);
+      }});
+      e.preventDefault();
     });
 
     $("body").on("click", "a.set_position", function () {
diff --git a/app/views/user/_map.html.erb b/app/views/user/_map.html.erb
index a142e99b9..94d1411a0 100644
--- a/app/views/user/_map.html.erb
+++ b/app/views/user/_map.html.erb
@@ -13,7 +13,6 @@ else
 end
 %>
 
-<%= javascript_include_tag 'openlayers.js' %>
 <%= javascript_include_tag 'map.js' %>
 
 <script type="text/javascript">
-- 
2.43.2