Problem/Motivation

eu_cookie_compliance.api.php line 27 is$geoip_match = FALSE;
I think it should be $geoip_match['in_eu'] = FALSE;

CommentFileSizeAuthor
#2 eu_cookie_compliance-geoip_match-wrongly-set-in-eu_cookie_compliance-api-php-3184478-2-8.x.patch388 bytessvenryen
#2 eu_cookie_compliance-geoip_match-wrongly-set-in-eu_cookie_compliance-api-php-3184478-2-7.x.patch388 bytessvenryen

Comments

oppure created an issue. See original summary.

svenryen’s picture

svenryen commented
StatusFileSize
new eu_cookie_compliance-geoip_match-wrongly-set-in-eu_cookie_compliance-api-php-3184478-2-7.x.patch388 bytes
new eu_cookie_compliance-geoip_match-wrongly-set-in-eu_cookie_compliance-api-php-3184478-2-8.x.patch388 bytes

Thanks for catching that.
Here's a patch to resolve the issue. Both d7 and d8 were affected.

  • svenryen committed 42c23da on 8.x-1.x 
    Issue #3184478 by svenryen, oppure: $geoip_match wrongly set in...

  • svenryen committed e4364d8 on 7.x-1.x 
    Issue #3184478 by svenryen, oppure: $geoip_match wrongly set in...
svenryen’s picture

svenryen commented
Status: Active » Fixed

Given that there's no executable code affected, this should require no review.

Status: Fixed » Closed (fixed)

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