Zn2O / gwt-google-apis

Automatically exported from code.google.com/p/gwt-google-apis
0 stars 0 forks source link

Request for reverse geocoding support in gwt maps #207

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Hi,

This feature was added to the javascript api a week ago : 
http://googlegeodevelopers.blogspot.com/2008/10/geocoding-in-reverse.html

Please add it!

--
Mehdi

Original issue reported on code.google.com by mehdi.ra...@gmail.com on 31 Oct 2008 at 10:41

GoogleCodeExporter commented 9 years ago
If you've come across this issue, please 'star' it so we can see how many 
developers
want this feature added.  There are many features in the Maps API that have not 
yet
made it into the bindings.

Original comment by galgwt.reviews@gmail.com on 31 Oct 2008 at 12:54

GoogleCodeExporter commented 9 years ago

Original comment by alemao...@gmail.com on 3 Nov 2008 at 1:48

GoogleCodeExporter commented 9 years ago
2 months have passed and this functionality hasn't been available yet!
At least in my understanding, all that is needed is to overload the getLocations
method in both Geocoder class and GeocoderImpl interface, changing the first
parameter type from String to LatLng. Is it that simple, or am I missing 
something??

I wrote a while ago, when reverse geocoding wasn't even available in google 
maps api,
a reverse geocoder wrapper that uses the reverse geocoder from Nico Goeminne.
http://code.google.com/p/gwt-reverse-geocoder/
http://nicogoeminne.googlepages.com/documentation.html

But it would be nice if I could use it directly from the gwt-maps api.

Original comment by vitor.tc...@gmail.com on 30 Dec 2008 at 2:04

GoogleCodeExporter commented 9 years ago
Here is a patch that adds reverse geocoding functionality to the api.
The changes are the same as I thought in the previous post. I've manually 
tested it,
and it works!

Original comment by vitor.tc...@gmail.com on 30 Dec 2008 at 6:37

Attachments:

GoogleCodeExporter commented 9 years ago
I'd like to add this patch to the build.  I need to ask you to fill out a CLA.  
See:

  http://code.google.com/webtoolkit/makinggwtbetter.html#contributingcode

In the mean time, I'll work on a unit test and demo and upload it to the review 
site.

Original comment by galgwt.reviews@gmail.com on 8 Jan 2009 at 1:22

GoogleCodeExporter commented 9 years ago
Done!

Original comment by vitor.tc...@gmail.com on 8 Jan 2009 at 2:57

GoogleCodeExporter commented 9 years ago
The patch looks good to me.  I added a unit test and demo.  Out for review at:

 http://galgwt-reviews.appspot.com/2601

Original comment by galgwt.reviews@gmail.com on 8 Jan 2009 at 4:12

GoogleCodeExporter commented 9 years ago
Is there a way to test without building from patched source? (ie can I download 
a jar 
somewhere?)

Original comment by mehdi.ra...@gmail.com on 11 Jan 2009 at 8:39

GoogleCodeExporter commented 9 years ago
The patch is still out for review.  I'm waiting for Vitor to review my revised 
patch.

Original comment by galgwt.reviews@gmail.com on 12 Jan 2009 at 12:39

GoogleCodeExporter commented 9 years ago
Committed as r1168 to trunk.

Original comment by galgwt.reviews@gmail.com on 12 Jan 2009 at 10:57

GoogleCodeExporter commented 9 years ago
Released in gwt-maps-1.0.3

Original comment by galgwt.reviews@gmail.com on 21 Feb 2009 at 1:28

GoogleCodeExporter commented 9 years ago
Updating account names

Original comment by zundel@google.com on 21 Dec 2009 at 2:00

GoogleCodeExporter commented 9 years ago
I need to use the patch for reverse Geocoding but don;t know how to add the 
patch to 
gwt-maps api? Can anyone help?

Setareh

Original comment by srafati...@gmail.com on 24 Mar 2010 at 7:53

GoogleCodeExporter commented 9 years ago
You don't need this patch any longer.  It is in the released version of 
gwt-maps-1.0.3

Original comment by zundel@google.com on 24 Mar 2010 at 9:39