Mapbox Provider prevents searching for countries

Offen

#1,107 erstellt am 15.1.2021 2 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: php
Bereich: api, backend

Rechercherichtung

Überprüfe Mapbox.php im Bereich der Zeilen 283 und 301, beginnend mit fetchUrl und der Verarbeitung von Mapbox-Ergebnissen ohne ein context-Feld. Verfolge die vorhandenen Modelle für context-Daten und stelle sicher, dass Ergebnisse für Länder zurückgegeben werden, ohne dass etwas kaputtgeht, während Städte, Bezirke und Adressen ihr bisheriges Verhalten beibehalten.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

I'm using the Mapbox Provider in my API to fetch coordinates for given entities. These entities can be either:

Countries
Cities
Districts
Addresses

However, the current implementation of the mapbox provider prevents from returning anything that doesn't contain a context.

When I run this snippet and watch the console, I see that the query to Mapbox is correct and data is returned.
$locations = $mapboxGeocoder->geocodeQuery(
GeocodeQuery::create($country->getName())
->withLocale('nl')
->withData('location_type', 'country')
);

However, the fetchUrl method in Mapbox.php simply breaks when a result without context is returned by Mapbox.
https://github.com/geocoder-php/mapbox-provider/blob/master/Mapbox.php#L283

When I remove that piece of code, and add an additional check here to check if context is available before iterating through it, the code returns the requested items.
https://github.com/geocoder-php/mapbox-provider/blob/master/Mapbox.php#L301

This isn't a clean solution and additional models are required to make it a better implementation. However, the repo is read-only so I cannot help with the required changes

Vorherrschende Sprache: PHP
Sterne: 4k
Forks: 525
Ø Merge: 8 Min.
Gemergte PRs (30 T.): 1

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus geocoder-php/Geocoder

Wrong LocationIQ Bounds Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 78/100

geocoder-php/Geocoder#1272 · Erstellt am 23.8.2026
OSM/Nomatim address search questions Offen

question

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100

geocoder-php/Geocoder#1267 · 2 Kommentare · Erstellt am 13.3.2026
GeocodeQuery WithBounds not strictly used across providers that support it Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

geocoder-php/Geocoder#1266 · Erstellt am 27.2.2026
PHP 8.4 compatibility (implicit nullable deprecation) [Geocoder\\Collection::slice(): Implicitly marking parameter $length as nullable is deprecated] Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 38/100

geocoder-php/Geocoder#1244 · 2 Kommentare · 2 Reaktionen · Erstellt am 06.2.2025
Add Support for IPQuery Offen

provider

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 30/100

geocoder-php/Geocoder#1243 · Erstellt am 31.1.2025

Alle Issues in geocoder-php/Geocoder