Hi,
we activated the module today on our site, works great with geoip. We followed the instructions and everything went smooth. Tested it from non-EU countries with proxies, everything is A-OK.
The problem is that even though we switched the interface to Greek and enabled it for the Greek language as well, the popup never shows up in Greek, which is the second language of our site.
The URL changes from :
http://www.domain.com to http://www.domain.com/gr when in Greek.
What should we check? Did anything like that happen to anyone else? Is there any report for this?
PS. We don't know if this is important, but for the user to view the site in Greek, he Clicks on "Greek" in the Language Switcher.
Also, we noticed that IF we try to login in Greek language, using a like URL http://www.domain.com/gr/user then the popup IS displayed(!).
The same goes (it works) if we try URLs like http://www.domain.com/gr/aboutus.
It DOESN'T display, if we go to the default Greek URL (/gr) (Greek 'homepage') AND if we click on ANY of our menu options that lead to a Views page (like /newest or /videoselections etc).
PS2. The strange behaviour was originating from caching. We had caching enabled for anonymous users. Once removed, it works. But it doesn't work if we click to use geoip along with Greek. Maybe the geoip integration+2nd language should be tested.
Comment | File | Size | Author |
---|---|---|---|
#11 | 2163815_eu_cookie_compliance_language_settings_11.patch | 2.94 KB | MrHaroldA |
Comments
Comment #1
littlecookingtips CreditAttribution: littlecookingtips commentedComment #2
littlecookingtips CreditAttribution: littlecookingtips commentedComment #3
littlecookingtips CreditAttribution: littlecookingtips commentedComment #4
littlecookingtips CreditAttribution: littlecookingtips commentedComment #5
littlecookingtips CreditAttribution: littlecookingtips commentedComment #6
svenryen CreditAttribution: svenryen commentedThis is a known issue with admin language. If your admin is in English, you will never see the settings for Greek, and it will not be enabled for Greek. The current version. 7.12, can't reliably handle multilingual sites, and this will be addressed in 7.13.
Comment #7
svenryen CreditAttribution: svenryen commentedUpdate, this issue will be fixed in version 2.0
Comment #8
MrHaroldA CreditAttribution: MrHaroldA commentedYeah ... this is quite a major drawback for us (and everybody else using the 'locale' module), so I've patched 1.x-dev to support editing settings other than the current user's language.
Not being able to edit the settings could even be marked as 'critical' as it breaks this module completely.
Comment #9
svenryen CreditAttribution: svenryen commentedI feel your pain, as I've been through the same problem several times. When workload allows, I'll try out your patch and update the module. Thank you for the patch! :)
Comment #10
MrHaroldA CreditAttribution: MrHaroldA commentedWhoops ... found a validation handler error ;)
Fixed it by storing the language object in the form.
Comment #11
MrHaroldA CreditAttribution: MrHaroldA commentedThis is going to be a sh*tty day if I keep failing like this ;)
I've unset the language form element in the validate function to prevent storing it as variable.
Comment #12
interdruper CreditAttribution: interdruper commented#11 works fine for me.
I think that this issue is a duplicate of #2166465, where the patch has a different approach.
This issue is relevant for the common multilingual sites that use English as admin language. Workaround hinted in https://www.drupal.org/node/2166465#comment-8788315 didn't work for me.
Comment #13
candelas CreditAttribution: candelas commented@MrHaroldA I tried your patch, but it keeps not showing the pop-up in the other language. I go to try the other hints.
Comment #14
candelas CreditAttribution: candelas commentedComment #15
Nchase CreditAttribution: Nchase commentedthe patch shows the possibility to switch between the languages. That is nice and appreciated.
After cache clear everything works fine. Patch in #11 works.
Comment #16
StephaneQ#11 works for me too
Comment #17
TuMMa CreditAttribution: TuMMa commentedWe use English language for admin side and Italian for anonymous users, i tried to use module of version 1.14 and even 7.x-1.x-dev, but it still doesn't show pop-up for anonymous users (permissions set correctly). And only if i set English as default language it shows pop-up for not logged in users. Can someone advice me how to solve the problem?
Comment #18
TuMMa CreditAttribution: TuMMa commentedComment #19
svenryen CreditAttribution: svenryen commentedHi Tumma! We're working on a better interface for managing multi languages in the same form. Expect this to be done in late May as I'm currently traveling. You need to edit the module settings also for Italian, and enable the popup. To do this, please set your admin language to Italian. After you save your EU Cookie Compliance admin form in Italian, you may change it back to English. Hope this helps.
Comment #20
TuMMa CreditAttribution: TuMMa commented@svenryen thank you it helped! :) but also i had to reinstall module, i hope it'll help someone else.
P.S. Have a good journey!
Comment #21
Gik000 CreditAttribution: Gik000 as a volunteer commentedThat helped me! Ty :)
Comment #22
nicolap CreditAttribution: nicolap commentedNote that the patch #11 is already included in ver 1.14.
But I still have problems with second language!
Comment #23
svenryen CreditAttribution: svenryen commentednicolap, have you tried this for your second language? Set your admin language to {second language}. After you save your EU Cookie Compliance admin form in {second language}, you may change it back to English. Hope this helps.
Comment #24
nicolap CreditAttribution: nicolap commentedYes!
I have two administrators, one with english language as default and the other with italian (the second and default language).
I do the settings for both language but the popup is displayed only for english.
Tried to change the current language temporary with "?language=it".
Tried to change the admin language.
Tried to change various options (position, thanks, etc).
Never worked for second language, also if specified with "?language=it"!
In the database there are the settings for both language (it seems ok).
Comment #25
svenryen CreditAttribution: svenryen commentedDid you check off the checkbox at the top to enable the module when you edited the italian settings?
Comment #26
nicolap CreditAttribution: nicolap commentedYes, of course ;-))
Variables:
eu_cookie_compliance_domain
s:0:"";
eu_cookie_compliance_en
a:21:{s:13:"popup_enabled";i:1;s:27:"popup_clicking_confirmation";i:1;s:14:"popup_position";i:0;s:26:"popup_agree_button_message";s:11:"OK, I agree";s:29:"popup_disagree_button_message";s:21:"No, give me more info";s:10:"popup_info";a:2:{s:5:"value";s:159:"
We use cookies on this site to enhance your user experience
By clicking any link on this page you are giving your consent for us to set cookies.
";s:6:"format";s:13:"filtered_html";}s:20:"popup_agreed_enabled";i:0;s:17:"popup_hide_agreed";i:0;s:30:"popup_find_more_button_message";s:9:"More info";s:25:"popup_hide_button_message";s:4:"Hide";s:12:"popup_agreed";a:2:{s:5:"value";s:108:"
Thank you for accepting cookies
You can now hide this message or find out more about cookies.
";s:6:"format";s:13:"filtered_html";}s:10:"popup_link";s:16:"www.sitename.it";s:21:"popup_link_new_window";i:0;s:12:"popup_height";s:0:"";s:11:"popup_width";s:4:"100%";s:11:"popup_delay";s:1:"1";s:12:"popup_bg_hex";s:6:"0779BF";s:14:"popup_text_hex";s:6:"ffffff";s:14:"domains_option";s:1:"1";s:12:"domains_list";s:0:"";s:13:"exclude_paths";s:0:"";}
eu_cookie_compliance_it
a:21:{s:13:"popup_enabled";i:1;s:27:"popup_clicking_confirmation";i:1;s:14:"popup_position";i:1;s:26:"popup_agree_button_message";s:11:"Si, accetto";s:29:"popup_disagree_button_message";s:31:"No, dammi maggiori informazioni";s:10:"popup_info";a:2:{s:5:"value";s:83:"
Questo sito fa uso di cookie, limitatamente al funzionamento indispensabile.
";s:6:"format";s:13:"filtered_html";}s:20:"popup_agreed_enabled";i:1;s:17:"popup_hide_agreed";i:1;s:30:"popup_find_more_button_message";s:21:"Maggiori informazioni";s:25:"popup_hide_button_message";s:8:"Nascondi";s:12:"popup_agreed";a:2:{s:5:"value";s:14:"
Grazie!
";s:6:"format";s:13:"filtered_html";}s:10:"popup_link";s:16:"www.sitename.it";s:21:"popup_link_new_window";i:1;s:12:"popup_height";s:0:"";s:11:"popup_width";s:4:"100%";s:11:"popup_delay";s:1:"1";s:12:"popup_bg_hex";s:6:"0779BF";s:14:"popup_text_hex";s:6:"ffffff";s:14:"domains_option";s:1:"0";s:12:"domains_list";s:0:"";s:13:"exclude_paths";s:0:"";}
Comment #27
svenryen CreditAttribution: svenryen commentedThat's really strange. Would you be okay sharing your database and files with me in a personal message so I can take a closer look? I'm not sure why it doesn't work on your Italian site.
Comment #28
epidrupal CreditAttribution: epidrupal commentedI've tried everything. Popup shows up only for default english. Four months with no solution.
Comment #29
ndobromirov CreditAttribution: ndobromirov commentedHi all,
I've also tested this (#11), on and English / Danish site with EN administration.
The patch provided easy setting of all the enabled languages and works like a charm.
The workaround would have been to go and change administration language from admin/config/regional/language and set settings then, switch to the next language and so on, but it is not user-friendly or intuitive, at all, even for developers.
UX improvement would be to have default values (or just use the default language ones for such) and just override them per language, as colors and behavior check-boxes are not likely to change etc., but this is nitpicking, so moving as RTBC, as the module is unusable in multilingual context without this patch.
BR,
Nikolay Dobromirov.
Comment #30
svenryen CreditAttribution: svenryen commentedHi ndobromirov. There is an upgrade in the works for multiple languages. Sorry that it's taking so long. I hope to get time to look more at it this week.
Comment #31
gingonap CreditAttribution: gingonap commentedI can confirm the issue: multilingual site in EN, DE, IT... but the popup shows up only for the english default language.
Comment #32
svenryen CreditAttribution: svenryen commentedgingonap, as a workaround, set your admin language to {second language}. After you save your EU Cookie Compliance admin form in {second language}, you may change it back to English. Hope this helps.
Comment #33
kinmen CreditAttribution: kinmen as a volunteer commentedSame here: 3 different languages and it works only with Eng, no matter I change the default language and setup the module again. Ver 1.14
Comment #34
svenryen CreditAttribution: svenryen commentedI'm finally able to work on this and will implement a solution within a week. Keep an eye on the dev branch.
Comment #35
svenryen CreditAttribution: svenryen commentedComment #36
ndobromirov CreditAttribution: ndobromirov commentedHi again,
I know this is old, but the old implementation idea was broken. The i18n module has variables support, so if it's integrated correctly, i think it can provide the localisation of the form out of the box, so no custom code needs to be really written in this module for supporting many languages... See the tutorial from Drupal.org.. Just add the localizable variables to a list of sort. I suspect there should be some info hook for this. This is just a guess, as I am not familiar with the API, but it can also be just fixed with a section in the README.txt like -> If you want translations for the variables do this, this and this for this variables:
, so the site builder can make this, if there is no API for it in i18n_variables.
This will impose a soft dependency to i18n_variables to the sites that need many languages, but as far as I know if you want more than 1 language, you are enabling the i18n module suite either way, so it should not be the overhead. This also gives a performance boost, because at the moment all the language configs are stored in as variables and loaded all the time. With i18n_variables you will load only the default English always and the current language ones additionally in the worst case, so this should lower memory usage (even if slightly).
Just my 2 cents for the new implantation.
I've placed the issue in RTBC before, because I was in a hurry and the patch was a solution at the time, but I know now that it's not the best one in terms of implementation, so back to needs work.
BR, Nick.
Comment #37
ilitur CreditAttribution: ilitur commentedI have the same problem.
Everything is ok with english language but doesnt' works with italian.
@gingonap hai poi risolto?
Comment #38
svenryen CreditAttribution: svenryen commentedThis issue has been fixed in 1.15-beta1. We now have a separate tab per language.