Skip to content

Added support of Google Place Autocomplete: - #1119

Closed
gdw96 wants to merge 3 commits into
geocoder-php:masterfrom
gdw96:master
Closed

gdw96 wants to merge 3 commits into
geocoder-php:masterfrom
gdw96:master

Conversation

@gdw96

@gdw96 gdw96 commented May 6, 2021

Copy link
Copy Markdown
  • Add buildPlaceAutocompleteQuery, parseMatchedSubstrings, parseStructuredFormatting and parseTerms methods to GoogleMapsPlaces.php.
  • Add classes GooglePlaceAutocomplete and StructuredFormatting(result of autocompleted queries).
  • Edit fetchURL method for Google Place Autocomplete (response are different).
  • Edit README.md for document the new GeocodeQuery mode.
  • Add testGeocodePlaceAutocompleteMode method to GooglePlaceTest.php. Test pass with success (and tested in a Symfony application successfully).

Hi,

I needed to use Google Place Autocomplete and noticed it wasn't supported, so I set out to add it.

Thank's

 - Add `buildPlaceAutocompleteQuery`, `parseMatchedSubstrings`, `parseStructuredFormatting` and `parseTerms` methods to `GoogleMapsPlaces.php`.
 - Add classes `GooglePlaceAutocomplete` and `StructuredFormatting`(result of autocompleted queries).
 - Edit `fetchURL` method for Google Place Autocomplete (response are different).
 - Edit `README.md` for document the new GeocodeQuery mode.
 - Add `testGeocodePlaceAutocompleteMode` method to `GooglePlaceTest.php`. Test pass with success (and tested in a Symfony application successfully).
@jbelien

jbelien commented May 7, 2021

Copy link
Copy Markdown
Member

Thanks a lot for your contribution @gdw96 !

First things first, could you make your PR pass the StyleCI test ?

I'll review it ASAP.

@gdw96

gdw96 commented May 7, 2021 •

Copy link
Copy Markdown
Author

Hi @jbelien,

Of course, I didn't have time yesterday, I'm correcting it today.

@gdw96

gdw96 commented May 7, 2021

Copy link
Copy Markdown
Author

@jbelien Done

@Nyholm Nyholm closed this Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants