Rename updateMarker() to addCreatedNoteMarker()

[rails.git] / app / assets / javascripts / index / new_note.js

diff --git a/app/assets/javascripts/index/new_note.js b/app/assets/javascripts/index/new_note.js
index 887ba043b12b00bda5478b4e2dfd3aeec305efd3..bd093e11adcb6fca2a45bd7fb72c6843dea0164e 100644 (file)
--- a/app/assets/javascripts/index/new_note.js
+++ b/app/assets/javascripts/index/new_note.js
@@ -5,7 +5,7 @@ OSM.NewNote = function (map) {
       content = $("#sidebar_content"),
       page = {},
       addNoteButton = $(".control-note .control-button"),
       content = $("#sidebar_content"),
       page = {},
       addNoteButton = $(".control-note .control-button"),

-      newNote,
+      newNoteMarker,

       halo;
 
   var noteIcons = {
       halo;
 
   var noteIcons = {


@@ -35,39 +35,21 @@ OSM.NewNote = function (map) {
     OSM.router.route("/note/new");
   });
 
     OSM.router.route("/note/new");
   });
 

-  function createNote(marker, form, url) {
-    var location = marker.getLatLng().wrap();
-
-    marker.options.draggable = false;
-    marker.dragging.disable();
-
-    $(form).find("input[type=submit]").prop("disabled", true);
-
+  function createNote(location, text, callback) {

     $.ajax({
     $.ajax({

-      url: url,
+      url: "/api/0.6/notes.json",

       type: "POST",
       oauth: true,
       data: {
         lat: location.lat,
         lon: location.lng,
       type: "POST",
       oauth: true,
       data: {
         lat: location.lat,
         lon: location.lng,

-        text: $(form.text).val()
+        text

       },
       },

-      success: function (feature) {
-        noteCreated(feature, marker);
-      }
+      success: callback

     });
     });

-
-    function noteCreated(feature, marker) {
-      content.find("textarea").val("");
-      updateMarker(feature);
-      newNote = null;
-      noteLayer.removeLayer(marker);
-      addNoteButton.removeClass("active");
-      OSM.router.route("/note/" + feature.properties.id);
-    }

   }
 
   }
 

-  function updateMarker(feature) {
+  function addCreatedNoteMarker(feature) {

     var marker = L.marker(feature.geometry.coordinates.reverse(), {
       icon: noteIcons[feature.properties.status],
       opacity: 0.9,
     var marker = L.marker(feature.geometry.coordinates.reverse(), {
       icon: noteIcons[feature.properties.status],
       opacity: 0.9,


@@ -124,20 +106,20 @@ OSM.NewNote = function (map) {
       padding: [50, 50]
     });
 
       padding: [50, 50]
     });
 

-    newNote = L.marker(markerLatlng, {
+    newNoteMarker = L.marker(markerLatlng, {

       icon: noteIcons.new,
       opacity: 0.9,
       draggable: true
     });
 
       icon: noteIcons.new,
       opacity: 0.9,
       draggable: true
     });
 

-    newNote.on("dragstart dragend", function (a) {
-      newHalo(newNote.getLatLng(), a.type);
+    newNoteMarker.on("dragstart dragend", function (a) {
+      newHalo(newNoteMarker.getLatLng(), a.type);

     });
 
     });
 

-    newNote.addTo(noteLayer);
-    newHalo(newNote.getLatLng());
+    newNoteMarker.addTo(noteLayer);
+    newHalo(newNoteMarker.getLatLng());

 
 

-    newNote.on("remove", function () {
+    newNoteMarker.on("remove", function () {

       addNoteButton.removeClass("active");
     }).on("dragend", function () {
       content.find("textarea").focus();
       addNoteButton.removeClass("active");
     }).on("dragend", function () {
       content.find("textarea").focus();


@@ -152,15 +134,29 @@ OSM.NewNote = function (map) {
     }
 
     content.find("input[type=submit]").on("click", function (e) {
     }
 
     content.find("input[type=submit]").on("click", function (e) {

+      const location = newNoteMarker.getLatLng().wrap();
+      const text = content.find("textarea").val();
+

       e.preventDefault();
       e.preventDefault();

-      createNote(newNote, e.target.form, "/api/0.6/notes.json");
+      $(this).prop("disabled", true);
+      newNoteMarker.options.draggable = false;
+      newNoteMarker.dragging.disable();
+
+      createNote(location, text, (feature) => {
+        content.find("textarea").val("");
+        addCreatedNoteMarker(feature);
+        noteLayer.removeLayer(newNoteMarker);
+        newNoteMarker = null;
+        addNoteButton.removeClass("active");
+        OSM.router.route("/note/" + feature.properties.id);
+      });

     });
 
     return map.getState();
   };
 
   page.unload = function () {
     });
 
     return map.getState();
   };
 
   page.unload = function () {

-    if (newNote) noteLayer.removeLayer(newNote);
+    if (newNoteMarker) noteLayer.removeLayer(newNoteMarker);

     if (halo) map.removeLayer(halo);
     addNoteButton.removeClass("active");
   };
     if (halo) map.removeLayer(halo);
     addNoteButton.removeClass("active");
   };