geocoder-php / geocoder-php/Geocoder
[Google Maps] Add output language support
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 4k
- Forks
- 525
- Ø Merge
- 8 Min.
- Gemergte PRs (30 T.)
- 1
Beschreibung
The following was submitted in the GeocoderLaravel repo by @xSoulRootx (https://github.com/geocoder-php/GeocoderLaravel/issues/144), but probably belongs here:
============
General Information
GeocoderLaravel Version: ^4.0
Laravel Version: 5.7.*
PHP Version: 7.2
Operating System and Version: Ubuntu
Issue Description
Output language in reverse query don't work
Resolve
vendor\geocoder-php\google-maps-provider\GoogleMaps.php
in
public function reverseQuery(ReverseQuery $query): Collection
before
return $this->fetchUrl($url, $query->getLocale(), $query->getLimit(), $query->getData('region', $this->region));
add
if (null !== $query->getLocale()) {
$url .= '&result_type='.urlencode($resultType);
} else {
$url .= '&language='.env('GOOGLE_MAPS_LOCALE', 'en-US');
}
or something like this
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in vendor/geocoder-php/google-maps-provider/GoogleMaps.php bei reverseQuery(ReverseQuery $query) und untersuche anschließend, wie locale und URL-Parameter für Google Maps-Anfragen verarbeitet werden. Die Aufgabe ist abgeschlossen, wenn Reverse-Abfragen die angeforderte Ausgabesprache berücksichtigen, einschließlich eines definierten Verhaltens, wenn kein locale angegeben ist, und das daraus resultierende Anfrageverhalten verifiziert wurde.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- api
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 40/100