Fix welcome page redirect, closed #70

[rails.git] / app / assets / javascripts / welcome.js

diff --git a/app/assets/javascripts/welcome.js b/app/assets/javascripts/welcome.js
index 5409c9ec9487d893177954a51c3a8cabc478d301..88b153af2e44e784f7faa65c0285381a882bdb83 100644 (file)
--- a/app/assets/javascripts/welcome.js
+++ b/app/assets/javascripts/welcome.js
@@ -2,53 +2,44 @@ $(document).ready(function() {
   var params = OSM.params();
 
   if (params.lat && params.lon) {
-    $('.edit-located').show();
-
-    $.ajax({
-      url: "http://nominatim.openstreetmap.org/reverse",
-      data: {
-        lat: params.lat,
-        lon: params.lon,
-        zoom: 10
-      },
-      success: function(xml) {
-        var result = $(xml).find('result');
-        if (result.length) {
-          $('.edit-located').hide();
-          $('.edit-geocoded').show();
-          $('.edit-geocoded-location').text(result.text());
-        }
-      }
-    });
+    params.lat = parseFloat(params.lat);
+    params.lon = parseFloat(params.lon);
+    params.zoom = params.zoom || 17;
 
-    $('.start-mapping').on('click', function(e) {
-      window.location = '/edit?zoom=17&lat=' + params.lat + '&lon=' + params.lon;
-    });
+    var url = '/edit';
 
-  } else if (navigator.geolocation) {
-    $('.edit-geolocated').show();
-
-    function geoSuccess(position) {
-      window.location = '/edit?zoom=17&lat=' + position.coords.latitude + '&lon=' + position.coords.longitude;
+    if (params.editor) {
+      url += '?editor=' + params.editor;
     }
 
-    function geoError() {
-      $('.start-mapping')
-        .removeClass('loading')
-        .addClass('error');
+    url += OSM.formatHash(params);
+
+    $('.start-mapping').attr('href', url);
+
+  } else {
+    function geoSuccess(position) {
+      window.location = '/edit' + OSM.formatHash({
+        zoom: 17,
+        lat: position.coords.latitude,
+        lon: position.coords.longitude
+      });
     }
 
     $('.start-mapping').on('click', function(e) {
       e.preventDefault();
-
-      $('.start-mapping')
-        .addClass('loading');
+      $('.start-mapping').addClass('loading');
 
       // handle firefox's weird implementation
       // https://bugzilla.mozilla.org/show_bug.cgi?id=675533
-      window.setTimeout(geoError, 4000);
+      window.setTimeout(manualEdit, 4000);
 
-      navigator.geolocation.getCurrentPosition(geoSuccess, geoError);
+      if (navigator.geolocation) {
+        navigator.geolocation.getCurrentPosition(geoSuccess, manualEdit);
+      }
     });
   }
+
+  function manualEdit() {
+    window.location = '/?edit_help=1';
+  }
 });