Skip to content

Commit

Permalink
Merge pull request #56 from wpengine/nojira-fix-continent-notice
Browse files Browse the repository at this point in the history
Check for existing countrycode before assigning it
  • Loading branch information
Ryan Hoover committed Aug 18, 2020
2 parents 61a98b2 + 0eafd66 commit 23127b7
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/class-geoip.php
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<?php
/**
* Plugin Name: WP Engine GeoTarget
* Version: 1.2.7
* Version: 1.2.8
* Description: Create a personalized user experienced based on location.
* Author: WP Engine
* Author URI: http://wpengine.com
Expand Down Expand Up @@ -245,7 +245,7 @@ public function continent( $country = '' ) {

$continent = '';

if ( empty( $country ) ) {
if ( empty( $country ) && ! empty( $this->geos['countrycode'] ) ) {
$country = $this->geos['countrycode'];
}

Expand Down
8 changes: 6 additions & 2 deletions src/readme.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
Contributors: wpengine, markkelnar, stevenkword, stephenlin, ryanshoover, taylor4484, nateinaction
Tags: wpe, wpengine, geotarget, geoip, localization, geolocation
Requires at least: 3.0.1
Tested up to: 5.2
Stable tag: 1.2.7
Tested up to: 5.5
Stable tag: 1.2.8

License: GPLv2 or later
License URI: http://www.gnu.org/licenses/gpl-2.0.html
Expand Down Expand Up @@ -263,6 +263,10 @@ Please contact the WP Engine [Support Team](https://my.wpengine.com/support).

== Changelog ==

= 1.2.8 =
- Fix for development sites that call the `continent` method
- Bump for WP 5.5 compatibility

= 1.2.7 =
- Optimize plugin name rename loop
- Bump for WP 5.2 compatibility
Expand Down

0 comments on commit 23127b7

Please sign in to comment.