When I create a rule with the condition "Check for user country detected by ip", the list is blank.

CommentFileSizeAuthor
#2 geoip-rules-condition-country-not-listing-2671242-2-7.x.patch943 bytesjonloh

Comments

jonloh created an issue. See original summary.

jonloh’s picture

jonloh commented
Status: Active » Needs review
StatusFileSize
new geoip-rules-condition-country-not-listing-2671242-2-7.x.patch943 bytes

Upon checking through the code, the condition is calling the Drupal's built-in country_get_list function - https://api.drupal.org/api/drupal/includes%21locale.inc/function/country...

But since it requires to include_once locale.inc file, I believe it's better to create as a separate function itself in geoip to do the necessary function calls and return the country list.

The patch attached should fix this. Hopefully it's the right way to do so.

cutmedia’s picture

cutmedia commented
Related issues: +#2645768: 500 internal server error on context creation/editing when upgrading GeoIP API to 7.x-2.0-beta1

this worked for me, also the blank list breaks integration with Context GeoIP see related issue

hanskuiters’s picture

hanskuiters commented
Status: Needs review » Reviewed & tested by the community

Works fine, thank you.

  • das-peter committed 8f712f5 on 7.x-2.x authored by jonloh 
    Issue #2671242 by jonloh, cutmedia, HansKuiters: Rules condition "Check...
das-peter’s picture

das-peter commented
Status: Reviewed & tested by the community » Fixed

Looks good - thanks all for the work.
Patched and pushed :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.