Closed (fixed)
Project:
EU Cookie Compliance (GDPR Compliance)
Version:
6.x-1.2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 May 2012 at 08:18 UTC
Updated:
22 May 2012 at 23:54 UTC
in eu_cookie_compliance.js, function Drupal.eu_cookie_compliance.getCurrentStatus, where it says if (offset < 1) should be if (offset < 0)
if the user consent cookie is the first cookie, then the offset will be 0, as a result the function will always return 0.
var search = 'cookie-agreed-'+Drupal.settings.eu_cookie_compliance.popup_language+'=';
var offset = document.cookie.indexOf(search);
if (offset < 1) {
return 0;
}
Comments
Comment #1
acouch commentedI had the same problem. Not sure how it is working on the demo site with that in there. Instead of fixing it as you described above I copied and renamed
Drupal.comment.getCookie().to get the cookie value and replaced the logic above. See #1593568: Let All Langauges Be Administrated in One Screen, Don't Make Users Agree in More than 1 Language, and Code CleanupComment #2
Miszel commentedThis is certainly a bug. The code has now been corrected in 6.x-1.x. Thanks for reporting it.
Comment #3
Miszel commentedFixed in 6.x-1.3