Set OpenLayers lang unobtrusively

We output the locale code as the lang attribute, so
use that rather than interpolating into embedded JS.

diff --git a/app/assets/javascripts/openlayers.js.erb b/app/assets/javascripts/openlayers.js.erb
index ebb9578a3f3b1bc709c9e8b59e2fbbc326d56f94..66d991f82039160658e322390a99b971d80362f8 100644 (file)
--- a/app/assets/javascripts/openlayers.js.erb
+++ b/app/assets/javascripts/openlayers.js.erb
@@ -35,3 +35,5 @@ OpenLayers.Util.origGetImageLocation = OpenLayers.Util.getImageLocation;
 OpenLayers.Util.getImageLocation = function(image) {
     return OpenLayers.Util.imageURLs[image] || OpenLayers.Util.origGetImageLocation(image);
 };
+
+OpenLayers.Lang.setCode($('html').attr('lang'));

diff --git a/app/views/browse/_map.html.erb b/app/views/browse/_map.html.erb
index 2c29a0d008999a2fc7491ef915a328b1602e9551..b8077e6824fcab470bc6016b1a1a66e4bd71fada 100644 (file)
--- a/app/views/browse/_map.html.erb
+++ b/app/views/browse/_map.html.erb
@@ -41,8 +41,6 @@
 
 <% if map.instance_of? Changeset or (map.instance_of? Node and map.version > 1) or map.visible %>
   <script type="text/javascript">
-    OpenLayers.Lang.setCode("<%= I18n.locale.to_s %>");
-
     function remoteEditHandler(event, bbox, select) {
       var left = bbox.left - 0.0001;
       var top = bbox.top + 0.0001;

diff --git a/app/views/changeset/_map.html.erb b/app/views/changeset/_map.html.erb
index 8016378c8a7178e1ab18a668477226e6aba91b3d..3357057b17e0b91c033b05ef65f5b0781de1de7f 100644 (file)
--- a/app/views/changeset/_map.html.erb
+++ b/app/views/changeset/_map.html.erb
@@ -5,8 +5,6 @@
 </div>
 
 <script type="text/javascript">
-  OpenLayers.Lang.setCode("<%= I18n.locale.to_s %>");
-
   var highlight;
 
   function highlightChangeset(id) {

diff --git a/app/views/diary_entry/edit.html.erb b/app/views/diary_entry/edit.html.erb
index 6b4d9c177935f9d665df20a7685ea198629dafd2..54ed0cc70ec94ec49d644d6fbc5632980b432973 100644 (file)
--- a/app/views/diary_entry/edit.html.erb
+++ b/app/views/diary_entry/edit.html.erb
@@ -56,9 +56,7 @@
   <!--
   var marker;
 
-  OpenLayers.Lang.setCode("<%= I18n.locale.to_s %>");
-
-  function setLocation( e ) { 
+  function setLocation( e ) {
     closeMapPopup();
 
     var lonlat = getEventPosition(e);

diff --git a/app/views/site/index.html.erb b/app/views/site/index.html.erb
index b7a9df068c618b042d522b389b3674abf2a50e0e..f604267d87c6b5da3fca1affe02631af954bd96f 100644 (file)
--- a/app/views/site/index.html.erb
+++ b/app/views/site/index.html.erb
@@ -134,8 +134,6 @@ end
   var marker;
   var map;
 
-  OpenLayers.Lang.setCode("<%= I18n.locale.to_s %>");
-
   function mapInit(){
     map = createMap("map");
 

diff --git a/app/views/user/_map.html.erb b/app/views/user/_map.html.erb
index 2d886527ccf437c2c2b0e5285af2776b9ef82a1f..a142e99b9b288ddb36192613f3ee034e6c73b52e 100644 (file)
--- a/app/views/user/_map.html.erb
+++ b/app/views/user/_map.html.erb
@@ -20,8 +20,6 @@ end
   <!--
   var marker;
 
-  OpenLayers.Lang.setCode("<%= I18n.locale.to_s %>");
-
   function init(){
     var centre = new OpenLayers.LonLat(<%= lon %>, <%= lat %>);
     <% if params[:scale].to_f > 0 then -%>