Clean up search JS

Move it to index.js/edit.js and eliminate the global doSearch
function. Not enough code is shared to justify it.

diff --git a/app/assets/javascripts/edit.js b/app/assets/javascripts/edit.js
index f5ada1508ccfc4c7879d76f337da4fc8eb844e30..0b2b329b5d358223c4c1487737a9ad8813a2d768 100644 (file)
--- a/app/assets/javascripts/edit.js
+++ b/app/assets/javascripts/edit.js
@@ -28,5 +28,16 @@ function minimiseMap() {
   handleResize();
 }
 
   handleResize();
 }
 

-$(document).ready(handleResize);
-$(window).resize(handleResize);
+$(document).ready(function () {
+  $(window).resize(handleResize);
+  handleResize();
+
+  $("#search_form").submit(function () {
+    $("#sidebar_title").html(I18n.t('site.sidebar.search_results'));
+    $("#sidebar_content").load($(this).attr("action"), {
+      query: $("#query").val()
+    }, openSidebar);
+
+    return false;
+  });
+});

diff --git a/app/assets/javascripts/index.js b/app/assets/javascripts/index.js
index 5f057b23439e2ee576d0d443a88526e05b2ddc7c..0ceb015a74380a8199fbf4c6c24befab8139dc6c 100644 (file)
--- a/app/assets/javascripts/index.js
+++ b/app/assets/javascripts/index.js
@@ -126,8 +126,22 @@ $(document).ready(function () {
     $("#exportanchor").click();
   }
 
     $("#exportanchor").click();
   }
 

-  var query;
-  if (query = getArgs(window.location.toString()).query) {
-    doSearch(query);
+  $("#search_form").submit(function () {
+    var extent = unproj(map.getExtent());
+
+    $("#sidebar_title").html(I18n.t('site.sidebar.search_results'));
+    $("#sidebar_content").load($(this).attr("action"), {
+      query: $("#query").val(),
+      minlon: extent.left,
+      minlat: extent.bottom,
+      maxlon: extent.right,
+      maxlat: extent.top
+    }, openSidebar);
+
+    return false;
+  });
+
+  if ($("#query").val()) {
+    $("#search_form").submit();

   }
 });
   }
 });

diff --git a/app/views/site/_search.html.erb b/app/views/site/_search.html.erb
index 9adc3dafe09c588f3a435c521e9c4d79e2dabeb2..bcab7e4047b7eed6073886aaf0644585478cb435 100644 (file)
--- a/app/views/site/_search.html.erb
+++ b/app/views/site/_search.html.erb
@@ -10,34 +10,6 @@
       zoom: args["zoom"]
     }, openSidebar);
   }
       zoom: args["zoom"]
     }, openSidebar);
   }

-
-  function doSearch(query) {
-    $("#sidebar_title").html("<%= t 'site.sidebar.search_results' %>");
-
-    <% if params[:action] == 'index' -%>
-    var extent = unproj(map.getExtent());
-
-    $("#sidebar_content").load("<%= url_for :controller => :geocoder, :action => :search %>", {
-      query: $("#query").val(),
-      minlon: extent.left,
-      minlat: extent.bottom,
-      maxlon: extent.right,
-      maxlat: extent.top
-    }, openSidebar);
-    <% else -%>
-    $("#sidebar_content").load("<%= url_for :controller => :geocoder, :action => :search %>", {
-      query: $("#query").val()
-    }, openSidebar);
-    <% end -%>
-  }
-
-  $(document).ready(function () {
-    $("#search_form").submit(function (event) {
-      doSearch($("#query").val());
-
-      return false;
-    });
-  });

 // -->
 </script>
 
 // -->
 </script>
 


@@ -45,7 +17,7 @@
   <div class="optionalbox">
     <div class="search_container">
     <div id="search_field">
   <div class="optionalbox">
     <div class="search_container">
     <div id="search_field">

-    <%= form_tag "#", :id => "search_form" do %>
+    <%= form_tag url_for(:controller => :geocoder, :action => :search), :id => "search_form" do %>

       <%= submit_tag t('site.search.submit_text') %>
       <%= text_field_tag :query, h(params[:query]), :tabindex => "1", :placeholder => t('site.search.search') %>
     <% end %>
       <%= submit_tag t('site.search.submit_text') %>
       <%= text_field_tag :query, h(params[:query]), :tabindex => "1", :placeholder => t('site.search.search') %>
     <% end %>

diff --git a/config/i18n-js.yml b/config/i18n-js.yml
index f80b142c5e9cfc7e3fb9f7b25cc80cd7deeede12..29855fe48ce47baf0d5dd1ea9e69144ce04b1062 100644 (file)
--- a/config/i18n-js.yml
+++ b/config/i18n-js.yml
@@ -27,3 +27,4 @@ translations:
     - "*.export.start_rjs.*"
     - "*.javascripts.*"
     - "*.site.index.remote_failed"
     - "*.export.start_rjs.*"
     - "*.javascripts.*"
     - "*.site.index.remote_failed"

+    - "*.site.sidebar.search_results"