Bug when use chain and GeoIp2 at first position with cache
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/100
Hướng nghiên cứu
Bắt đầu với các phương thức executeQuery và geocodeQuery của provider GeoIp2, sau đó theo dõi cách Chain::geocodeQuery xử lý một collection rỗng từ provider đầu tiên và việc đọc cache cho các provider tiếp theo. Tái hiện một chain với GeoIp2 đứng đầu và một provider tùy chỉnh đứng thứ hai; hoàn tất khi một phản hồi rỗng từ GeoIp2 không còn ngăn provider tiếp theo được sử dụng và hành vi lỗi dự kiến đã được xác minh.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
GeoIp2 return empty collection and save it in cache after that next providers(custom) load from cache empty collection instead of using his code
$result = json_decode($this->executeQuery($address));
if (null === $result) {
return new AddressCollection([]);
}
$result = json_decode(''); //NULL
private function executeQuery(string $address): string
{
$uri = sprintf('file://geoip?%s', $address);
try {
$result = $this->adapter->getContent($uri);
} catch (AddressNotFoundException $e) {
return '';
} catch (AuthenticationException $e) {
throw new InvalidCredentials(
$e->getMessage(),
$e->getCode(),
$e
);
} catch (OutOfQueriesException $e) {
throw new QuotaExceeded(
$e->getMessage(),
$e->getCode(),
$e
);
}
return $result;
}
I think there should not be return '';, but there should be fatal
public function geocodeQuery(GeocodeQuery $query): Collection
{
foreach ($this->providers as $provider) {
try {
$result = $provider->geocodeQuery($query);
if (!$result->isEmpty()) {
return $result;
}
} catch (\Throwable $e) {
$this->log(
'alert',
sprintf('Provider "%s" could geocode address: "%s".', $provider->getName(), $query->getText()),
['exception' => $e]
);
}
}
return new AddressCollection();
}
\Geocoder\Provider\Chain\Chain::geocodeQuery now always return empty collectioon \Geocoder\Model\AddressCollection();
- Ngôn ngữ chính
- PHP
- Star
- 4k
- Fork
- 525
- Merge trung bình
- 8 phút
- Pull request đã merge (30 ngày)
- 1
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của geocoder-php/Geocoder
-
Wrong LocationIQ Bounds Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 78/100
geocoder-php/Geocoder#1272 ·
-
question
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
geocoder-php/Geocoder#1267 · 2 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
geocoder-php/Geocoder#1266 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 38/100
geocoder-php/Geocoder#1244 · 2 bình luận · 2 reaction ·
-
Add Support for IPQuery Đang mởprovider
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 30/100
geocoder-php/Geocoder#1243 ·
Tất cả issue của geocoder-php/Geocoder
Issue tương tự
-
jira-created
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 70/100
nunomaduro/phpinsights#745 ·
-
status/awaiting_triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
WordPress/plugin-check#1486 ·
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
az-digital/az_quickstart#6019 ·