Merge pull request #386 from Chris53897/feature/remove-maxmind-binary-provider

feat: remove MaxMindBinary Provider

Author: norkunas

composer.json

@@ -46,7 +46,6 @@
     "geocoder-php/locationiq-provider": "^1.4",
     "geocoder-php/mapbox-provider": "^1.4",
     "geocoder-php/mapquest-provider": "^4.3",
-    "geocoder-php/maxmind-binary-provider": "^4.4",
     "geocoder-php/maxmind-provider": "^4.5",
     "geocoder-php/nominatim-provider": "^5.8",
     "geocoder-php/open-cage-provider": "^4.7",

doc/services.md

@@ -29,7 +29,6 @@ Here is a list of all provider factories and their options.
 | `Bazinga\GeocoderBundle\ProviderFactory\MapboxFactory` | http_client, api_key, country, mode
 | `Bazinga\GeocoderBundle\ProviderFactory\MapQuestFactory` | http_client, api_key, licensed
 | `Bazinga\GeocoderBundle\ProviderFactory\MapzenFactory` | http_client, api_key
-| `Bazinga\GeocoderBundle\ProviderFactory\MaxMindBinaryFactory` | dat_file, open_flag
 | `Bazinga\GeocoderBundle\ProviderFactory\MaxMindFactory` | http_client, api_key, endpoint
 | `Bazinga\GeocoderBundle\ProviderFactory\NominatimFactory` | http_client, root_url
 | `Bazinga\GeocoderBundle\ProviderFactory\OpenCageFactory` | http_client, api_key

src/ProviderFactory/MaxMindBinaryFactory.php

@@ -18,7 +18,6 @@
 use Geocoder\Provider\BingMaps\BingMaps;
 use Geocoder\Provider\Chain\Chain;
 use Geocoder\Provider\FreeGeoIp\FreeGeoIp;
-use Geocoder\Provider\Geoip\Geoip;
 use Geocoder\Provider\GeoIP2\GeoIP2;
 use Geocoder\Provider\Geonames\Geonames;
 use Geocoder\Provider\GeoPlugin\GeoPlugin;
@@ -33,7 +32,6 @@
 use Geocoder\Provider\Mapbox\Mapbox;
 use Geocoder\Provider\MapQuest\MapQuest;
 use Geocoder\Provider\MaxMind\MaxMind;
-use Geocoder\Provider\MaxMindBinary\MaxMindBinary;
 use Geocoder\Provider\Nominatim\Nominatim;
 use Geocoder\Provider\OpenCage\OpenCage;
 use Geocoder\Provider\PickPoint\PickPoint;
@@ -103,7 +101,6 @@ public function getProviders(): iterable
         yield [BingMaps::class, ['acme']];
         yield [Chain::class, ['acme']];
         yield [FreeGeoIp::class, ['empty', 'acme']];
-        // yield [Geoip::class, ['empty']];
         yield [GeoIP2::class, ['acme']];
         yield [Geonames::class, ['acme']];
         yield [GeoPlugin::class, ['empty']];
@@ -118,7 +115,6 @@ public function getProviders(): iterable
         yield [Mapbox::class, ['acme']];
         yield [MapQuest::class, ['acme']];
         yield [MaxMind::class, ['acme']];
-        yield [MaxMindBinary::class, ['acme']];
         yield [Nominatim::class, ['empty', 'acme']];
         yield [OpenCage::class, ['acme']];
         yield [PickPoint::class, ['acme']];

tests/Functional/ProviderFactoryTest.php

@@ -16,6 +16,3 @@
 php composer.phar install --dev --prefer-source
 ');
 }
-
-require_once __DIR__.'/../vendor/geoip/geoip/src/geoip.inc';
-require_once __DIR__.'/../vendor/geoip/geoip/src/geoipcity.inc';