GeocodeQuery WithBounds not strictly used across providers that support it
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza inspeccionando el manejo existente de viewbox y bounded en el proveedor Nominatim y, a continuación, sigue los proveedores Pelias a través de getGeocodeQueryUrl. Compara cómo GeocodeQuery expone getBounds() con los parámetros de request compatibles con cada proveedor. La tarea estará terminada cuando los proveedores que admiten límites rectangulares utilicen de forma coherente los límites de la query sin requerir datos específicos del proveedor duplicados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The WithBounds logic needs to be added to providers, as the method is a base method in the query.
E.g. in Nominatims provider we see:
$viewbox = $query->getData('viewbox');
if (!is_null($viewbox) && is_array($viewbox) && 4 === count($viewbox)) {
$url .= '&'.http_build_query([
'viewbox' => implode(',', $viewbox),
], '', '&', PHP_QUERY_RFC3986);
$bounded = $query->getData('bounded');
if (!is_null($bounded) && true === $bounded) {
$url .= '&'.http_build_query([
'bounded' => 1,
], '', '&', PHP_QUERY_RFC3986);
}
}
The above should instead or in-addition accept $query->getBounds(), because otherwise it forces us to write code like:
// Bounds of EU/UK where our customers are
$euBounds = new Bounds(34.857764, -30.428009, 75.400203, 41.642303);
$geocodeQuery = GeocodeQuery::create($query)
->withBounds($euBounds)
// For Nominatim
->withData('viewbox', $euBounds->toArray());
Pelias Providers and those that extend it don't support it at all -even though Pelias does:
https://github.com/pelias/documentation/blob/master/search.md#search-within-a-rectangular-region
/**
* @param array<string, mixed> $query_data additional query data (API key for instance)
*
* @throws \Geocoder\Exception\Exception
*/
protected function getGeocodeQueryUrl(GeocodeQuery $query, array $query_data = []): string
{
$address = $query->getText();
// This API doesn't handle IPs
if (filter_var($address, FILTER_VALIDATE_IP)) {
throw new UnsupportedOperation(sprintf('The %s provider does not support IP addresses, only street addresses.', $this->getName()));
}
$data = [
'text' => $address,
'size' => $query->getLimit(),
'layers' => null !== $query->getData('layers') ? implode(',', $query->getData('layers')) : null,
'boundary.country' => null !== $query->getData('boundary.country') ? implode(',', $query->getData('boundary.country')) : null,
];
return sprintf('%s/search?%s', $this->root, http_build_query(array_merge($data, $query_data)));
}
- Lenguaje dominante
- PHP
- Estrellas
- 4k
- Forks
- 525
- Merge medio
- 8 min
- PR fusionados (30 d)
- 1
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de geocoder-php/Geocoder
-
Wrong LocationIQ Bounds Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
geocoder-php/Geocoder#1272 ·
-
question
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
geocoder-php/Geocoder#1267 · 2 comentarios ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 38/100
geocoder-php/Geocoder#1244 · 2 comentarios · 2 reacciones ·
-
Add Support for IPQuery Abiertoprovider
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
geocoder-php/Geocoder#1243 ·
-
Mapbox Geocoding v6 Abiertoprovider
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
geocoder-php/Geocoder#1237 · 1 comentario · 1 reacción ·
Todos los issues de geocoder-php/Geocoder
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
getgrav/grav-plugin-api#45 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
RSS-Bridge/rss-bridge#5098 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
phingofficial/phing#2025 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
silverstripe/developer-docs#911 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100