Eliminate interpolation in index JS
authorJohn Firebaugh <john.firebaugh@gmail.com>
Sat, 29 Sep 2012 15:17:36 +0000 (08:17 -0700)
committerTom Hughes <tom@compton.nu>
Sat, 29 Sep 2012 17:00:04 +0000 (18:00 +0100)
app/views/layouts/_head.html.erb
app/views/layouts/site.html.erb
app/views/site/index.html.erb
config/i18n-js.yml

index 91a862f0ab730a32fdff276e72d6e5657029aa9b..c85594189a96257d4a5f4e48aa84073c9fd58270 100644 (file)
@@ -29,6 +29,8 @@
     <% if session[:location] %>
     OSM.location = <%= session[:location].to_json.html_safe %>;
     <% end %>
+
+    OSM.preferred_editor = <%= preferred_editor.to_json.html_safe %>;
   </script>
   <title><%= t 'layouts.project_name.title' %><%= ' | '+ @title if @title %></title>
 </head>
index 064fbed0a291f434353bf87e5ab93804456570fe..31f87eb856da5c55253e8664f3af887ff4282c60 100644 (file)
@@ -32,7 +32,7 @@
         <li><%= link_to h(t('layouts.edit')) + content_tag(:span, "▾", :class => "menuicon"), edit_path, {
           :id => 'editanchor',
           :title => t('javascripts.site.edit_tooltip'),
-          :data => { :minzoom => 13 },
+          :data => { :minzoom => 13, :editor => preferred_editor },
           :class => 'geolink llz object disabled'
         } %></li>
         <li><%= link_to t('layouts.history'), browse_changesets_path, {
@@ -44,6 +44,7 @@
         <li><%= link_to t('layouts.export'), export_path, {
           :id => 'exportanchor',
           :title => t('layouts.export_tooltip'),
+          :data => { :url => url_for(:controller => :export, :action => :start) },
           :class => 'geolink llz layers'
         } %></li>
       </ul>
@@ -54,7 +55,7 @@
           <li><%= link_to t('layouts.edit_with',
             :editor => t("editor.#{editor}.description")),
             edit_path(:editor => editor), {
-            :id => editor + 'anchor',
+            :data => { :editor => editor },
             :class => "geolink llz object"
           } %></li>
         <% end %>
index 657db8474d17a42939121862769b8c25e5ddcf3d..98a3a1ce72d0c198d85328665d9414b9921f6abf 100644 (file)
     $("#linkloader").attr("src", "http://127.0.0.1:8111/load_and_zoom?left=" + extent.left + "&top=" + extent.top + "&right=" + extent.right + "&bottom=" + extent.bottom);
 
     setTimeout(function () {
-      if (!loaded) alert("<%=j t('site.index.remote_failed') %>");
+      if (!loaded) alert(I18n.t('site.index.remote_failed'));
     }, 1000);
 
     return false;
   }
 
   function installEditHandler() {
-    $("#remoteanchor").click(remoteEditHandler);
+    $("a[data-editor=remote]").click(remoteEditHandler);
 
-    <% if preferred_editor == "remote" %>
-      $("#editanchor").click(remoteEditHandler);
-
-      <% if params[:action] == "edit" %>
-        remoteEditHandler();
-      <% end %>
-    <% end %>
+    if (OSM.preferred_editor == "remote" && $('body').hasClass("site-edit")) {
+      remoteEditHandler();
+    }
   }
 
   $(document).ready(mapInit);
 
   $(document).ready(function () {
     $("#exportanchor").click(function (e) {
-      $.ajax({ url: "<%= url_for :controller => :export, :action => :start %>", success: function (sidebarHtml) {
+      $.ajax({ url: $(this).data('url'), success: function (sidebarHtml) {
         startExport(sidebarHtml);
       }});
       e.preventDefault();
     });
 
-    <% if params[:export] -%>
-    $("#exportanchor").click();
-    <% end -%>
+    if (window.location.pathname == "/export") {
+      $("#exportanchor").click();
+    }
 
-    <% if params[:query] -%>
-    doSearch("<%= params[:query] %>");
-    <% end %>
+    var query;
+    if (query = getArgs(window.location.toString()).query) {
+      doSearch(query);
+    }
   });
 // -->
 </script>
index 261fb6c0f93649a94f2fb4aac1a78ed79f015532..f80b142c5e9cfc7e3fb9f7b25cc80cd7deeede12 100644 (file)
@@ -26,3 +26,4 @@ translations:
     - "*.browse.start_rjs.*"
     - "*.export.start_rjs.*"
     - "*.javascripts.*"
+    - "*.site.index.remote_failed"