-
Notifications
You must be signed in to change notification settings - Fork 109
chore: remove unused + bump dev dependencies #380
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
c64a526
099a4ed
2376a16
92a7fa7
157976f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,211 +1,197 @@ | ||||||
| <?php | ||||||
|
|
||||||
| declare(strict_types=1); | ||||||
| <?php declare(strict_types = 1); | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix the declare statement formatting. The space around 🔎 Proposed fix-<?php declare(strict_types = 1);
+<?php declare(strict_types=1);Based on static analysis hints. 📝 Committable suggestion
Suggested change
🧰 Tools🪛 GitHub Check: PHP-CS-Fixer[warning] 1-1: [warning] 1-1: [warning] 1-1: 🤖 Prompt for AI Agents |
||||||
|
|
||||||
| $ignoreErrors = []; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$name of method Geocoder\\\\ProviderAggregator\\:\\:using\\(\\) expects string, mixed given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/Command/GeocodeCommand.php', | ||||||
| 'path' => __DIR__ . '/src/Command/GeocodeCommand.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$string of function strtolower expects string, string\\|null given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/Command/GeocodeCommand.php', | ||||||
| 'path' => __DIR__ . '/src/Command/GeocodeCommand.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$text of static method Geocoder\\\\Query\\\\GeocodeQuery\\:\\:create\\(\\) expects string, mixed given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/Command/GeocodeCommand.php', | ||||||
| 'path' => __DIR__ . '/src/Command/GeocodeCommand.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#3 \\.\\.\\.\\$values of function sprintf expects bool\\|float\\|int\\|string\\|null, mixed given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__ . '/src/Command/GeocodeCommand.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: foreach.nonIterable | ||||||
| 'message' => '#^Argument of an invalid type mixed supplied for foreach, only iterables are supported\\.$#', | ||||||
| 'count' => 3, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: offsetAccess.nonOffsetAccessible | ||||||
| 'message' => '#^Cannot access offset \'aliases\' on mixed\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: offsetAccess.nonOffsetAccessible | ||||||
| 'message' => '#^Cannot access offset \'enabled\' on mixed\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: offsetAccess.nonOffsetAccessible | ||||||
| 'message' => '#^Cannot access offset \'factory\' on mixed\\.$#', | ||||||
| 'count' => 5, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: offsetAccess.nonOffsetAccessible | ||||||
| 'message' => '#^Cannot access offset \'id\' on mixed\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: offsetAccess.nonOffsetAccessible | ||||||
| 'message' => '#^Cannot access offset \'options\' on mixed\\.$#', | ||||||
| 'count' => 4, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: offsetAccess.nonOffsetAccessible | ||||||
| 'message' => '#^Cannot access offset \'reference\' on mixed\\.$#', | ||||||
| 'count' => 2, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: staticMethod.nonObject | ||||||
| 'message' => '#^Cannot call static method validate\\(\\) on mixed\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: cast.string | ||||||
| 'message' => '#^Cannot cast mixed to string\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$alias of method Symfony\\\\Component\\\\DependencyInjection\\\\ContainerBuilder\\:\\:setAlias\\(\\) expects string, mixed given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$callback of function array_map expects \\(callable\\(mixed\\)\\: mixed\\)\\|null, Closure\\(string\\)\\: Symfony\\\\Component\\\\DependencyInjection\\\\Reference given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$factoryServiceId of static method Bazinga\\\\GeocoderBundle\\\\DependencyInjection\\\\Compiler\\\\FactoryValidatorPass\\:\\:addFactoryServiceId\\(\\) expects string, mixed given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$id of class Symfony\\\\Component\\\\DependencyInjection\\\\Reference constructor expects string, mixed given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$id of method Symfony\\\\Component\\\\DependencyInjection\\\\ContainerBuilder\\:\\:getDefinition\\(\\) expects string, mixed given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$object_or_class of function class_implements expects object\\|string, mixed given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$options of method Bazinga\\\\GeocoderBundle\\\\DependencyInjection\\\\BazingaGeocoderExtension\\:\\:findReferences\\(\\) expects array\\<int\\|string, mixed\\>, mixed given\\.$#', | ||||||
| 'count' => 2, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$string of function ltrim expects string, mixed given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#2 \\$config of method Bazinga\\\\GeocoderBundle\\\\DependencyInjection\\\\BazingaGeocoderExtension\\:\\:configureProviderPlugins\\(\\) expects array\\<int\\|string, mixed\\>, mixed given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#2 \\.\\.\\.\\$values of function sprintf expects bool\\|float\\|int\\|string\\|null, mixed given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: encapsedStringPart.nonString | ||||||
| 'message' => '#^Part \\$providerName \\(mixed\\) of encapsed string cannot be cast to string\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| 'path' => __DIR__ . '/src/DependencyInjection/BazingaGeocoderExtension.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: method.nonObject | ||||||
| 'message' => '#^Cannot call method getLatitude\\(\\) on Geocoder\\\\Model\\\\Coordinates\\|null\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/Doctrine/ORM/GeocoderListener.php', | ||||||
| 'path' => __DIR__ . '/src/Doctrine/ORM/GeocoderListener.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: method.nonObject | ||||||
| 'message' => '#^Cannot call method getLongitude\\(\\) on Geocoder\\\\Model\\\\Coordinates\\|null\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/Doctrine/ORM/GeocoderListener.php', | ||||||
| 'path' => __DIR__ . '/src/Doctrine/ORM/GeocoderListener.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$text of method Geocoder\\\\Query\\\\GeocodeQuery\\:\\:withText\\(\\) expects string, string\\|null given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/Plugin/FakeIpPlugin.php', | ||||||
| 'path' => __DIR__ . '/src/Plugin/FakeIpPlugin.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#2 \\$replace of function str_replace expects array\\|string, string\\|null given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/Plugin/FakeIpPlugin.php', | ||||||
| 'path' => __DIR__ . '/src/Plugin/FakeIpPlugin.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$accountId of class GeoIp2\\\\WebService\\\\Client constructor expects int, int\\|null given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/ProviderFactory/GeoIP2Factory.php', | ||||||
| 'path' => __DIR__ . '/src/ProviderFactory/GeoIP2Factory.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$filename of class GeoIp2\\\\Database\\\\Reader constructor expects string, string\\|null given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/ProviderFactory/GeoIP2Factory.php', | ||||||
| 'path' => __DIR__ . '/src/ProviderFactory/GeoIP2Factory.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#1 \\$geoIpProvider of class Geocoder\\\\Provider\\\\GeoIP2\\\\GeoIP2Adapter constructor expects GeoIp2\\\\ProviderInterface, GeoIp2\\\\ProviderInterface\\|null given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/ProviderFactory/GeoIP2Factory.php', | ||||||
| 'path' => __DIR__ . '/src/ProviderFactory/GeoIP2Factory.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: argument.type | ||||||
| 'message' => '#^Parameter \\#2 \\$licenseKey of class GeoIp2\\\\WebService\\\\Client constructor expects string, string\\|null given\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/ProviderFactory/GeoIP2Factory.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: class.notFound | ||||||
| 'message' => '#^Class Geocoder\\\\Provider\\\\Geoip\\\\Geoip not found\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/ProviderFactory/GeoipFactory.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: class.notFound | ||||||
| 'message' => '#^Instantiated class Geocoder\\\\Provider\\\\Geoip\\\\Geoip not found\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/ProviderFactory/GeoipFactory.php', | ||||||
| ]; | ||||||
| $ignoreErrors[] = [ | ||||||
| // identifier: return.type | ||||||
| 'message' => '#^Method Bazinga\\\\GeocoderBundle\\\\ProviderFactory\\\\GeoipFactory\\:\\:getProvider\\(\\) should return Geocoder\\\\Provider\\\\Provider but returns Geocoder\\\\Provider\\\\Geoip\\\\Geoip\\.$#', | ||||||
| 'count' => 1, | ||||||
| 'path' => __DIR__.'/src/ProviderFactory/GeoipFactory.php', | ||||||
| 'path' => __DIR__ . '/src/ProviderFactory/GeoIP2Factory.php', | ||||||
| ]; | ||||||
|
|
||||||
| return ['parameters' => ['ignoreErrors' => $ignoreErrors]]; | ||||||
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Algolia Places API is no longer functional.
The Algolia Places API shut down on May 31st, 2022. While the geocoder-php/algolia-places-provider package still exists and receives maintenance updates, the underlying service is non-functional. If this provider is retained solely for testing the bundle's integration patterns, this is acceptable. Otherwise, this dependency should be removed and replaced with an alternative geocoding provider.