From 656170c450e36087a11a438dcb66735ba73b7543 Mon Sep 17 00:00:00 2001
From: "Michael J. Rubinsky" <mrubinsk@horde.org>
Date: Sun, 29 Nov 2009 18:29:49 -0500
Subject: [PATCH] Implement forward/reverse geocoding, along with some changes
 to standardize return values from various methods from each hordemap driver.

When marker is moved, new textual location is filled in the Location field
(if able), if new location is typed into Location field, and "Find In Maps" is
clicked, the marker will move to that location if found.
---
 kronolith/js/kronolith.js          | 31 ++++++++++++++++++++++++-------
 kronolith/templates/index/edit.inc |  2 +-
 2 files changed, 25 insertions(+), 8 deletions(-)

diff --git a/kronolith/js/kronolith.js b/kronolith/js/kronolith.js
index 774e26eb7..c62d6c0a1 100644
--- a/kronolith/js/kronolith.js
+++ b/kronolith/js/kronolith.js
@@ -3480,7 +3480,8 @@ KronolithEventMap =
                 {
                     'elt': 'kronolithEventMap',
                     'delayed': true,
-                    'layers': layers
+                    'layers': layers,
+                    'markerDragEnd': this.onMarkerDragEnd.bind(this)
                 });
         this._map.display();
         this._marker = this._map.addMarker(
@@ -3491,8 +3492,8 @@ KronolithEventMap =
                     'dragend': this.onMarkerDragEnd
                 });
 
-                this._map.setCenter({lat:38.7115479, lon: -9.13774}, 10);
-                this._initialized = true;
+        this._map.setCenter({lat:38.7115479, lon: -9.13774}, 10);
+        this._initialized = true;
    },
 
    isInitialized: function()
@@ -3500,13 +3501,29 @@ KronolithEventMap =
        return this._initialized;
    },
 
+   /**
+    * Callback for handling marker drag end.
+    *
+    * @param object r  An object that implenents a getLonLat() method to obtain
+    *                  the new location of the marker.
+    */
    onMarkerDragEnd: function(r)
    {
+       var ll = r.getLonLat();
        var gc = new HordeMap.Geocoder[Kronolith.conf.maps.geocoder]();
-       gc.reverseGeocode(r.getLonLat(), this.onReverseGeocode.bind(this), this.onError.bind(this) );
+       gc.reverseGeocode(ll, this.onReverseGeocode.bind(this), this.onError.bind(this) );
    },
 
-   onReverseGeocode: function(r) { $('kronolithEventLocation').value = r; },
+   /**
+    * Callback for handling a reverse geocode request.
+    *
+    * @param array r  An array of objects containing the results. Each object in
+    *                 the array is {lat:, lon:, address}
+    */
+   onReverseGeocode: function(r) {
+       $('kronolithEventLocation').value = r[0].address;
+       // Do something else with the lonlat?
+   },
 
    onError: function(r) {  },
 
@@ -3516,8 +3533,8 @@ KronolithEventMap =
        var gc = new HordeMap.Geocoder[Kronolith.conf.maps.geocoder]();
        gc.geocode(a, function(r) {
            r = r.shift();
-           ll = new OpenLayers.LonLat(r.Longitude, r.Latitude);
-           //this._marker.setLonLat(ll);
+           ll = new OpenLayers.LonLat(r.lon, r.lat);
+           this._map.moveMarker(this._marker, { lat: r.lat, lon: r.lon });
            this._map.setCenter(ll);
        }.bind(this),
        this.onError);
diff --git a/kronolith/templates/index/edit.inc b/kronolith/templates/index/edit.inc
index a431e6a71..a6bf4f45b 100644
--- a/kronolith/templates/index/edit.inc
+++ b/kronolith/templates/index/edit.inc
@@ -18,7 +18,7 @@
   <td>
     <label for="kronolithEventLocation"><?php echo _("Location") ?>:</label><br />
     <input type="text" name="location" id="kronolithEventLocation" size="40" />
-    &middot; <span class="kronolithEventGeo"><a href="#"><?php echo _("Find in Maps") ?></a></span>
+    &middot; <span><a class="kronolithEventGeo" href="#"><?php echo _("Find in Maps") ?></a></span>
   </td>
 </tr></tbody></table>
 </div>
-- 
2.11.0