Problem/Motivation

The UK is flagged as an EU country, which is not the case anymore. This is in the function eu_cookie_compliance_user_in_eu
I'd be happy to correct it myself, I just do not know how to connect to your source code repository and how to connect it. I believe it would take you 3 seconds to correct it.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#6 3325342-6.patch1.49 KBmahima_mathur23
#3 3325342-3.patch678 bytes_pratik_

Comments

pureh2o created an issue. See original summary.

_pratik_’s picture

_pratik_
Primary language English
Location Indore
commented
Assigned: Unassigned » _pratik_
_pratik_’s picture

_pratik_
Primary language English
Location Indore
commented
Assigned: _pratik_ » Unassigned
StatusFileSize
new 3325342-3.patch678 bytes

Please check this.
Thanks

karanpagare’s picture

karanpagare commented
Status: Active » Reviewed & tested by the community

Patch #3 applies cleanly and UK is removed from eu_cookie_compliance_user_in_eu function

svenryen’s picture

svenryen commented
Status: Reviewed & tested by the community » Needs work

Thanks for the report and the patch. The UK still has the same cookie legislation as they had during their EU membership. Until that's changed I think it makes sense to have the country included, but instead rename the feature to "EU, EEC countries and the UK" rather than just "EU".

You can for example see that the Guardian still displays a cookie banner when you access the site from a UK ip address.

mahima_mathur23’s picture

mahima_mathur23 commented
StatusFileSize
new 3325342-6.patch1.49 KB

Attaching the patch for changing the setting page to display EU, EEC countries and the UK rather than only EU countries.
As it is in Drupal 7 whose support will end in this year, I haven't made much changes, just some significantly required ones.

mahima_mathur23’s picture

mahima_mathur23 commented
Status: Needs work » Needs review

svenryen’s picture

svenryen commented
Status: Needs review » Fixed

Thanks for the review and patch.

Status: Fixed » Closed (fixed)

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