search

Zakay / geoip

A more complete fork of PECL MaxMind geoip extension
66 stars 20 forks source link

GeoIP Object #3

Open Zakay opened 13 years ago

Zakay commented 13 years ago

Current Requirements:

katanacrimson commented 13 years ago

Digging around on PHP.net, found this issue that it might be worth working around: http://us2.php.net/manual/en/geoip.setup.php#103742

When setting the directory, making sure the files to be used are readable could save some sanity - instead of the developer having to call geoip_db_avail themselves, just throw an exception if the database file(s) are unusable. After all, if the app developer is instantiating an object to load up the geoip database, they're not doing it for grins, it's because they're going to use it in some fashion (get the db info, look something up, whatever).

Zakay commented 13 years ago

Agreed.