August 9

ozh wants to merge geoip2 into master

Update GeoIP to GeoIP2

Current Status:

Analysed
View on GitHub
Good work!

A positive pull request.

  • +1 Issues
  • - Duplication
  • = Complexity
  • - Coverage
includes/functions.php
Avoid variables with short names like $db. Configured minimum length is 3.
$db = yourls_apply_filter('geo_ip_path_to_db', YOURLS_INC.'/geo/GeoLite2-Country.mmdb');
More
includes/functions.php
Avoid variables with short names like $db. Configured minimum length is 3.
$db = 'GeoIP.dat';
Avoid variables with short names like $gi. Configured minimum length is 3.
$gi = geoip_open( YOURLS_INC . '/geo/' . $db, GEOIP_STANDARD );
Avoid unused local variables such as '$temp'.
$temp = yourls_geo_ip_to_countrycode( '127.0.0.1' );
Avoid variables with short names like $id. Configured minimum length is 3.
$id = $geo->GEOIP_COUNTRY_CODE_TO_NUMBER[ $code ];
More

Loading...

Loading...

Loading...

Loading...