Geoips not working (api change)

[Thinkscape]

It seems that geoips format has changed... the provider is not working anymore.

Here's an example structure of a result:

[Thinkscape]

Yeah... it changed: http://www.geoips.com/en/developer/api-guide

[willdurand]

Ah.. Bad luck.. Could you come up with a patch?

[Thinkscape]

I'll take care of that mate :-)

[willdurand]

Thank you!

[willdurand]

poke @Thinkscape

[Thinkscape]

ugh.

Still here :-) sorry, got distracted.

[willdurand]

poke (again) @Thinkscape

[Thinkscape]

Interesting ... I can't seem to install deps because:

Loading composer repositories with package information
Installing dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for ext-geoip 1.0.8 -> satisfiable by ext-geoip[1.0.8].
    - geoip/geoip v1.13 conflicts with ext-geoip[1.0.8].
    - Installation request for geoip/geoip ~1.13 -> satisfiable by geoip/geoip[v1.13].

Did I miss something ? Why does geoip/geoip conflict with the extension ?

[willdurand]

doh! I have no clue yet..

[Thinkscape]

$ php --ri geoip

geoip

geoip support => enabled
geoip extension version => 1.0.8
geoip library version => 1004005

Directive => Local Value => Master Value
geoip.custom_directory => no value => no value

[Thinkscape]

Meh...

PHP Fatal error:  Cannot redeclare geoip_country_code_by_name() in /Projects/Geocoder/vendor/geoip/geoip/geoip.inc on line 439

Fatal error: Cannot redeclare geoip_country_code_by_name() in /Projects/Geocoder/vendor/geoip/geoip/geoip.inc on line 439

That's why it conflicts. Apparently it declares the same set of functions. I believe a fixup of composer.json is in place (so it used either of those, instead of trying to install both)

[willdurand]

Ah yes... There is a lib that mimics the API of the geoip ext

[Thinkscape]

Wait... the format changes depending on query ... ugh. Hold on. I'll get to the bottom of this.

[Thinkscape]

There ya go!